Skip to content

[STM32L0XX] Remove stabilization of the timer #2086

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

Merged
merged 1 commit into from
Jul 4, 2016

Conversation

svastm
Copy link
Contributor

@svastm svastm commented Jul 1, 2016

As we discussed here #1743 the stabilization of the timer for STM32L0 targets do not do the trick. Because it cause also the fail of MBED_23 for all the STM32L0 except NUCLEO_L073RZ, we should remove it.

+--------+---------------------+-----------+---------+------------------+--------------------+---------------+-------+
| Result | Target              | Toolchain | Test ID | Test Description | Elapsed Time (sec) | Timeout (sec) | Loops |
+--------+---------------------+-----------+---------+------------------+--------------------+---------------+-------+
| OK     | NUCLEO_L053R8[CA2B] | uARM      | MBED_23 | Ticker Int us    |        11.4        |       15      |  1/1  |
| OK     | NUCLEO_L073RZ[F55F] | uARM      | MBED_23 | Ticker Int us    |       11.37        |       15      |  1/1  |
| OK     | NUCLEO_L011K4[F4B9] | uARM      | MBED_23 | Ticker Int us    |       11.75        |       15      |  1/1  |
| OK     | NUCLEO_L031K6[C8AA] | uARM      | MBED_23 | Ticker Int us    |       11.35        |       15      |  1/1  |
+--------+---------------------+-----------+---------+------------------+--------------------+---------------+-------+

@0xc0170
Copy link
Contributor

0xc0170 commented Jul 4, 2016

As we discussed here #1743 the stabilization of the timer for STM32L0 targets do not do the trick.

What does? is there still bug to be fixed?

@svastm
Copy link
Contributor Author

svastm commented Jul 4, 2016

To me, the fix without the stabilization is OK. I can't reproduce the initial bug. The stabilization make the first us_read fail and then put the first event in the past.

@0xc0170
Copy link
Contributor

0xc0170 commented Jul 4, 2016

@mbed-bot: TEST

HOST_OSES=windows
BUILD_TOOLCHAINS=ARM,GCC_ARM,IAR
TARGETS=NUCLEO_L053R8

@mbed-bot
Copy link

mbed-bot commented Jul 4, 2016

[Build 574]
FAILURE: Something went wrong when building and testing.

@0xc0170
Copy link
Contributor

0xc0170 commented Jul 4, 2016

Tests results matches the above results, LGTM

@0xc0170 0xc0170 merged commit 4c03193 into ARMmbed:master Jul 4, 2016
@svastm svastm deleted the fix_mbed_23_l0 branch July 18, 2016 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants