-
Notifications
You must be signed in to change notification settings - Fork 7.6k
boards: Add support WizNet W5500 EVB Pico2 #84834
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
.. _w5500_evb_pico2: | ||
|
||
Wiznet W5500 Evaluation Pico | ||
############################ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.. _w5500_evb_pico2: | |
Wiznet W5500 Evaluation Pico | |
############################ | |
.. zephyr:board:: w5500_evb_pico2 |
|
||
.. figure:: img/w5500_evb_pico2_side.png | ||
:align: center | ||
:alt: W5500 Evaluation Board | ||
|
||
Wiznet W5500_EVB_PICO evaluation board (Image courtesy of Wiznet) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.. figure:: img/w5500_evb_pico2_side.png | |
:align: center | |
:alt: W5500 Evaluation Board | |
Wiznet W5500_EVB_PICO evaluation board (Image courtesy of Wiznet) |
Pin Mapping | ||
=========== | ||
|
||
The peripherals of the RP2040 SoC can be routed to various pins on the board. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
RP2040? 🤔
Overview | ||
******** | ||
|
||
W5500-EVB-Pico is a microcontroller evaluation board based on the Raspberry |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pico --> Pico2? Looks like you need to do a global pass just like with RP2040 vs RP2350
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, this is still draft...
I haven't written the documentation yet.
I plan to update it based on rp2040.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry didn't notice it was in draft, my bad!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't mind. Thank you for caring.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have made some general revisions and removed the draft. Please review it again.
Supported Features | ||
================== | ||
|
||
The w5500_evb_pico2 board configuration supports the following |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The w5500_evb_pico2 board configuration supports the following | |
The ``w5500_evb_pico2`` board supports the following |
If you know or can try it, could you let me know whether the When I tried to check the operation of w5500_evb_pico2, I ran the test using the following command, but it failed. So, I would like to know the result on w5500_evb_pico. west twister --device-testing --device-serial /dev/ttyACM0 -p w5500_evb_pico2/rp2350a/m33 -s tests/net/conn_mgr_monitor/net.conn_mgr.dad -xOPENOCD=/usr/local/bin/openocd |
80727a3
to
e4de44b
Compare
The following west manifest projects have changed revision in this Pull Request:
✅ All manifest checks OK Note: This message is automatically posted and updated by the Manifest GitHub Action. |
The suite is filtered out for the w5500_evb_pico target, so to "technically' runs successfully. |
Thank you for the information. I understand the background. However, It is seems we need to consideration... |
7d574bc
to
ec6f320
Compare
ec6f320
to
290416e
Compare
da089f4
to
87c9c53
Compare
53b3ede
to
286ac1e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I could be wrong, but I think the silk screen says "W5500 EVB PICO" on it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, It's a terrible mistake...
Thank you for catching it.
I fetched the correct image and replaced it from https://docs.wiznet.io/Product/iEthernet/W5500/w5500-evb-pico2.
@@ -0,0 +1,11 @@ | |||
# Copyright (c) 2024 TOKITA Hiroshi |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2025?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your pointing out.
Fixed it.
6e331c2
to
35c8f10
Compare
@@ -0,0 +1,6 @@ | |||
board: | |||
name: w5500_evb_pico2 | |||
full_name: W5500 Evaluation Pico2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
W5500-EVB-Pico2
Same mistake was made for the Pico-based board -- this should be the commercial name
Wiznet W5500 Evaluation Board Pico2 | ||
################################### | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wiznet W5500 Evaluation Board Pico2 | |
################################### |
|
||
.. figure:: img/w5500_evb_pico2.webp | ||
:align: center | ||
:alt: W5500 Evaluation Board | ||
|
||
Wiznet W5500_EVB_PICO evaluation board (Image courtesy of Wiznet) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Zephyr board sphinx directive takes care of adding the image to the overview box
.. figure:: img/w5500_evb_pico2.webp | |
:align: center | |
:alt: W5500 Evaluation Board | |
Wiznet W5500_EVB_PICO evaluation board (Image courtesy of Wiznet) |
group1 { | ||
pinmux = <UART0_TX_P0>; | ||
}; | ||
group2 { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
newline needed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed it.
@@ -0,0 +1,11 @@ | |||
CONFIG_SYS_CLOCK_HW_CYCLES_PER_SEC=150000000 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should be set in the soc Kconfig
file from the dts property (can be fixed in another PR)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Moved it into Kconfig.defconfig.
Add initial support for W5500 EVB Pico2 Signed-off-by: TOKITA Hiroshi <tokita.hiroshi@gmail.com>
Use `W5500-EVB_Pico` - the commercial name of this board - for `full_name` field of `board.yml`. Signed-off-by: TOKITA Hiroshi <tokita.hiroshi@gmail.com>
@raveious |
Same test(s) as before? I'll take a look later tonight. |
Add initial support for W5500 EVB Pico2
note:#84834 (comment)I am aware that thetests/net/conn_mgr_monitor/net.conn_mgr.dad
test will fail if thenetif:eth
function support is explicitly stated in yaml, but I will submit it following the settings ofw5500_evb_pico
, which only differs in soc.#85816