-
Notifications
You must be signed in to change notification settings - Fork 456
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
rtmouse.cの分割(リファクタリング) #92
Conversation
Raspberry Pi 4B(4GB)上のUbuntu Server 24.04(6.8.0-1013-raspi)でビルドとサンプルプログラムの動作を確認しました。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ソースコード中に4箇所(内容は同じ)のコメントをしました。
既に実装された関数に関するもので、今回のPRの本質とは少しばかりずれていますが、不具合を引き起こす可能性があると思ったためコメントしています。今回のPRで修正するべきでないと思った場合は無視してももらってもかまいません。その場合は、こちらで別途PRを作成します。
動作確認ありがとうございます。 |
承知しました。 |
What does this implement/fix?
すべての処理が
rtmouse.c
に記述されているため、メンテナンスしやすいようにファイルを分割します。主な変更内容は下記の通りです。
rtmouse.c
はrtmouse_main.c
へ名称変更rtmouse_main.c
には変数の定義と主要な関数の定義を記述rtmouse_dev.c
にデバイス関連の関数を記述rtmouse_spi.c
にSPI関連の関数を記述rtmouse_i2c.c
にI2C関連の関数を記述rtmouse_gpio.c
にGPIO関連の関数を記述rtmouse.h
で外部変数をexternで宣言Makefile
を修正.test/lint.sh
でフォーマットをチェックするファイルを追加・修正ファイル構成は下記の通りです。
Does this close any currently open issues?
しません。
How has this been tested?
下記の環境ですべてのサンプルプログラムが動作することを確認しました。
CIのビルドテストのチェックが通ることを確認しました。
Any other comments?
Checklists