Add support for W6300-EVB-Pico and W6300-EVB-Pico2 boards #2999

Merged
wiznet-mason merged 2 commits from W6300 into master 2025-06-19 21:16:54 +03:00
wiznet-mason commented 2025-06-18 10:51:58 +03:00 (Migrated from github.com)

Hello,

This PR adds support for the W6300-EVB-Pico and W6300-EVB-Pico2 boards from WIZnet.
These boards combine RP2040 or RP2350 with WIZnet's W6300 Ethernet controller via a QSPI interface implemented using PIO.

https://docs.wiznet.io/Product/iEthernet/W6300/w6300-evb-pico
https://docs.wiznet.io/Product/iEthernet/W6300/w6300-evb-pico2

Thank you very much for your consideration!

Hello, This PR adds support for the W6300-EVB-Pico and W6300-EVB-Pico2 boards from WIZnet. These boards combine RP2040 or RP2350 with WIZnet's W6300 Ethernet controller via a QSPI interface implemented using PIO. https://docs.wiznet.io/Product/iEthernet/W6300/w6300-evb-pico https://docs.wiznet.io/Product/iEthernet/W6300/w6300-evb-pico2 Thank you very much for your consideration!
earlephilhower (Migrated from github.com) requested changes 2025-06-18 18:14:26 +03:00
@@ -0,0 +8,4 @@
const char* host = "djxmmx.net";
const uint16_t port = 17;
Wiznet6300lwIP eth(1 /* chip select */);
earlephilhower (Migrated from github.com) commented 2025-06-18 18:14:21 +03:00

I think this is not going to work, the CS needs to be set to the proper GPIO. Had the same issue on the other combined WizNet+Pico chipset.

Per the schematics it seems like it should be Wiznet6300lwIP eth(16), no?

I think this is not going to work, the CS needs to be set to the proper GPIO. Had the same issue on the other combined WizNet+Pico chipset. Per the schematics it seems like it should be `Wiznet6300lwIP eth(16)`, no?
wiznet-mason commented 2025-06-19 04:58:08 +03:00 (Migrated from github.com)

Hi,

Thanks for your feedback. However, since W6300 requires PIO to emulate GPIOs to be QSPI pins, therefore we included definitions for all related pins in w6300.h. Thus it is not needed to define separately.

Upon tests on our side, everything works smoothly. Let us know if this approach is wrong and we will fix it for this chip and others too.

Hi, Thanks for your feedback. However, since W6300 requires PIO to emulate GPIOs to be QSPI pins, therefore we included definitions for all related pins in w6300.h. Thus it is not needed to define separately. Upon tests on our side, everything works smoothly. Let us know if this approach is wrong and we will fix it for this chip and others too.
earlephilhower (Migrated from github.com) approved these changes 2025-06-19 21:16:45 +03:00
earlephilhower (Migrated from github.com) left a comment

Thanks for verifying that it does work. The issue with the CS pin needing to be set properly was seen on the W55RP20 #2903 but I suppose this is a whole QSPI implementation and not really related.

Thanks for verifying that it does work. The issue with the CS pin needing to be set properly was seen on the W55RP20 #2903 but I suppose this is a whole QSPI implementation and not really related.
wiznet-mason commented 2025-06-20 05:07:10 +03:00 (Migrated from github.com)

Thank you for merging.
Should you have any issues, please contact me.

Thank you for merging. Should you have any issues, please contact me.
Sign in to join this conversation.