Skip to content

Commit

Permalink
Removed checking for valid tx pin for Victron (#455)
Browse files Browse the repository at this point in the history
Please check wiki documentation, and change/extend it accordingly.
  • Loading branch information
philippsandhaus authored Sep 28, 2023
1 parent 933345d commit 4ee49a6
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
3 changes: 1 addition & 2 deletions src/PinMapping.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -307,8 +307,7 @@ bool PinMappingClass::isValidEthConfig()

bool PinMappingClass::isValidVictronConfig()
{
return _pinMapping.victron_rx >= 0
&& _pinMapping.victron_tx >= 0;
return _pinMapping.victron_rx >= 0;
}

bool PinMappingClass::isValidHuaweiConfig()
Expand Down
2 changes: 1 addition & 1 deletion src/VictronSmartShunt.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ bool VictronSmartShunt::init(bool verboseLogging)
MessageOutput.printf("[VictronSmartShunt] Interface rx = %d, tx = %d\r\n",
pin.battery_rx, pin.battery_tx);

if (pin.battery_rx < 0 || pin.battery_tx < 0) {
if (pin.battery_rx < 0) {
MessageOutput.println(F("[VictronSmartShunt] Invalid pin config"));
return false;
}
Expand Down

0 comments on commit 4ee49a6

Please sign in to comment.