Skip to content
This repository has been archived by the owner on Feb 19, 2021. It is now read-only.

I2CSlaveDevice.write16() がwrite8と同じ動作になる #152

Closed
2 tasks done
tadfmac opened this issue Nov 2, 2016 · 5 comments
Closed
2 tasks done

I2CSlaveDevice.write16() がwrite8と同じ動作になる #152

tadfmac opened this issue Nov 2, 2016 · 5 comments

Comments

@tadfmac
Copy link

tadfmac commented Nov 2, 2016

2箇所の実装によるもののようです。

 関連 issue

  • 2. Geckoに実装されているI2cService.WriteWord()(I2cService.h)(I2cService.cpp)の第3引数の型がuint8_tとなっており、2byteのデータが渡せない

@naobsd さんに2. の修正(上記I2cService.WtiteWord()の第3引数をuint16_tに変更)ビルドをご協力頂き、1. も併せて修正することでwrite16()が利用できるようになる(Polyfillを利用したアプリからI2CSlaveDevice.write16()を呼び出すことで対象のテストデバイスまでwriteコマンドでレジスタ1byte+データ2byte計3byte書き込める)ことを確認させて頂きました。@naobsd さんご協力ありがとうございます。

@tadfmac
Copy link
Author

tadfmac commented Nov 2, 2016

先に 2. からPull Requestを発行させていただく予定でおります。
chirimen-oh/gecko-dev#3
こちらに登録させていただきました。

@tadfmac
Copy link
Author

tadfmac commented Nov 3, 2016

そして、Polyfill側の修正についてもPRを発行させていただきました。
club-wot/WebGPIO#21

※ビルド方法がわからず、@gurezo さんに多大なるご協力を頂きました。 この場を借りて御礼申し上げます。

@MSakamaki
Copy link
Member

club-wot/WebGPIO#21

こちら確認してマージしておきました、最新に適用済みです。

@tadfmac
Copy link
Author

tadfmac commented Nov 4, 2016

@MSakamaki さん、Polyfill側修正反映いただき、ありがとうございます!
本issueは、polyfill側とgecko側の2箇所直す必要がありますので、下記PR反映まで本issueはOpenのままとさせてください。
chirimen-oh/gecko-dev#3
よろしくお願いします。

@tadfmac tadfmac reopened this Nov 4, 2016
@MSakamaki
Copy link
Member

MSakamaki commented Nov 4, 2016

@tacoyaky すいません間違えてこちらもcloseしちゃってました、了解です。

@tadfmac tadfmac reopened this Nov 4, 2016
dadaa added a commit to chirimen-oh/gecko-dev that referenced this issue Nov 4, 2016
@ghost ghost added this to the 201702xx release milestone Feb 5, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants