Skip to content

add nxmodbus doc and examples#18755

Open
raiden00pl wants to merge 4 commits intoapache:masterfrom
raiden00pl:nxmodbus
Open

add nxmodbus doc and examples#18755
raiden00pl wants to merge 4 commits intoapache:masterfrom
raiden00pl:nxmodbus

Conversation

@raiden00pl
Copy link
Copy Markdown
Member

Summary

Add documentation for NxModbus and examples

Impact

doc and examples

Testing

nucleo-c071rb as RTU server
simulator as RTU server and RTU client
simulator as ASCII server
qemu-intel64 as TPC server and TCP client

simbit18
simbit18 previously approved these changes Apr 17, 2026
@github-actions github-actions bot added Area: Documentation Improvements or additions to documentation Size: L The size of the change in this PR is large Board: arm Board: simulator Board: x86_64 labels Apr 17, 2026
linguini1
linguini1 previously approved these changes Apr 18, 2026
acassis
acassis previously approved these changes Apr 18, 2026
add nxmodbus doc

Signed-off-by: raiden00pl <raiden00@railab.me>
add nxmodbus tools for qemu-intel64/jumbo config

Signed-off-by: raiden00pl <raiden00@railab.me>
add nxmodbus over RTU demo for sim

Signed-off-by: raiden00pl <raiden00@railab.me>
add nxmodbus client and server examples to jumbo

Signed-off-by: raiden00pl <raiden00@railab.me>
@simbit18
Copy link
Copy Markdown
Contributor

simbit18 commented Apr 20, 2026

In this case, we are faced with a situation identical to this PR #18744 (comment)

the job Linux (x86_64-01) fails here because
During the Fetch-Source phase

  • use @raiden00pl nuttx branch as a reference
  • use Apache apps master as a reference

Since the Apache apps master doesn't contain the additional changes required, it rightly fails

I tested it here

I only tested it for the job Linux (x86_64-01)

using this new #18568 tool and didn’t encounter any issues

Set it up as follows:

Nuttx repository: raiden00pl/nuttx
    Nuttx branch: nxmodbus
 Apps repository: raiden00pl/nuttx-apps
     Apps branch: nxmodbus
    Architecture: x86_64
   Host platform: linux

@raiden00pl Have you run any tests on your repository?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Documentation Improvements or additions to documentation Board: arm Board: simulator Board: x86_64 Size: L The size of the change in this PR is large

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants