Skip to content

boards/mips: Add networking support to CI20 board#19459

Merged
acassis merged 1 commit into
apache:masterfrom
ldube:ci20
Jul 17, 2026
Merged

boards/mips: Add networking support to CI20 board#19459
acassis merged 1 commit into
apache:masterfrom
ldube:ci20

Conversation

@ldube

@ldube ldube commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Summary

Introduce networking capabilities to the Creator CI20 board by leveraging the pre-existing dm9000 ethernet driver.

To achieve this, the following changes were made:

  • Integrated the jz4780 GPIO module to properly configure and enable the interrupt pin required by the ethernet controller.
  • Added dm90x0.h to export the dm9000 initialization function, allowing the board-specific setup code to initialize the network interface.

Testing

$
$ ping 192.168.2.3
PING 192.168.2.3 (192.168.2.3) 56(84) bytes of data.
64 bytes from 192.168.2.3: icmp_seq=1 ttl=64 time=0.301 ms
64 bytes from 192.168.2.3: icmp_seq=2 ttl=64 time=0.333 ms
64 bytes from 192.168.2.3: icmp_seq=3 ttl=64 time=0.356 ms
^C
--- 192.168.2.3 ping statistics ---
3 packets transmitted, 3 received, 0% packet loss, time 2025ms
rtt min/avg/max/mdev = 0.301/0.330/0.356/0.022 ms
$ telnet 192.168.2.3
Trying 192.168.2.3...
Connected to 192.168.2.3.
Escape character is '^]'.

NuttShell (NSH) NuttX-13.0.0
nsh> uname -a
NuttX 13.0.0 a02f926 Jul 16 2026 10:55:00 mips ci20
nsh> ls /dev
/dev:
console
null
telnet
telnet0
ttyS0
zero
nsh>

@github-actions github-actions Bot added Area: Networking Effects networking subsystem Arch: mips Issues related to the MIPS architecture Size: L The size of the change in this PR is large Board: arm Board: mips labels Jul 16, 2026
@github-actions

github-actions Bot commented Jul 16, 2026

Copy link
Copy Markdown

MemBrowse Memory Report

No memory changes detected for:

Comment thread boards/arm/dm320/ntosd-dm320/src/dm320_network.c
linguini1
linguini1 previously approved these changes Jul 16, 2026

@linguini1 linguini1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! Can you add the new config under Configurations in the docs like you did with the nsh config in the last patch?

acassis
acassis previously approved these changes Jul 16, 2026
jerpelea
jerpelea previously approved these changes Jul 17, 2026
@acassis

acassis commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

@xiaoxiang781216 could you please review? Please wait for the board profile Documentation

@xiaoxiang781216

Copy link
Copy Markdown
Contributor

@xiaoxiang781216 could you please review? Please wait for the board profile Documentation

I am waiting @ldube fix the ci break, the change look good.

@ldube

ldube commented Jul 17, 2026

Copy link
Copy Markdown
Contributor Author

@xiaoxiang781216 You committed the fix, I think, e35129b

@ldube
ldube dismissed stale reviews from xiaoxiang781216, acassis, jerpelea, and linguini1 via 471bc89 July 17, 2026 15:02
@github-actions github-actions Bot added Area: Documentation Improvements or additions to documentation and removed Area: Networking Effects networking subsystem labels Jul 17, 2026
@ldube

ldube commented Jul 17, 2026

Copy link
Copy Markdown
Contributor Author

@linguini1 Yes, I updated the document.

linguini1
linguini1 previously approved these changes Jul 17, 2026
Introduce networking capabilities to the Creator CI20 board by leveraging
the pre-existing dm9000 ethernet driver.

To achieve this, the following changes were made:
- Integrated the jz4780 GPIO module to properly configure and enable the
  interrupt pin required by the ethernet controller.
- Added `dm90x0.h` to export the dm9000 initialization function, allowing
  the board-specific setup code to initialize the network interface.

Signed-off-by: Lwazi Dube <lwazeh@gmail.com>
@acassis

acassis commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

@xiaoxiang781216 You committed the fix, I think, e35129b

Sorry @ldube I only see your question now. I think you already figured it out. When you add a new board profile you should add Documentation to let people know how to use it

@acassis acassis left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ldube please change the "net" name to "netnsh" this is the default name that Greg selected when telnet is used to access the nsh.
We have this standard namings:
nsh -> only basic serial console access
netnsh -> nsh access over telnet
usbnsh -> nsh access over CDC/ACM device

You can look at other boards to see this pattern, maybe we need to document this rule in some place to avoid contributors to spend time with it.

I didn't pay attention to it when I approved this PR, only after reading your Documentation I noticed your goal to use telnet to access nsh.

@acassis

acassis commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

I'm confuse here, I noticed you enabled "CONFIG_16550_UART0_SERIAL_CONSOLE=y", so your goal is not to have netnsh be default, correct? If this is the case "net" name is fine!

@acassis
acassis merged commit 2f73fe2 into apache:master Jul 17, 2026
54 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Arch: mips Issues related to the MIPS architecture Area: Documentation Improvements or additions to documentation Board: mips 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