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

Imx9 lptpm pwm #12054

Merged
merged 2 commits into from
Apr 5, 2024
Merged

Imx9 lptpm pwm #12054

merged 2 commits into from
Apr 5, 2024

Conversation

jlaitine
Copy link
Contributor

@jlaitine jlaitine commented Apr 4, 2024

Summary

This adds an LPTPM timer based PWM control to IMX9 platform. On i.MX93 there are 6 TPM modules with 4 timers each.

In addition, this configures one PWM output on imx93-evk board's 40 pin header (GPIO_IO24).

Impact

Affects only IMX9 based designs

Testing

Some of the channels of TPM1, TPM3 and TPM4 modules have been tested on EVK pins. All can't be tested on the EVK board.

@jlaitine jlaitine force-pushed the imx9_lptpm_pwm branch 2 times, most recently from 398fe36 to 0d03eed Compare April 4, 2024 19:29
@jlaitine
Copy link
Contributor Author

jlaitine commented Apr 4, 2024

rebased due to Kconfig & Make.defs conficts

arch/arm64/src/imx9/hardware/imx9_tpm.h Outdated Show resolved Hide resolved
arch/arm64/src/imx9/hardware/imx9_tpm.h Outdated Show resolved Hide resolved
arch/arm64/src/imx9/imx9_tpm_pwm.c Outdated Show resolved Hide resolved
arch/arm64/src/imx9/imx9_tpm_pwm.c Outdated Show resolved Hide resolved
arch/arm64/src/imx9/imx9_tpm_pwm.c Outdated Show resolved Hide resolved
@jlaitine
Copy link
Contributor Author

jlaitine commented Apr 5, 2024

I added one patch to this same PR, just found a bug that it was not possible to re-start the PWM outputs if it had been stopped.

Signed-off-by: Jukka Laitinen <jukkax@ssrc.tii.ae>
… GPIO_IO24

Signed-off-by: Jukka Laitinen <jukkax@ssrc.tii.ae>
@jlaitine
Copy link
Contributor Author

jlaitine commented Apr 5, 2024

Squashed all the fixes together, re-tested with logic analyzer

@acassis acassis merged commit 16b30ea into apache:master Apr 5, 2024
26 checks passed
@jerpelea jerpelea added this to To-Add in Release Notes - 12.5.1 Apr 8, 2024
@jerpelea jerpelea moved this from To-Add to Not Applicable in Release Notes - 12.5.1 Apr 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants