libmodbus icon indicating copy to clipboard operation
libmodbus copied to clipboard

Encapsulated support (RTU over TCP) (updated #260)

Open v-zhuravlev opened this issue 8 years ago • 14 comments

Hello, updated this pull request https://github.com/stephane/libmodbus/pull/260 to 3.1.4 and main branch(as #260 was based on 3.0.3 as far as I see it).

It would be great if encapsulated is supported in the main branch somehow.

Thanks.

v-zhuravlev avatar Jun 01 '17 11:06 v-zhuravlev

It will be great if encapsulated will be supported in the main branch

fatkhrus avatar Aug 10 '17 16:08 fatkhrus

ping?

evgkrsk avatar Oct 07 '17 10:10 evgkrsk

why is this not merged? I need this patch 😄

strahlc avatar Jan 25 '18 15:01 strahlc

Me too

25 sty 2018 4:36 PM "Christian Strahl" [email protected] napisał(a):

why is this not merged? I need this patch 😄

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/stephane/libmodbus/pull/385#issuecomment-360503192, or mute the thread https://github.com/notifications/unsubscribe-auth/AAK-69w8rjuyttd45AOgFjWwzPqoSV-Uks5tOJ-LgaJpZM4Ns09C .

kiug avatar Jan 25 '18 16:01 kiug

This patch is very useful when using RS485 over IP converters without installing virtual com-port software, can it be merged in mainline?

bashlakov avatar Feb 15 '18 11:02 bashlakov

@v-zhuravlev I think it would help the integration (if Stephane finds the time) if there would a test for this functionality. Having said that, I know that, I wouldn't know how to test as it seems to be only a client-side implementation.

Maybe with the _raw-function somehow.

And documentation for the rtutcp-function is needed as well.

pboettch avatar Feb 15 '18 14:02 pboettch

I’m not sure that I’m capable to do all that properly. But I m ready to help if given proper guideance.

Отправлено с телефона

15 февр. 2018 г., в 17:38, Patrick Boettcher [email protected] написал(а):

@v-zhuravlev I think it would help the integration (if Stephane finds the time) if there would a test for this functionality. Having said that, I know that, I wouldn't know how to test as it seems to be only a client-side implementation.

And documentation for the rtutcp-function is needed as well.

Maybe with the _raw-function somehow.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

v-zhuravlev avatar Feb 15 '18 14:02 v-zhuravlev

For the documentation part you can inspire yourself with what's done in modbus_tcp_connect().

For the testing part, ideally we'd also support rtutcp-server here in libmodbus (if that makes sense) then writing tests would be simpler.

pboettch avatar Feb 15 '18 15:02 pboettch

But are there any plans to include this in the first place ? Implementing server is overkill I think

Отправлено с телефона

15 февр. 2018 г., в 18:12, Patrick Boettcher [email protected] написал(а):

For the documentation part you can inspire yourself with what's done in modbus_tcp_connect().

For the testing part, ideally we'd also support rtutcp-server here in libmodbus (if that makes sense) then writing tests would be simpler.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

v-zhuravlev avatar Feb 15 '18 15:02 v-zhuravlev

I think it will be included. It's just a question of time.

pboettch avatar Feb 15 '18 15:02 pboettch

Please consider my approach (#445). It contains both server, client, test, documentation code. however I assumed that the slave id must be set and considered. Please comment especially whether the slave id should be considered or not. I assumed RTU considers it so RTU over TCP should do it, too.

myssta avatar Aug 30 '18 17:08 myssta

any updates on this?

strahlc avatar Sep 04 '20 11:09 strahlc

It is so annoying to keep adapting the patches to new versions of this library. Please merged this MR.

strahlc avatar Feb 13 '23 16:02 strahlc