Skip to content

Conversation

JakubSzczudlo
Copy link

Problem accounter when I wanted to use TFTP to send files from server to raspberry. I found some dead links like the ones below:
https://lore.kernel.org/all/CANAwSgRQMQwTO0GjFpk0a9LHGhcwC2_C-YvWao8FnuQavZibRw@mail.gmail.com/T/
https://lists.denx.de/pipermail/u-boot/2024-February/546677.html
After some debugging I find the solution. It's easy to understand, don't break things and doesn't modify uboot in big manner
- What I did
Fixes problems with init of ethernet device in uboot
- How I did it
Add default MAC value not only for sequence 0 but also for sequence 1 device.

Also added screenshots
before
after_patch

@JakubSzczudlo JakubSzczudlo requested a review from agherzan March 22, 2025 17:16
@agherzan agherzan force-pushed the fix/rpi_3b_plus_uboot_ethernet branch from 1ca9b5f to ed9b0cc Compare August 20, 2025 10:33
Copy link
Owner

@agherzan agherzan left a comment

Choose a reason for hiding this comment

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

Thanks for this. Could you squash the commits and check https://meta-raspberrypi.readthedocs.io/en/latest/contributing.html for general contribution guidelines? Do you foresee any impact on non-RPi3+ boards?

@JakubSzczudlo JakubSzczudlo force-pushed the fix/rpi_3b_plus_uboot_ethernet branch from ed9b0cc to c65c4a5 Compare August 22, 2025 17:34
When initializing network devices on raspberry pi 3b+ the lan78xx_eth get seq 1. Because of that
this network device doesn't get MAC address from BCM2835_MBOX and fail to bind device. To address
this issue and don't make much changes in u-boot we can also add MAC address for device of
sequence number=1.

Signed-off-by: Jakub Szczudlo <[email protected]>
@JakubSzczudlo JakubSzczudlo force-pushed the fix/rpi_3b_plus_uboot_ethernet branch from c65c4a5 to 782443a Compare August 22, 2025 17:37
@JakubSzczudlo
Copy link
Author

@agherzan commits squashed and commit message updated as in guidelines.

I don't see any impact on other raspberries because each one have only one ethernet port.

@JakubSzczudlo JakubSzczudlo requested a review from agherzan August 22, 2025 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants