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 new TVBox aml-c400-plus: Magicsee C400 Plus #7581

Merged
merged 1 commit into from
Dec 18, 2024

Conversation

jomadeto
Copy link
Contributor

Description

This PR adds support for the Magicsee C400 Plus

http://m.magicsees.com/m/prod_view.aspx?TypeId=66&Id=191&Fid=t3:66:3

Boot directly with moddified s905_autoscript and emmc_autocript without need of u-boot (cannot boot from emmc with u-boot)

Documentation summary for feature / change

With generic S9xx builds and with aml-c400-plus.dtb all working well from SD and USB but cannot boot from EMMC (start u-boot but not detect EMMC).

After a lot of test and error get working everything with SD, USB and EMMC without u-boot, only need correct s905_autoscript and emmc_autoscript. Also need few changes for use armbianEnv.txt instead extlinux.conf, and correct in root/install-aml.sh.

Maybe with this changes another old tvbox with s912 processor can boot from EMMC changing only dtb(read in forums that this problem is generic).

How Has This Been Tested?

  • ./compile.sh build BOARD=aml-c400-plus BRANCH=current BUILD_DESKTOP=yes BUILD_MINIMAL=no DESKTOP_APPGROUPS_SELECTED='desktop_tools editors internet multimedia office programming' DESKTOP_ENVIRONMENT=xfce DESKTOP_ENVIRONMENT_CONFIG_NAME=config_base KERNEL_CONFIGURE=no RELEASE=noble

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 Needs review Seeking for review Hardware Hardware related like kernel, U-Boot, ... labels Dec 15, 2024
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.

looks good, tvb - if it builds, its ok to merge.

@igorpecovnik igorpecovnik added Ready to merge Reviewed, tested and ready for merge 02 Milestone: First quarter release and removed Needs review Seeking for review labels Dec 18, 2024
@igorpecovnik igorpecovnik merged commit 81e8382 into armbian:main Dec 18, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
02 Milestone: First quarter release Hardware Hardware related like kernel, U-Boot, ... Ready to merge Reviewed, tested and ready for merge size/large PR with 250 lines or more
Development

Successfully merging this pull request may close these issues.

2 participants