Skip to content

HardwareTimer: add non regression test #16

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
Merged
merged 1 commit into from
Dec 9, 2019
Merged

HardwareTimer: add non regression test #16

merged 1 commit into from
Dec 9, 2019
+570 −0

Conversation

ABOSTM
Copy link
Contributor

@ABOSTM ABOSTM commented Dec 2, 2019 •
edited by fpistm
Loading

Non regression test include:
Mode:

  • TIMER_OUTPUT_COMPARE
  • TIMER_INPUT_FREQ_DUTY_MEASUREMENT
  • TIMER_OUTPUT_COMPARE_PWM1
  • TIMER_OUTPUT_COMPARE_PWM2
  • TIMER_OUTPUT_COMPARE_TOGGLE

API:

  • hasInterrupt, attachInterrupt, detachInterrupt for update/channel interrupts
  • pause, resume, pauseChannel, resumeChannel
  • pauseChannel/resumeChannel
  • setMode, setPWM, setOverflow, setCaptureCompare, getPrescaleFactor, setPrescaleFactor
@ABOSTM ABOSTM force-pushed the HARDWARE_TIMER_NON_REGRESSION_TEST branch from 28a2db5 to e76106f Compare December 2, 2019 16:26
fpistm
fpistm requested changes Dec 9, 2019
View reviewed changes
Copy link
Member

@fpistm fpistm left a comment

Choose a reason for hiding this comment

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

Example is not available in the Arduino IDE.
The directory and the .ino have to have the same name.
I proposed this:
examples/NonReg/HardwareTimer/HardwareTimer_OutputInput_test/HardwareTimer_OutputInput_test.ino

@ABOSTM ABOSTM force-pushed the HARDWARE_TIMER_NON_REGRESSION_TEST branch from e76106f to d0da439 Compare December 9, 2019 14:53
@fpistm fpistm merged commit 2ce161f into stm32duino:master Dec 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@ABOSTM @fpistm