Fix energyReset command issue #6561

[pablozg]

Now we only use the hardware energy total counter as reference if the "setOption72" is enabled.

[pablozg]

Hi, I'm sorry but I pressed by mistake the close PR button.

When setoption is set to 1 we use hardware as reference, when 0 we use the firmware counter.

The default option is a question for Theo 😅

[andrethomas]

Is there a specific reason why this would need to be changed during runtime?

Assuming it would be either enabled or disabled permanently it may be beneficial to set this in my_user_config.h

Just a thought...

[meingraham]

even better - user_config_override.h 😉

[pablozg]

Assuming it would be either enabled or disabled permanently it may be beneficial to set this in my_user_config.h

As the "sonoff-sensors" firmware has activated all supported meters, I think we have to support a precompiled option to enable or disable the hardware sync, them the users can choose what they needs without to have to compile all firmware every time they want to change it, and the change don't have a code impact because only use a existing bit.

[arendst]

Agree with @pablozg and will merge. The default will be the long standing firmware counter (0), the hardware counter should be (1).

Thx all for input and considerations

[meingraham]

Is this applicable only to the DDS238?

[arendst]

No. It's applicable to all sensors with hardware energy total support. Currently they are

• All PZEMs
• SDM120
• SDM630
• DDS238
• DDSU666