Skip to content

RaspberryPi OSカーネルrpi-6.12への対応 #99

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

KuraZuzu
Copy link
Contributor

@KuraZuzu KuraZuzu commented May 21, 2025

What does this implement/fix?

Raspberry Pi OSのカーネルrpi-6.12系に対応します。

Does this close any currently open issues?

#98

How has this been tested?

Raspberry Pi Mouse V3の実機でビルドし、サンプルプログラム(step.1~6)の動作を確認しました。

動作環境

以下の環境で動作を確認しました。

  • Raspberry Pi Mouse V3
    • Raspberry Pi 4B+ (4GB)
  • OS
    • Raspberry Pi OS (64bit)
      • 6.12.25+rpt-rpi-v8
    • Ubuntu Server 24.04
      • 6.8.0-1028-raspi
    • Ubuntu Server 22.04
      • 5.15.0-1077-raspi

Any other comments?

GitHub ActionsのUbuntu 20.04サポート終了に伴い、worlfowsのファイルから該当構成を削除しました。

Checklists

@KuraZuzu KuraZuzu self-assigned this May 21, 2025
@KuraZuzu KuraZuzu added Type: Bug Bug or Bug fixes Type: Maintenance Repository Maintenance labels May 21, 2025
@KuraZuzu
Copy link
Contributor Author

rpi-6.12系で削除されたspi_masterマクロをspi_controllerに置き換えしましたが、spi_busnum_to_master関数との関係性を分かりやすくするために、変数名はmasterのままにしています。

@YusukeKato YusukeKato self-requested a review May 23, 2025 07:47
Copy link
Contributor

@YusukeKato YusukeKato left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

私のほうでも下記の環境でビルドできることを確認しました。

  • Ubuntu 22.04 LTS
    • 5.15.0-1077-raspi
  • Raspberry Pi OS (64bit)
    • 6.12.25+rpt-rpi-v8
  • Raspberry Pi OS (32bit)
    • 6.6.31+rpt-rpi-v7l
    • 6.12.25+rpt-rpi-v7l

修正内容問題ないかと思います。
一つだけコメントしましたのでご確認をお願いいたします。

@KuraZuzu
Copy link
Contributor Author

レビューありがとうございます。提案されたコミットを適用しました。

@KuraZuzu KuraZuzu requested a review from YusukeKato May 23, 2025 08:59
Copy link
Contributor

@YusukeKato YusukeKato left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

修正ありがとうございました!
LGTMです!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Bug or Bug fixes Type: Maintenance Repository Maintenance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants