Arduino freezes when I'm writing EEPROM

[Yveaux]

@jhonnygolpe What Arduino hardware are you using?
Did you try writing to another address?

[jhonnygolpe]

Well, i'm working with Arduino Uno, directly connected to the 220V/5v converter, so the power can't be the problem. I will enclose the code, and to help the reading of it, i'll try to explain it. On the Arduino there are connected a DHT22 sensor, and 2 relays and 2 buttons to control the blinds. I'm using my former motorized blinds, with its wall-mounted controllers (now one button to UP, another to DOWN). I want to use the previous controllers, and add remote control to move the blinds. So in the loop() function, I check if any of the buttons has been pressed, or if it's the time to send some info to the gateway. I use to have problems when I move down the blind remotely, with the receive function. With the debug activated, it seems to freeze after finishing the downAction() function...
By the time, I'm gonna try the code with another arduino and another EEPROM address.
As you will see I'm using another DEBUG class, very similar to the mysensors'oneSerialDebug.h blindsControlEL.ino

[mfalkvidd]

@jhonnygolpe said:

With the debug activated, it seems to freeze after finishing the downAction() function...

Are you sure it exits the downAction? How about adding debug statements before and after each funtion call in loop()? That would help determine which fuction freezes.

[jhonnygolpe]

[EDIT] I've found that I missed a return; call on the receive() function... and now it's working fine...
Really I'm not sure where it fails. And yes, the next step it's write a lot of 'DEBUG'. I've tested changing the eeprom address with the same results, so I'm assuming the fault it's in my code :worried:
But always falls after a downAction movement...

[mfalkvidd]

I think it hangs in this loop:

while ( valueU == HIGH
                  && ((nowMillis - startMillis) / OPERATION_TIME) < 100
                  && (positionValue + ((nowMillis - startMillis) / OPERATION_TIME)) < 100
                  &&  (nowMillis - startMillis) < OPERATION_TIME );

None of the variables in this loop will be updated while the loop is running, so it will run forever. I don't know what you wanted to do, but I don't think you wanted an infinite loop.

[jhonnygolpe]

Not really... inside the do loop, that it's repeated anytime, updates valueU and nowMillis values, so it's not an infinite loop.
I think it was a problem with the receive function... after calling

    else {
      moveBlind(dimvalue); 
      return;
    }
  }

the moveBlind function I missed the return;
Now it seems to work fine.... I'm sure I'll have some more errors, but i'm satisfied by the moment.
Thanks a lot to everybody

[mfalkvidd]

What's inside the while loop? To me it looks like nothing. There is no { after the while conditions, just a ;. The ; means that the while loop has no contents except re-evaluating the conditions.

[jhonnygolpe]

It's a

do { 
...t 
} while();

loop. It's similar to the while, but with this one you can control that the routine inside the do{...} it's always executed at least once.

[jhonnygolpe]

Nevertheless I must review all my code.... It's been working for a time... But lately it freezes again

[mfalkvidd]

Oh. Sorry. I wasn't aware of the do-while construct. Thanks for teaching me.