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

nanopi-r3s: add support for current #7768

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

EvilOlaf
Copy link
Member

Description

  • Bump edge u-boot to 2025.01
  • Add current kernel support
  • Remove vendor related stuff from board config

How Has This Been Tested?

  • Build
  • Boot - untested, no hw

Checklist:

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

ToDo:

Verify functionality of test images:
https://fi.mirror.armbian.de/.testing/Armbian-unofficial_25.02.0-trunk_Nanopi-r3s_trixie_edge_6.13.0_minimal.img.xz
https://fi.mirror.armbian.de/.testing/Armbian-unofficial_25.02.0-trunk_Nanopi-r3s_trixie_current_6.12.11_minimal.img.xz

Check whether it is a good idea to make changes to the network driver board specific only

@github-actions github-actions bot added the size/large PR with 250 lines or more label Jan 31, 2025
add support for `current`
@github-actions github-actions bot added Needs review Seeking for review Hardware Hardware related like kernel, U-Boot, ... Patches Patches related to kernel, U-Boot, ... labels Jan 31, 2025
@EvilOlaf
Copy link
Member Author

@shenyunet @xperiazu21 If you got a chance feel free to test

@shenyunet
Copy link

shenyunet commented Jan 31, 2025

dmesg.txt
OK

@xperiazu21
Copy link

I tested both the current and edge builds using SD and eMMC, and both images were able to boot successfully.
The network ports and LEDs are working properly, and both network ports performed normally in iperf3 speed tests.

I recommend modifying the board name and specification description in the nanopi-r3s.csc file to maintain consistency with the previous NanoPi series.
# Rockchip RK3566 quad core 1GB/2GB RAM eMMC 2x GbE USB3
BOARD_NAME="NanoPi R3S"

@EvilOlaf
Copy link
Member Author

EvilOlaf commented Feb 1, 2025

Thank you.

When I go to their website it is only available with 2GB memory. Where did you get it with 1 since you mention?

@EvilOlaf EvilOlaf marked this pull request as ready for review February 1, 2025 05:34
@xperiazu21
Copy link

Thank you.

When I go to their website it is only available with 2GB memory. Where did you get it with 1 since you mention?

On FriendlyELEC's Taobao store, both 1GB and 2GB RAM options are available, with an optional 32GB eMMC for both.

I only have the 2GB/32GB version, so whether the current image works on the 1GB RAM version may require testing.

@EvilOlaf
Copy link
Member Author

EvilOlaf commented Feb 1, 2025

I see. Well for now I prefer sticking to the info from the official website rather than a 3rd party store. Other info was updated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Hardware Hardware related like kernel, U-Boot, ... Needs review Seeking for review Patches Patches related to kernel, U-Boot, ... size/large PR with 250 lines or more
Development

Successfully merging this pull request may close these issues.

3 participants