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.

Sign up for GitHub

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

Jump to bottom

Support CherryBa M1 board #7226

Merged
merged 2 commits into from
Sep 13, 2024
Merged

Support CherryBa M1 board #7226

merged 2 commits into from
Sep 13, 2024

Conversation

IsMrX
Copy link
Contributor

@IsMrX IsMrX commented Sep 13, 2024 •
edited
Loading

Description

Support CherryBa M1 board
Hardware: allwinner H618 LPDDR4 1G/2G EMMC 8G/16G
Size: 3.52mm*2.82mm
titile

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

@github-actions github-actions bot added size/large PR with 250 lines or more Hardware Hardware related like kernel, U-Boot, ... Patches Patches related to kernel, U-Boot, ... labels Sep 13, 2024
igorpecovnik
igorpecovnik requested changes Sep 13, 2024
View reviewed changes
Copy link
Member

@igorpecovnik igorpecovnik left a comment

Choose a reason for hiding this comment

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

Please provide some test results - does it boot, which features were tested.

config/boards/cherryba-m1.csc Outdated Show resolved Hide resolved
config/boards/cherryba-m1.csc Show resolved Hide resolved
@igorpecovnik igorpecovnik added 11 Milestone: Fourth quarter release Needs review Seeking for review labels Sep 13, 2024
@IsMrX
Copy link
Contributor Author

IsMrX commented Sep 13, 2024 •
edited
Loading

Please provide some test results - does it boot, which features were tested.

All peripherals supported by the board have been tested and are running normally
1.emmc
2.tf
3.hdmi & hdmi audio
4.100Mhz Ethernet
5.I2C
6.GPIO

Do I need to upload a picture of my test results?

@igorpecovnik igorpecovnik added the Ready to merge Reviewed, tested and ready for merge label Sep 13, 2024
@igorpecovnik
Copy link
Member

Do I need to upload a picture of my test results?

No, its fine. Thank you.

@igorpecovnik igorpecovnik merged commit ce938c6 into armbian:main Sep 13, 2024
10 checks passed
@igorpecovnik
Copy link
Member

@IsMrX does not build https://paste.armbian.de/uwalinecez

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@igorpecovnik igorpecovnik igorpecovnik approved these changes

@AGM1968 AGM1968 Awaiting requested review from AGM1968 AGM1968 is a code owner

@JohnTheCoolingFan JohnTheCoolingFan Awaiting requested review from JohnTheCoolingFan JohnTheCoolingFan is a code owner

@PanderMusubi PanderMusubi Awaiting requested review from PanderMusubi PanderMusubi is a code owner

@Tonymac32 Tonymac32 Awaiting requested review from Tonymac32 Tonymac32 is a code owner

@janprunk janprunk Awaiting requested review from janprunk janprunk is a code owner

@krachlatte krachlatte Awaiting requested review from krachlatte krachlatte is a code owner

@pyavitz pyavitz Awaiting requested review from pyavitz pyavitz is a code owner

@schwar3kat schwar3kat Awaiting requested review from schwar3kat schwar3kat is a code owner

@sgjava sgjava Awaiting requested review from sgjava sgjava is a code owner

@teknoid teknoid Awaiting requested review from teknoid teknoid is a code owner

Assignees
No one assigned
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/large PR with 250 lines or more
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@IsMrX @igorpecovnik