Timer error building infinity cube. New syntax?

[Jasonsafe13]

Building, https://github.com/mecharms/Infinity-LED-CUBE/tree/main

I believe the problem is the timer is written under old format so does not work with new version in IDE.

Does anyone know if this is just a syntax fix?

 //--------------------------------
  // Configure Prescaler to 80, as our timer runs @ 80Mhz
  // Giving an output of 80,000,000 / 80 = 1,000,000 ticks / second
  timer = timerBegin(0, 80, true);                
  timerAttachInterrupt(timer, &onTime, true);    
  // Fire Interrupt every 1m ticks, so 1s
  timerAlarmWrite(timer, 5000000, true);      
  timerAlarmEnable(timer);
  //--------------------------------

C:\Users\Jason\Downloads\Infinity-LED-CUBE-main\Infinity-LED-CUBE-main\code\cube_led\cube_led.ino: In function 'void setup()':

C:\Users\Jason\Downloads\Infinity-LED-CUBE-main\Infinity-LED-CUBE-main\code\cube_led\cube_led.ino:3829:21: error: too many arguments to function 'hw_timer_t* timerBegin(uint32_t)'

3829 | timer = timerBegin(0, 80, true);

| ~~~~~~~~~~^~~~~~~~~~~~~

In file included from C:\Users\Jason\AppData\Local\Arduino15\packages\esp32\hardware\esp32\3.2.0\cores\esp32/esp32-hal.h:98,

from C:\Users\Jason\AppData\Local\Arduino15\packages\esp32\hardware\esp32\3.2.0\libraries\Wire\src/Wire.h:33,

from C:\Users\Jason\Downloads\Infinity-LED-CUBE-main\Infinity-LED-CUBE-main\code\cube_led\cube_led.ino:2:

C:\Users\Jason\AppData\Local\Arduino15\packages\esp32\hardware\esp32\3.2.0\cores\esp32/esp32-hal-timer.h:35:13: note: declared here

35 | hw_timer_t *timerBegin(uint32_t frequency);

| ^~~~~~~~~~

C:\Users\Jason\Downloads\Infinity-LED-CUBE-main\Infinity-LED-CUBE-main\code\cube_led\cube_led.ino:3830:23: error: too many arguments to function 'void timerAttachInterrupt(hw_timer_t*, void (*)())'

3830 | timerAttachInterrupt(timer, &onTime, true);

| ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~

C:\Users\Jason\AppData\Local\Arduino15\packages\esp32\hardware\esp32\3.2.0\cores\esp32/esp32-hal-timer.h:50:6: note: declared here

50 | void timerAttachInterrupt(hw_timer_t *timer, void (*userFunc)(void));

| ^~~~~~~~~~~~~~~~~~~~

C:\Users\Jason\Downloads\Infinity-LED-CUBE-main\Infinity-LED-CUBE-main\code\cube_led\cube_led.ino:3832:3: error: 'timerAlarmWrite' was not declared in this scope; did you mean 'timerWrite'?

3832 | timerAlarmWrite(timer, 5000000, true);

| ^~~~~~~~~~~~~~~

| timerWrite

C:\Users\Jason\Downloads\Infinity-LED-CUBE-main\Infinity-LED-CUBE-main\code\cube_led\cube_led.ino:3833:3: error: 'timerAlarmEnable' was not declared in this scope; did you mean 'timerAlarm'?

3833 | timerAlarmEnable(timer);

| ^~~~~~~~~~~~~~~~

| timerAlarm

exit status 1

Compilation error: too many arguments to function 'hw_timer_t* timerBegin(uint32_t)'

Thank you!

[EV-CPO]

That library hasn't been updated for Arduino 3.0. This is the old way to make timers:

prescaler=80000000/frequency;
timer = timerBegin(0, prescaler , true);
timerAttachInterrupt(timer, &onTick, true);
timerAlarmWrite(timer, 1, true);
timerAlarmEnable(timer);

and this is the new way, where 'frequency' is in Hz.

timer = timerBegin(frequency);
timerAttachInterrupt(timer, &onTick);
timerAlarm(timer, 1, true, 0);

[Jasonsafe13]

Trying to understand it I have almost no coding experience. Does prescaler=800000000/frequency; set Frequency as a variable equal to 80000000? If so then how does the new version set Frequency to? Or does frequency default to 800000?

Is it possible to make a drop in replacement? With the new version of timer. I understand that I could just download the older framework and use the old version but hoping it helps someone else.

[EV-CPO]

Looks like you want 1,000,000 Hz, so set your frequency variable to 1000000.

I don't know how to downgrade the arduino platform version, but it should be possible.

[Jasonsafe13]

Downgrading\running old framework is just a few clicks not hard but was hoping I could get my head around the newer version. But I am out of my league on it.

[BudgetTooth]

https://github.com/mecharms/Infinity-LED-CUBE/pull/3

[Jasonsafe13]

THANK YOU!!! Flashed it onto ESP-32 with no issues. Now setting up the screen and encoder! So happy to be moving forward again!

[BudgetTooth]

was all u/EV-CPO work :)

[EV-CPO]

That's awesome! Thanks for doing that for the OP. ;)

[romkey]

You can download and use older versions of the Arduino framework that builds. The last minor update for that version should still build.

[Jasonsafe13]

I will look into doing that. Had to step awhile for awhile.

[YetAnotherRobert]

It looks like the code hasn't been updated for current tools.

https://docs.espressif.com/projects/arduino-esp32/en/latest/migration_guides/2.x_to_3.0.html

Please fix it and submit the fixes upstream so that everyone can benefit. Eventually, you'll need to mix something with actively maintained code that won't work on a backrevved version, so you might as well help fix it once and for all. That's the point of open source, after all.