Request to improve the PWM tests in order to support single edge trigger MCUs. about ci-test-shield HOT 8 CLOSED

Hi, team
FYI. @khj098765 is our communication channel and an engineer in WIZnet Inc., mbed Silicon partner. Please regard this issue with reasonable priority. Thanks,

from ci-test-shield.

Adding this as an enhancement. We will add this in the next round of enhancements to the tests. Currently targeting end of Q3 for the next release.

For basic tests we will turn the existing single PWM test into 3 tests. One for Rise, one for Fall and one for Rise_Fall.

For Level testing we will propogate these tests into the level 1 tests as well as corner case testing.

The lack of dual edge support on the Wiznet board should not be viewed as a 'failure', but it should be noted and the note should be put on the board page so users are made aware as well. We will make changes tot he CI Test Shield project to reflect this and make it more clear what is happeneing.

@mray19027 for reference

from ci-test-shield.

The new folder structure will be as follows:

• Assumptions
• API
  |-- PWM_RISE
  |-- PWM_FALL
  |-- PWM_RISE_FALL
• Concurrent

PWM_RISE and PWM_FALL tests will only run 1 period with 1 duty cycle to avoid long test times
PWM_RISE_FALL will run the 3 designated periods and duty cycles

from ci-test-shield.

OK, I already done.
I put on the board page so users are made aware as well.
https://developer.mbed.org/platforms/WIZwiki-W7500/
content is below
Notices : Due to low need, Dual Edge Triggered GPIO will not be supported. InterruptIn class’ rise() or fall() member function will override the previous callback setting.

is it right?

from ci-test-shield.

Might I suggest :

<<warning title="PWM - single edge">>
PWM on this platform is single trigger. That means you can use either `pwm.rise()` or `pwm.fall()` but not both at the same time. If you try to use both then the software will only respect the last requested edge trigger. 
<</warning>>

from ci-test-shield.

@BlackstoneEngineering @khj098765 Added feature in PR #60. Can you verify that the 3 new tests satisfy the feature requested?

from ci-test-shield.

It looks like it will do the job. This should make it so that the Wiznet board passes the rise and fall tests but fails the rise_fall test.

@khj098765 can you please test out this PR and verify on your hardware as we do not have any wiznet boards in our office.

from ci-test-shield.

@BlackstoneEngineering @mray19027 @andrewc-arm
Sorry for the delay.
The test failed but we could not find any reason yet.
Internal testing takes time.

from ci-test-shield.