[bsp][gd32] fix pwm enable channel check#11547
Conversation
|
👋 感谢您对 RT-Thread 的贡献!Thank you for your contribution to RT-Thread! 为确保代码符合 RT-Thread 的编码规范,请在你的仓库中执行以下步骤运行代码格式化工作流(如果格式化CI运行失败)。 🛠 操作步骤 | Steps
完成后,提交将自动更新至 如有问题欢迎联系我们,再次感谢您的贡献!💐 |
|
ci-attach也麻烦添加下吧,避免后面出现类似情况。教程:https://club.rt-thread.org/ask/article/d273bbcd1f8779bc.html |
|
已补充 ci-attach:新增 这个 BSP 已在 本地已检查:
新 CI 已由提交触发,正在跑。 |
CI报错了麻烦处理一下 |
|
更新一下 CI 结果: 失败日志里 我已经在 package 仓库补了对应修复:RT-Thread-packages/gd32-arm-series#11。等这个 package PR 合入后,重新跑本 PR 的 BSP CI attach 应该就能覆盖并通过 |
Summary
Root cause
drv_pwm_enable()checkschannel_num, but the local variable was never initialized in that function after the GD32 PWM refactor. The adjacent period and pulse helpers already derive it fromconfiguration->channelbefore the same validation.Closes #11331
Testing
git diff --checkbsp/gd32/arm/libraries/gd32_drivers/drv_pwm.cNot run: GD32 BSP build. This machine does not have
scons,gcc,arm-none-eabi-gcc, orclanginstalled.