Skip to content

remove references to PWM_MULTICHAN option#3475

Open
raiden00pl wants to merge 2 commits into
apache:masterfrom
raiden00pl:pwm_refactor
Open

remove references to PWM_MULTICHAN option#3475
raiden00pl wants to merge 2 commits into
apache:masterfrom
raiden00pl:pwm_refactor

Conversation

@raiden00pl
Copy link
Copy Markdown
Member

Summary

remove references to PWM_MULTICHAN option

Impact

apps part to apache/nuttx#18864

Testing

CI

@xiaoxiang781216
Copy link
Copy Markdown
Contributor

@raiden00pl fix the style:

    1: .codespellrc
Error: /home/runner/work/nuttx-apps/nuttx-apps/apps/examples/pwm/pwm_main.c:150:2: error: Bad left brace alignment
Error: /home/runner/work/nuttx-apps/nuttx-apps/apps/examples/pwm/pwm_main.c:167:2: error: Bad right brace alignment

jerpelea
jerpelea previously approved these changes May 11, 2026
@raiden00pl raiden00pl dismissed stale reviews from jerpelea and xiaoxiang781216 via 1362aa8 May 11, 2026 10:21
@raiden00pl raiden00pl force-pushed the pwm_refactor branch 2 times, most recently from 1362aa8 to ed86122 Compare May 11, 2026 11:14
remove references to CONFIG_PWM_MULTICHAN

Signed-off-by: raiden00pl <raiden00@railab.me>
remove references to CONFIG_PWM_MULTICHAN

Signed-off-by: raiden00pl <raiden00@railab.me>
@lupyuen
Copy link
Copy Markdown
Member

lupyuen commented May 12, 2026

Sorry @raiden00pl could you implement retry for NTFC git clone? It failed 4 times today, we're hoping to conserve our precious GitHub Runners whenever git clone fails thanks :-) FYI @simbit18

$ git clone -b release-0.0.1 https://github.com/apache/nuttx-ntfc-testing
fatal: unable to access 'https://github.com/apache/nuttx-ntfc-testing/': The requested URL returned error: 500
Screenshot 2026-05-12 at 8 37 56 AM

https://lupyuen.github.io/nuttx-github-jobs/build-monitor

@raiden00pl
Copy link
Copy Markdown
Member Author

@lupyuen PR is here #3478 but I don't know if this will help in all cases. This is an error on the GitHub infra side. If GitHub is unable to clone from its own repos, it means GitHub is temporarily broken and it may take longer than our repeated attempts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants