Skip to content

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

Merged
merged 2 commits into from
Mar 12, 2025

Conversation

soburi
Copy link
Member

@soburi soburi commented Jan 28, 2025

Add initial support for W5500 EVB Pico2


note:

#84834 (comment)

I am aware that the tests/net/conn_mgr_monitor/net.conn_mgr.dad test will fail if the netif:eth function support is explicitly stated in yaml, but I will submit it following the settings of w5500_evb_pico, which only differs in soc.

#85816

@soburi soburi added the platform: Raspberry Pi Pico Raspberry Pi Pico (RPi Pico) label Jan 28, 2025
Comment on lines 1 to 4
.. _w5500_evb_pico2:

Wiznet W5500 Evaluation Pico
############################
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
.. _w5500_evb_pico2:
Wiznet W5500 Evaluation Pico
############################
.. zephyr:board:: w5500_evb_pico2

Comment on lines 33 to 39

.. figure:: img/w5500_evb_pico2_side.png
:align: center
:alt: W5500 Evaluation Board

Wiznet W5500_EVB_PICO evaluation board (Image courtesy of Wiznet)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
.. 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.
Copy link
Collaborator

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
Copy link
Collaborator

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

Copy link
Member Author

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.

Copy link
Collaborator

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!

Copy link
Member Author

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.

Copy link
Member Author

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
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
The w5500_evb_pico2 board configuration supports the following
The ``w5500_evb_pico2`` board supports the following

@soburi
Copy link
Member Author

soburi commented Feb 3, 2025

@raveious

If you know or can try it, could you let me know whether the tests/net/conn_mgr_monitor/net.conn_mgr.dad test passes on the w5500_evb_pico device?

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

@soburi soburi added the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Feb 5, 2025
@zephyrbot
Copy link
Collaborator

zephyrbot commented Feb 5, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff

All manifest checks OK

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@raveious
Copy link
Contributor

raveious commented Feb 8, 2025

@raveious

If you know or can try it, could you let me know whether the tests/net/conn_mgr_monitor/net.conn_mgr.dad test passes on the w5500_evb_pico device?

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

The suite is filtered out for the w5500_evb_pico target, so to "technically' runs successfully.

@soburi
Copy link
Member Author

soburi commented Feb 11, 2025

@raveious
If you know or can try it, could you let me know whether the tests/net/conn_mgr_monitor/net.conn_mgr.dad test passes on the w5500_evb_pico device?
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

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...

@soburi soburi force-pushed the w5500_evb_pico2 branch 2 times, most recently from 7d574bc to ec6f320 Compare February 11, 2025 21:32
@zephyrbot zephyrbot removed manifest manifest-hal_rpi_pico DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Feb 11, 2025
@soburi soburi force-pushed the w5500_evb_pico2 branch 2 times, most recently from da089f4 to 87c9c53 Compare February 19, 2025 23:24
@soburi soburi force-pushed the w5500_evb_pico2 branch 3 times, most recently from 53b3ede to 286ac1e Compare March 7, 2025 23:51
@soburi soburi changed the title boards: Add suupport WizNet W5500 EVB Pico2 boards: Add support WizNet W5500 EVB Pico2 Mar 7, 2025
@soburi
Copy link
Member Author

soburi commented Mar 8, 2025

@kartben @nordicjm @raveious
I have made some general revisions and removed the draft. Please review it again.

@soburi soburi marked this pull request as ready for review March 8, 2025 00:48
@soburi soburi requested review from kartben and nordicjm March 8, 2025 00:48
@zephyrbot zephyrbot added the area: ADC Analog-to-Digital Converter (ADC) label Mar 8, 2025
@zephyrbot zephyrbot requested a review from anangl March 8, 2025 00:49
Copy link
Contributor

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.

Copy link
Member Author

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
Copy link
Contributor

Choose a reason for hiding this comment

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

2025?

Copy link
Member Author

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.

@soburi soburi force-pushed the w5500_evb_pico2 branch 3 times, most recently from 6e331c2 to 35c8f10 Compare March 8, 2025 02:25
@@ -0,0 +1,6 @@
board:
name: w5500_evb_pico2
full_name: W5500 Evaluation Pico2
Copy link
Collaborator

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

Comment on lines 3 to 5
Wiznet W5500 Evaluation Board Pico2
###################################

Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Wiznet W5500 Evaluation Board Pico2
###################################

Comment on lines 34 to 39

.. figure:: img/w5500_evb_pico2.webp
:align: center
:alt: W5500 Evaluation Board

Wiznet W5500_EVB_PICO evaluation board (Image courtesy of Wiznet)
Copy link
Collaborator

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

Suggested change
.. figure:: img/w5500_evb_pico2.webp
:align: center
:alt: W5500 Evaluation Board
Wiznet W5500_EVB_PICO evaluation board (Image courtesy of Wiznet)

@soburi soburi force-pushed the w5500_evb_pico2 branch from 35c8f10 to 2564540 Compare March 8, 2025 03:01
kartben
kartben previously approved these changes Mar 8, 2025
group1 {
pinmux = <UART0_TX_P0>;
};
group2 {
Copy link
Collaborator

Choose a reason for hiding this comment

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

newline needed

Copy link
Member Author

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
Copy link
Collaborator

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)

Copy link
Member Author

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.

soburi added 2 commits March 10, 2025 18:09
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>
@soburi
Copy link
Member Author

soburi commented Mar 10, 2025

@raveious
Sorry, but could you check again?

@raveious
Copy link
Contributor

@raveious Sorry, but could you check again?

Same test(s) as before? I'll take a look later tonight.

@soburi
Copy link
Member Author

soburi commented Mar 10, 2025

@raveious Sorry, but could you check again?

Same test(s) as before? I'll take a look later tonight.

The tests that were failing before now work correctly with the fix in #85816.
Your Change Request was still open, so I'd like you to check if there are any problems.
Thank you.

@kartben kartben merged commit 5b28a82 into zephyrproject-rtos:main Mar 12, 2025
17 checks passed
@soburi soburi deleted the w5500_evb_pico2 branch March 12, 2025 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: ADC Analog-to-Digital Converter (ADC) platform: Raspberry Pi Pico Raspberry Pi Pico (RPi Pico)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants