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

drivers: cc13xx_cc26xx: pwm: introduce pwm driver #60828

Merged

Conversation

fgrandel
Copy link
Contributor

This change introduces a new PWM driver for all CC13/26xx SoC.

See the documentation in ti,cc13xx-cc26xx-timer-pwm.yaml for detailed usage instructions.

@fgrandel fgrandel requested a review from galak as a code owner July 26, 2023 12:41
@zephyrbot zephyrbot added area: PWM Pulse Width Modulation area: Devicetree Binding PR modifies or adds a Device Tree binding platform: TI SimpleLink Texas Instruments SimpleLink MCU labels Jul 26, 2023
vaishnavachath
vaishnavachath previously approved these changes Aug 10, 2023
Copy link
Contributor

@vaishnavachath vaishnavachath left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution @fgrandel , looks good to me, I will test the changes later today.

drivers/pwm/pwm_cc13xx_cc26xx_timer.c Outdated Show resolved Hide resolved
vaishnavachath
vaishnavachath previously approved these changes Aug 10, 2023
Copy link
Member

@cfriedt cfriedt left a comment

Choose a reason for hiding this comment

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

LGTM, but can we also have this driver added to a build_all suite?

This change introduces a new PWM driver for all CC13/26xx SoC.

See the documentation in ti,cc13xx-cc26xx-timer-pwm.yaml for detailed
usage instructions.

Signed-off-by: Florian Grandel <fgrandel@code-for-humans.de>
@fgrandel
Copy link
Contributor Author

LGTM, but can we also have this driver added to a build_all suite?

@cfriedt Thanks for your review. Fixed. :-)

@fabiobaltieri fabiobaltieri merged commit b954ce4 into zephyrproject-rtos:main Aug 14, 2023
17 checks passed
@fgrandel fgrandel deleted the feat/cc13xx_cc26xx_pwm branch August 14, 2023 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Devicetree Binding PR modifies or adds a Device Tree binding area: PWM Pulse Width Modulation platform: TI SimpleLink Texas Instruments SimpleLink MCU
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants