T4 , set_arm_clock and micros()?

**WMXZ** · 10-18-2019, 12:13 PM

I get wrong values using micros() after changing CPU speed with set_arm_clock on a T4.

Code:

#if defined(__IMXRT1062__)
extern "C" uint32_t set_arm_clock(uint32_t frequency);
#endif

void setup() {
  // put your setup code here, to run once:
#if defined(__IMXRT1062__)
  set_arm_clock(24000000); // comment here to get full speed
#endif
}

void loop() {
  // put your main code here, to run repeatedly:
 static uint32_t t0;
 if(millis()>(t0+1000))
 {
   t0=millis();
   uint32_t t1=micros();
   delay(50);
   Serial.println(micros()-t1);
 }
}

this code gives me 1050 as printout
when commenting the clock change I get the correct value of 50000

**defragster** · 10-18-2019, 06:09 PM

Originally Posted by WMXZ

I get wrong values using micros() after changing CPU speed with set_arm_clock on a T4.
...
this code gives me 1050 as printout
when commenting the clock change I get the correct value of 50000

@Paul - you'll recognize something here in a flash ... clock change order not leaving millis() clock right?

Interesting - the problem is millis() when the clock drops - if it drops in the RIGHT order the CLOCKS change over - if not millis() is broken:

Code:

 F_CPU=600000000	deg  C=32
	50ms delay:: 50000 us and 50 ms
 F_CPU=240000000	deg  C=33
	50ms delay:: 50000 us and 50 ms
 F_CPU=110000000	deg  C=32
	50ms delay:: 1010 us and 1 ms
 F_CPU=24000000	deg  C=30
	50ms delay:: 50006 us and 50 ms
 F_CPU=600000000	deg  C=30
	50ms delay:: 50000 us and 50 ms
 F_CPU=129600000	deg  C=34
	50ms delay:: 50001 us and 50 ms
 F_CPU=110000000	deg  C=32
	50ms delay:: 50002 us and 50 ms
 F_CPU=24000000	deg  C=31
	50ms delay:: 50006 us and 50 ms
 F_CPU=600000000	deg  C=30
	50ms delay:: 50000 us and 50 ms
 F_CPU=24000000	deg  C=34
	50ms delay:: 1050 us and 1 ms
 DONE 
 	 F_CPU=24000000	deg  C=31

Code:

#if defined(__IMXRT1062__)
extern "C" uint32_t set_arm_clock(uint32_t frequency);
#endif

void setup() {
  while ( !Serial);
  Serial.println("\n" __FILE__ " " __DATE__ " " __TIME__);
  // put your setup code here, to run once:
#if defined(__IMXRT1062__)

  TimeDiff();
  set_arm_clock(240000000); // comment here to get full speed
  TimeDiff();
  set_arm_clock(110000000); // comment here to get full speed
  TimeDiff();
  set_arm_clock(24000000); // comment here to get full speed
  TimeDiff();
  set_arm_clock(600000000); // comment here to get full speed
  TimeDiff();
  set_arm_clock(130000000); // comment here to get full speed
  TimeDiff();
  set_arm_clock(110000000); // comment here to get full speed
  TimeDiff();
  set_arm_clock(24000000); // comment here to get full speed
  TimeDiff();
  set_arm_clock(600000000); // comment here to get full speed
  TimeDiff();
  set_arm_clock(24000000); // comment here to get full speed
  TimeDiff();
  Serial.printf("\n DONE \n \t F_CPU=%u", F_CPU_ACTUAL );
  Serial.printf( "\tdeg  C=%u\n" , (uint32_t)tempmonGetTemp() );
#endif

}

static uint32_t t0;


void loop() {
  // put your main code here, to run repeatedly:
}

void TimeDiff() {
#if defined(__IMXRT1062__)
  Serial.printf("\n F_CPU=%u", F_CPU_ACTUAL );
  Serial.printf( "\tdeg  C=%u\n" , (uint32_t)tempmonGetTemp() );
#endif
 while (!(millis()>(t0+1000)));
   t0=millis();
   uint32_t t1=micros(), t2=millis();
   delay(50);
   Serial.printf("\t50ms delay:: %u us and %u ms", micros()-t1, millis()-t2);
}

**WMXZ** · 10-19-2019, 02:48 PM

the following code

Code:

#if defined(__IMXRT1062__)
extern "C" uint32_t set_arm_clock(uint32_t frequency);
#endif

extern volatile uint32_t systick_cycle_count;

uint32_t m_micros(void)
{
  uint32_t ccdelta, usec, smc, scc;
//  do {
//    __LDREXW(&systick_safe_read);
    smc = systick_millis_count;
    scc = systick_cycle_count;
//  } while ( __STREXW(1, &systick_safe_read));
  ccdelta = ARM_DWT_CYCCNT - scc;
  usec = 1000*smc + (ccdelta/(F_CPU_ACTUAL/1000000));
  return usec;
}

void setup() {
  // put your setup code here, to run once:
#if defined(__IMXRT1062__)
  set_arm_clock(24000000); // comment here to get full speed
#endif
}

void loop() {
  // put your main code here, to run repeatedly:
 static uint32_t t0;
 if(millis()>(t0+1000))
 {
   t0=millis();
   uint32_t t2=millis();
   uint32_t t1=m_micros();
   delay(50);
   Serial.printf("%d %d\n", millis()-t2, m_micros()-t1);
 }
}

gives me

Code:

that is, result is frequently correct

but uncommenting the ARM code

Code:

#if defined(__IMXRT1062__)
extern "C" uint32_t set_arm_clock(uint32_t frequency);
#endif

#include "arm_math.h"  // micros() synchronization
extern volatile uint32_t systick_millis_count;
extern volatile uint32_t systick_cycle_count;
extern uint32_t systick_safe_read;   // micros() synchronization

uint32_t m_micros(void)
{
  uint32_t ccdelta, usec, smc, scc;
  do {
    __LDREXW(&systick_safe_read);
    smc = systick_millis_count;
    scc = systick_cycle_count;
  } while ( __STREXW(1, &systick_safe_read));
  ccdelta = ARM_DWT_CYCCNT - scc;
  usec = 1000*smc + (ccdelta/(F_CPU_ACTUAL/1000000));
  return usec;
}

void setup() {
  // put your setup code here, to run once:
#if defined(__IMXRT1062__)
  set_arm_clock(24000000); // comment here to get full speed
#endif
}

void loop() {
  // put your main code here, to run repeatedly:
 static uint32_t t0;
 if(millis()>(t0+1000))
 {
   t0=millis();
   uint32_t t2=millis();
   uint32_t t1=m_micros();
   delay(50);
   Serial.printf("%d %d\n", millis()-t2, m_micros()-t1);
 }
}

gives consistently

Code:

**WMXZ** · 10-19-2019, 05:27 PM

doubling the set_arm_clock command seems to work

Code:

#if defined(__IMXRT1062__)
extern "C" uint32_t set_arm_clock(uint32_t frequency);
#endif

void setup() {
  // put your setup code here, to run once:
#if defined(__IMXRT1062__)
  set_arm_clock(24000000); // comment here to get full speed
  set_arm_clock(24000000); // comment here to get full speed
#endif
}

void loop() {
  // put your main code here, to run repeatedly:
 static uint32_t t0;
 if(millis()>(t0+1000))
 {
   t0=millis();
   uint32_t t2=millis();
   uint32_t t1=micros();
   delay(50);
   Serial.printf("%d %d\n", millis()-t2, micros()-t1);
 }
}

produces

Code:

conclusion: are there some wait missing in the set_arm_clock function?

**mjs513** · 10-19-2019, 06:22 PM

Since I was playing I tried various clock settings using just one setArmClock call. It seems to start going off below 200Mhz. At 180Mhz am seeing:

Code:

above that its right on. At 150Mhz you start seeing what you are seeing and just gets worse as you lower the clock. At 150Mhz

Code:

Even if I double up on the set clock calls at 150Mhz it is still off.

Can't remember but 150Mhz sounds familiar for another clock as well Hmm

**defragster** · 10-19-2019, 07:26 PM

Originally Posted by mjs513

Since I was playing I tried various clock settings using just one setArmClock call. It seems to start going off below 200Mhz. At 180Mhz am seeing:

…

above that its right on. At 150Mhz you start seeing what you are seeing and just gets worse as you lower the clock. At 150Mhz
…
Even if I double up on the set clock calls at 150Mhz it is still off.

Can't remember but 150Mhz sounds familiar for another clock as well Hmm

As @mjs513 notes - and I saw in p#2 code - : there is something off in the transitions with set_arm_clock - with orderly transitions some affect on millis() clock is resolved - on some others it is not.

The '__LDREXW' code 'normally' only triggers if the millis_tick happens during the few cycles of the micros() code - i.e. in testing I ran it in tight loops to get EVERY microsecond change - that is multiple calls per us and only once per thousand micros() would the millis_tick happen even then.

The problem is the millis() code even never calling micros() - a 50 ms delay only advances 1 millis() {that is the LEFT number} - which properly tells micros() only 1000 micros advanced in the 50 ms delay.

1 1007
49 50000

Paul understands the clocks and knows of a specific interaction 'I Assume' from notes I've seen in the code when the PLL clock or whatever it was comes and goes during speed transitions that has to change to drive USB Serial or something.

I did clock speed transition testing in Beta as I did micros() … but of course I did uniform steps of … down … up … which triggers what shows here where the one value/setting needing updated is caught.

**defragster** · 10-23-2019, 03:10 PM

Originally Posted by WMXZ

doubling the set_arm_clock command seems to work

Code:

#if defined(__IMXRT1062__)
extern "C" uint32_t set_arm_clock(uint32_t frequency);
#endif

void setup() {
  // put your setup code here, to run once:
#if defined(__IMXRT1062__)
  set_arm_clock(24000000); // comment here to get full speed
  set_arm_clock(24000000); // comment here to get full speed
#endif
}

void loop() {
  // put your main code here, to run repeatedly:
 static uint32_t t0;
 if(millis()>(t0+1000))
 {
   t0=millis();
   uint32_t t2=millis();
   uint32_t t1=micros();
   delay(50);
   Serial.printf("%d %d\n", millis()-t2, micros()-t1);
 }
}

produces

Code:

conclusion: are there some wait missing in the set_arm_clock function?

Re-reading this - it points to the problem and the conclusion seems an explanation in the right direction.

**defragster** · 12-10-2019, 08:29 AM

Paul - TeensyDuino 1.49 fixable? Certain passes through set_arm_clock() with clock speed transition depending on current settings leave the clocking for millis() not properly set.

Post #2 the code in code block #2 does repro on 1.49b1 cores.

**defragster** · 12-15-2019, 06:19 AM

UPDATE - second thread started HERE :: https://forum.pjrc.com/threads/58688...-delay-and-SPI

Thread: T4 , set_arm_clock and micros()?

Thread Tools

Search Thread

Display

T4 , set_arm_clock and micros()?

Solved?

Posting Permissions