Skip to content

Conversation

@npmenard
Copy link
Contributor

Give user context if hardware pwm pins cannot be used
see viam-modules/pinctrl#27

@npmenard npmenard changed the title warn when pwm config is missing [RSDK-12176] warn when pwm config is missing Dec 19, 2025
@JohnN193
Copy link
Collaborator

haven't used this part of the board config before, but would we be able to support the setting through the board_settings attribute as well?

@npmenard
Copy link
Contributor Author

npmenard commented Dec 29, 2025

haven't used this part of the board config before, but would we be able to support the setting through the board_settings attribute as well?

IMO it is bad design to change a config file in the boot process without any regards to potential configuration conflicts
The default behavior is to use software PWM which should be enough for most cases so I don't think we need the change the config for the user

@npmenard npmenard merged commit fbc7076 into viam-modules:main Dec 31, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants