-
Notifications
You must be signed in to change notification settings - Fork 14.6k
feat: configurable board pwm freq from Kconfig #24787
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
Conversation
@dagar all fixed, shold be good to go. |
Looks good now, but submodule changes slipped in. |
did you mean to include the gz submodule changes @Pedro-Roque ? |
This pull request has been mentioned on Discussion Forum for PX4, Pixhawk, QGroundControl, MAVSDK, MAVLink. There might be relevant details there: https://discuss.px4.io/t/px4-dev-call-june-4-2025-team-sync-and-community-q-a/45866/1 |
@mrpollo fixed |
🔎 FLASH Analysispx4_fmu-v5x [Total VM Diff: 8 byte (0 %)]
px4_fmu-v6x [Total VM Diff: 8 byte (0 %)]
Updated: 2025-07-23T11:57:57 |
@PetervdPerk @PetervdPerk-NXP I set the Kconfig to look for the ARCH_CHIP but it seems that these variables are not set at boardconfig time, as right now the platforms menu is empty. I'm probably missing something obvious, any pointers I should look at? Edit: Notes for discussion:
|
This pull request has been mentioned on Discussion Forum for PX4, Pixhawk, QGroundControl, MAVSDK, MAVLink. There might be relevant details there: https://discuss.px4.io/t/px4-dev-call-june-25-2025-team-sync-and-community-q-a/46161/1 |
Allows to expose nuttx into PX4 if needed
Only if applicable i.e. defined in the kconfig
…lot into pr-kconfig-board-pwm-freq
This pull request has been mentioned on Discussion Forum for PX4, Pixhawk, QGroundControl, MAVSDK, MAVLink. There might be relevant details there: https://discuss.px4.io/t/px4-dev-call-july-2-2025-team-sync-and-community-q-a/46266/3 |
This pull request has been mentioned on Discussion Forum for PX4, Pixhawk, QGroundControl, MAVSDK, MAVLink. There might be relevant details there: https://discuss.px4.io/t/px4-dev-call-july-16-2025-team-sync-and-community-q-a/46513/3 |
Solved Problem
Allows for configurable board PWM frequency from Kconfig. For slow PWMs (like 10Hz) this value needs to change due to timer overflow on a 16bit register.
Solution
Changelog Entry
For release notes:
Alternatives
None me and @dagar could think of.
Test coverage
Context
Spacecraft solenoid valves requiring slow PWM.