MCUXpresso_MCUS: Apply K64F us_ticker fix across all MCU's

[mmahadevan108]

No description provided.

[cmonr]

@mmahadevan108 Please fill out this PR's description.

Basic information such as where the fix is coming and/or what issues this is fixing would be appreciated. We provide a PR template as a quick way to get it started.

[mmahadevan108]

The below change was applied to K64F.

---

commit b6a01de
Author: Przemyslaw Stekiel [email protected]
Date: Fri Feb 9 14:29:24 2018 +0100

K64F lp ticker driver - calculation bug fix.

Delta calculation from lp_ticker_set_interrupt() function:
delta_us = timestamp > now_us ? timestamp - now_us : (uint32_t)((uint64_t)timestamp + 0xFFFFFFFF - now_us);

Lets assume that timestam == now_us.
Expected delta value should be 0 and in this current version is 0xFFFFFFFF.

The following condition:
timestamp > now_us
should have the following form:
timestamp >= now_us

Additionally modified us ticker driver to provide the same logic.

diff --git a/targets/TARGET_Freescale/TARGET_MCUXpresso_MCUS/TARGET_MCU_K64F/us_ticker.c b/targets/TARGET_Freescale/TARGET_MCUXpresso_MCUS/TARGET_MCU_K64F/us_ticker.c
index dc6ab710f..e4796608a 100644
--- a/targets/TARGET_Freescale/TARGET_MCUXpresso_MCUS/TARGET_MCU_K64F/us_ticker.c
+++ b/targets/TARGET_Freescale/TARGET_MCUXpresso_MCUS/TARGET_MCU_K64F/us_ticker.c
@@ -83,10 +83,15 @@ void us_ticker_clear_interrupt(void)

void us_ticker_set_interrupt(timestamp_t timestamp)
{

• uint32_t now_us, delta_us;
• now_us = us_ticker_read();
• delta_us = timestamp >= now_us ? timestamp - now_us : (uint32_t)((uint64_t)timestamp + 0xFFFFFFFF - now_us);
• uint32_t delta = timestamp - us_ticker_read();
  PIT_StopTimer(PIT, kPIT_Chnl_3);
  PIT_StopTimer(PIT, kPIT_Chnl_2);

• PIT_SetTimerPeriod(PIT, kPIT_Chnl_3, (uint32_t)delta);

• PIT_SetTimerPeriod(PIT, kPIT_Chnl_3, (uint32_t)delta_us);
  PIT_EnableInterrupts(PIT, kPIT_Chnl_3, kPIT_TimerInterruptEnable);
  PIT_StartTimer(PIT, kPIT_Chnl_3);
  PIT_StartTimer(PIT, kPIT_Chnl_2);

---

This PR includes this change for other MCUXpresso platforms.

[cmonr]

@mprse Fyi.

[cmonr]

/morph build

[mbed-ci]

Build : SUCCESS

Build number : 1371
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/6287/

Triggering tests

/morph test
/morph uvisor-test
/morph export-build
/morph mbed2-build

[mbed-ci]

Exporter Build : SUCCESS

Build number : 1022
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/exporter/6287/

[mbed-ci]

Test : SUCCESS

Build number : 1152
Test logs :http://mbed-os-logs.s3-website-us-west-1.amazonaws.com/?prefix=logs/6287/1152

[0xc0170]

@mmahadevan108 The info shared above about the fix should be part of the commit message. It is not clear what is this fixing and how (can you amend the commit message to add there the info from #6287 (comment)) ?

[mmahadevan108]

@0xc0170 I have updated the commit message.

[0xc0170]

/morph build

[mbed-ci]

Build : SUCCESS

Build number : 1380
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/6287/

Triggering tests

/morph test
/morph uvisor-test
/morph export-build
/morph mbed2-build

[mbed-ci]

Exporter Build : SUCCESS

Build number : 1034
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/exporter/6287/

[mbed-ci]

Test : FAILURE

Build number : 1162
Test logs :http://mbed-os-logs.s3-website-us-west-1.amazonaws.com/?prefix=logs/6287/1162

[0xc0170]

/morph test

[mbed-ci]

Test : FAILURE

Build number : 1163
Test logs :http://mbed-os-logs.s3-website-us-west-1.amazonaws.com/?prefix=logs/6287/1163

[0xc0170]

/morph test

[mbed-ci]

Test : SUCCESS

Build number : 1171
Test logs :http://mbed-os-logs.s3-website-us-west-1.amazonaws.com/?prefix=logs/6287/1171

[mbed-ci]

Test : SUCCESS

Build number : 1173
Test logs :http://mbed-os-logs.s3-website-us-west-1.amazonaws.com/?prefix=logs/6287/1173