Skip to content
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

WDTの監視周期設定コマンドの追加 #380

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

771-8bit
Copy link

概要

  • WDTの監視周期をコマンドで変更、テレメで確認できるようにした

詳細

  • もともとHALに存在する関数WDT_set_timer()をコマンドとして露出させた
  • HALのWDT_Config.timer_settingをテレメとして登録した

検証結果

DevToolsでexamples/mobc,subobcともにWDT_SET_TIMERコマンドでHK.WDT.TIMERテレメを変更できることを確認した

@771-8bit 771-8bit self-assigned this Nov 13, 2024
@771-8bit 771-8bit requested review from sksat and removed request for meltingrabbit November 13, 2024 06:55
771-8bit and others added 5 commits November 15, 2024 18:00
Co-authored-by: Ryo Suzumoto <ryo.suzumoto@gmail.com>
Co-authored-by: Ryo Suzumoto <ryo.suzumoto@gmail.com>
Copy link
Member

@meltingrabbit meltingrabbit left a comment

Choose a reason for hiding this comment

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

よさそう

お疲れ様です
テストも書いてくれてGoodです

Copy link
Member

@sksat sksat 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
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

3 participants