Skip to content

Qtmr capture support #97698

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Qtmr capture support #97698

wants to merge 5 commits into from
+414 −40

Conversation

FelixWang47831
Copy link
Contributor 

1. Enable PWM capture for QTMR driver
2. Fix qtmr set cycles bug, which can not generate 100% and 0% duty cycle PWM wave
3. Improve pwm_loopback to support change timing parameters from kconfig
4. Enable qtmr test cases, including pwm_api, pwm_loopback.

@FelixWang47831
tests: drivers: pwm: pwm_api: enable qtmr test
04b5150 
Add mimxrt1180_evk_mimxrt1189_cm33_qtmr.overlay for test.

Signed-off-by: Felix Wang <[email protected]>
@FelixWang47831
drivers: pwm: enable pwm capture for qtmr driver
a1956f7 
Supported mode: single capture, continus capture,
pulse capture, period capture.

Signed-off-by: Felix Wang <[email protected]>
@FelixWang47831
drivers: pwm: Fix qtmr set cycles bug.
7a921eb 
The mcux_qtmr_pwm_set_cycles can not set 100% and 0%
duty cycle PWM wave.
Set output compare setting based on pulse_cycles and
period_cycles:
1. If pulse_cycles is 0, generate 0% duty cycle wave.
2. If pulse_cycles equals period_cycles but not 0,
   generate 100% duty cycle wave.
3. Otherwise toggle output when compare value matched.

Signed-off-by: Felix Wang <[email protected]>
@FelixWang47831
tests: drivers: pwm: pwm_loopback: test improvement.
13fe276 
Move TEST_PWM_PERIOD and TEST_PWM_PULSE to kconfig
and update macro name with "CONFIG_" prefix.

The default timing parameters may be too big for timer
peripheral, because they have high frequence, can not
generate such slow PWM wave.

To keep compatibility, the default value in kconfig
are consistent with the code.

Signed-off-by: Felix Wang <[email protected]>
@FelixWang47831
tests: drivers: pwm: pwm_loopback: Enable QTMR test.
9a8010a 
Provide kconfig conf file to set timing parameters.
Provide overlay file for QTMR channel and pin configuration.

Signed-off-by: Felix Wang <[email protected]>
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@FelixWang47831 FelixWang47831 marked this pull request as ready for review October 16, 2025 14:22
@zephyrbot zephyrbot added platform: NXP NXP area: PWM Pulse Width Modulation area: Tests Issues related to a particular existing or missing test labels Oct 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@anangl anangl Awaiting requested review from anangl

@dbaluta dbaluta Awaiting requested review from dbaluta

@decsny decsny Awaiting requested review from decsny

@dleach02 dleach02 Awaiting requested review from dleach02

@henrikbrixandersen henrikbrixandersen Awaiting requested review from henrikbrixandersen

@manuargue manuargue Awaiting requested review from manuargue

@mmahadevan108 mmahadevan108 Awaiting requested review from mmahadevan108

@nashif nashif Awaiting requested review from nashif

At least 2 approving reviews are required to merge this pull request.

Assignees

@mmahadevan108 mmahadevan108

@dleach02 dleach02

Labels

area: PWM Pulse Width Modulation area: Tests Issues related to a particular existing or missing test platform: NXP NXP

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@FelixWang47831 @mmahadevan108 @zephyrbot @dleach02