wrong order of if statement?

[IgorYbema]

Should this be the other way around?

Sofar2mqtt/Sofar2mqtt/Sofar2mqtt.ino

Lines 459 to 462 in 4f06180

	if(!readSingleReg(SOFAR_SLAVE_ID, SOFAR_REG_GRIDW, &rs))
	p = ((rs.data[0] << 8) | rs.data[1]);
	else
	Serial.println("modbus error");

[cmcgerty]

Quite possibly. It certainly looks like p would only be set if readSingleReg fails. Do you have one connected with a serial console open? Does it constantly log “modbus error” when in battery save mode? I’ll take a look when I get a chance.

[IgorYbema]

Not using batterysave but I just noticed it while modifying the code a bit to suit my own situation