Skip to content

Commit a1ff2a0

Browse files
committed
Continue config only if previous steps were successful
1 parent 5d4e90c commit a1ff2a0

File tree

1 file changed

+13
-7
lines changed

1 file changed

+13
-7
lines changed

Firmware/RTK_Everywhere/GNSS_LG290P.ino

Lines changed: 13 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -265,18 +265,24 @@ bool GNSS_LG290P::configureOnce()
265265
response &=
266266
setRadioBaudRate(settings.radioPortBaud); // LG290P UART3 is connected to the locking JST connector
267267

268-
if (settings.debugGnss)
268+
if (response == false && settings.debugGnss)
269269
systemPrintln("configureOnce: setBauds failed.");
270270
}
271271

272272
// Enable PPS signal with a width of 200ms
273-
response &= _lg290p->setPPS(200, false, true); // duration time ms, alwaysOutput, polarity
274-
if (settings.debugGnss && response == false)
275-
systemPrintln("configureOnce: setPPS failed");
273+
if (response == true)
274+
{
275+
response &= _lg290p->setPPS(200, false, true); // duration time ms, alwaysOutput, polarity
276+
if (settings.debugGnss && response == false)
277+
systemPrintln("configureOnce: setPPS failed");
278+
}
276279

277-
response &= setConstellations();
278-
if (settings.debugGnss && response == false)
279-
systemPrintln("configureOnce: setConstellations failed");
280+
if (response == true)
281+
{
282+
response &= setConstellations();
283+
if (settings.debugGnss && response == false)
284+
systemPrintln("configureOnce: setConstellations failed");
285+
}
280286

281287
// We do not set Rover or fix rate here because fix rate only applies in rover mode.
282288

0 commit comments

Comments
 (0)