Skip to content
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

Add Radxa E20C board #7157

Merged
merged 3 commits into from
Sep 13, 2024
Merged

Add Radxa E20C board #7157

merged 3 commits into from
Sep 13, 2024

Conversation

mattx433
Copy link
Contributor

@mattx433 mattx433 commented Aug 31, 2024

Description

Adds the Radxa E20C board, which is based on the Rockchip RK3528 SoC.
U-Boot config from radxa/u-boot@5202c42
Depends on armbian/linux-rockchip#231 (which, as of writing, is not merged) (which is now merged)

How Has This Been Tested?

  • Boots (from microSD)
  • USB serial via debug port works (see note)
  • LAN port works (RK3528 Ethernet)
  • WAN port works (RTL8168 Ethernet)
  • Debug USB-C port works (can run USB gadgets)
  • USB-A port works (can use USB devices)
  • User button works (via /dev/input/event0)
  • LED control works (via /sys/class/leds)
  • eMMC access works

Not tested:

  • RKMPP (did not find any tools that worked with the 6.1 kernel, only 5.10)
  • eMMC boot (should work, though flashing tools are unreliable)

Note that USB serial is unreliable, likely due to the CH340B USB serial chip on the board not being able to handle the RK3528's baudrate. Changing the baudrate can only be done by editing the device tree (as stty will not affect it).

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules

@github-actions github-actions bot added size/medium PR with more then 50 and less then 250 lines Hardware Hardware related like kernel, U-Boot, ... Patches Patches related to kernel, U-Boot, ... labels Aug 31, 2024
@EvilOlaf
Copy link
Member

EvilOlaf commented Sep 1, 2024

The CH430B can handle up to 2Mbaud AFAIK so maybe there is some different issue. Could also be due to poor design which would not be fixable by software anyway.

@RadxaStephen
Copy link

Note that USB serial is unreliable, likely due to the CH340B USB serial chip on the board not being able to handle the RK3528's baudrate. Changing the baudrate can only be done by editing the device tree (as stty will not affect it).

We have noticed that when the OS sets the debug serial port baud rate to 1.5M, the serial port tool will lose characters. When setting baud rate to 115200, the serial port will not lose characters.

We found that it is a problem with CH340B itself.

To solve this problem, we need to use another Pin to Pin IC, CH340C.
We will release a new Radxa E20C with CH340C.

@mattx433 mattx433 marked this pull request as ready for review September 2, 2024 13:28
@igorpecovnik igorpecovnik added 11 Milestone: Fourth quarter release Needs review Seeking for review Good first issue Feel free to contribute :) and removed Good first issue Feel free to contribute :) labels Sep 3, 2024
@igorpecovnik igorpecovnik added the Ready to merge Reviewed, tested and ready for merge label Sep 12, 2024
@igorpecovnik igorpecovnik merged commit 8052919 into armbian:main Sep 13, 2024
@mattx433 mattx433 deleted the add-radxa-e20c branch September 13, 2024 20:18
@overrunner
Copy link

Nice work! Is there a how to for installing Armbian on the E20C available?

@igorpecovnik
Copy link
Member

igorpecovnik commented Sep 16, 2024

Is there a how to for installing Armbian on the E20C available?

This is always the same.
https://docs.armbian.com/User-Guide_Getting-Started/

... but you will need:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
11 Milestone: Fourth quarter release Hardware Hardware related like kernel, U-Boot, ... Needs review Seeking for review Patches Patches related to kernel, U-Boot, ... Ready to merge Reviewed, tested and ready for merge size/medium PR with more then 50 and less then 250 lines
Development

Successfully merging this pull request may close these issues.

6 participants