delay

[DanSkareda]

Hi. I try this sketch on attiny414 (this is only part of my program):

void loop()
{
lcd.setCursor(0, 0);
adr=adr+1;
lcd.print(adr);
delay(500);
}

When I compile this sketch with last version MegaTinyCore (I instaled it to "hardware" program directory manually), delay not works.
When I compile this sketch with some older version MegaTinyCore (I installed it through "json" web adress), all ok, delay works.
Is it fault, or I have something wrong set up ? Thanks.

[SpenceKonde]

Thanks, I'll see what I managed to break with delay - I did make some changes around delay a couple of days ago that are in master, but not in released version, I guess I didn't actually test whether the new delay implementation gave the right delay >.>

I had realized the old delay implementation would break when a very long delay (more than 4.2 million milliseconds - ie, someone trying to wait for a couple of hours with delay - which is bad practice, but, you know people will do it). further investigation indicated that it was done to save 24 bytes of flash - but if the new implementation just straight up doesn't work, well... there is more to it I suppose... On a 4k part you should be getting the new implementation (I kept the old one for 2k parts where that 24 bytes really matters), but if its not behaving like delay should, I need to correct that.

[SpenceKonde]

Broken in 2.0.0,. worked in 1.1.10..... no changes that could have conceivably had any effect on this were made between the 1.1.10 and 2.0.0

[DanSkareda]

I don't know if I understand what was written well, but I was looking for a version of MegaTinyCore on my PC. The version installed using the "http://drazzy.com/package_drazzy.com_index.json" link is probably 2.2.9, it works properly (I found it in the location "c: \ Users \ Dan \ AppData \ Local \ Arduino15 \ packages \ megaTinyCore \ hardware \ megaavr \ 2.2.9"). The version downloaded manually is according to the file ChangeLog.md - 2.3.0 (I copied it to "c: \ Users \ Dan \ Documents \ Arduino \ hardware \ megaTinyCore-master"), it does not work properly - the delay is always zero, with the parameter change the delay does not change.

[SpenceKonde]

Yeah.... I figured out the problem - A couple of days ago to fix an overflow bug in delay that caused delay(4300000) to delay for only 5 seconds I switched to the "standard" delay()
The "standard" delay reacts very badly if micros() ever goes backwards. But micros() never should go backwards and I thought that was fixed since 1.1.9. But... no, my fixes from back then never actually worked for TCA0 and TCD0 as millis source - only TCB as millis worked with9ut micros() glitching,

I've checked in fixes With TCA0 for millis, all now works as it should, and if TCD0 is used for millis, for now it will use the old delay() implementation, though I need to fix this problem with TCD0 as millis source...

Thanks for reporting this - there were several problems contributing to this....

[SpenceKonde]

aaaahahaaa!!! I think I've got it fixed for TCD0 now too! Tests will take mins to finish before I know for sure (my test sketch now calls micros() 20.000.000 times; 1,000,000 each with 20 slightly different loops that run at different speeds

[DanSkareda]

Great! I tried the latest version of the library and the delay is now right. Thanks.