From 4ae905ffd54ddb7b761a085857d11117adf6ebdb Mon Sep 17 00:00:00 2001 From: Michael Croes Date: Thu, 14 Mar 2019 22:25:25 +0100 Subject: [PATCH] Set PDU size to 960 Allows communication with Siemens Logo 0BA8, currently no Siemens PLC is known to support >960 PDU size. --- S7.Net.UnitTest/ConnectionRequestTest.cs | 2 +- S7.Net/PLCHelpers.cs | 2 +- S7.Net/Protocol/ConnectionRequest.cs | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/S7.Net.UnitTest/ConnectionRequestTest.cs b/S7.Net.UnitTest/ConnectionRequestTest.cs index db7cabf..440a962 100644 --- a/S7.Net.UnitTest/ConnectionRequestTest.cs +++ b/S7.Net.UnitTest/ConnectionRequestTest.cs @@ -75,7 +75,7 @@ namespace S7.Net.UnitTest destTsap1, destTsap2, //Destination TASP 192, //Parameter Code (tpdu-size) 1, //Parameter Length - 11 //TPDU Size (2^11 = 2048) + 10 //TPDU Size (2^11 = 2048) }; } } diff --git a/S7.Net/PLCHelpers.cs b/S7.Net/PLCHelpers.cs index 30aaaf0..d4e5f18 100644 --- a/S7.Net/PLCHelpers.cs +++ b/S7.Net/PLCHelpers.cs @@ -186,7 +186,7 @@ namespace S7.Net private byte[] GetS7ConnectionSetup() { return new byte[] { 3, 0, 0, 25, 2, 240, 128, 50, 1, 0, 0, 255, 255, 0, 8, 0, 0, 240, 0, 0, 3, 0, 3, - 7, 128 //Try 1920 PDU Size. Same as libnodave. + 3, 192 // Use 960 PDU size }; } diff --git a/S7.Net/Protocol/ConnectionRequest.cs b/S7.Net/Protocol/ConnectionRequest.cs index 0535cb2..f0a4567 100644 --- a/S7.Net/Protocol/ConnectionRequest.cs +++ b/S7.Net/Protocol/ConnectionRequest.cs @@ -21,7 +21,7 @@ namespace S7.Net.Protocol 3, 0, //Destination TASP 192, //Parameter Code (tpdu-size) 1, //Parameter Length - 11 //TPDU Size (2^11 = 2048) + 10 //TPDU Size (2^10 = 1024) }; switch (cpu)