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

Sunxi-6.11: Fix a broken package for EDGE #7474

Merged
merged 5 commits into from
Nov 14, 2024

Conversation

The-going
Copy link
Contributor

@The-going The-going commented Nov 14, 2024

Description

  • sunxi-6.11: Fix the installation of overlay files in the subdir
  • sunxi-6.11: Switch to v6.11.7
  • sunxi-6.6: Switch to v6.6.60

Close 7456 reference: AR-2529

How Has This Been Tested?

sunxi-6.6:

  • Test build sunxi
  • Test work on BPI-M3
  • Test build sunxi64
  • Test work on BPI-M64

sunxi-6.11:

  • Test build sunxi
  • Test work on BPI-M3
  • Test build sunxi64
  • Test work on BPI-M64

@github-actions github-actions bot added size/medium PR with more then 50 and less then 250 lines Needs review Seeking for review Hardware Hardware related like kernel, U-Boot, ... Patches Patches related to kernel, U-Boot, ... labels Nov 14, 2024
@igorpecovnik igorpecovnik added the 11 Milestone: Fourth quarter release label Nov 14, 2024
@igorpecovnik igorpecovnik added the Ready to merge Reviewed, tested and ready for merge label Nov 14, 2024
@igorpecovnik igorpecovnik merged commit 781400e into armbian:main Nov 14, 2024
12 checks passed
@The-going
Copy link
Contributor Author

Igor, I understand that I want to hurry up.
But we have a remaining inconsistency in the scripts when searching for DTB for the board and overlay files.

@igorpecovnik
Copy link
Member

But we have a remaining inconsistency

Oh? I understand that it works now? As I reverted overlay handling to previous state. Is "current" kernel on both working correctly? This is target for release. Edge we can deal with later.

@The-going
Copy link
Contributor Author

But we have a remaining inconsistency

Oh? I understand that it works now? As I reverted overlay handling to previous state. Is "current" kernel on both working correctly? This is target for release. Edge we can deal with later.

I'll check it out. And I will try to fix the scripts for sunxi so that they are universal.
In other words, we still have a difference in paths:
${prefix}dtb/${fdtfile}
${prefix}dtb/overlay/${overlay_prefix}-${overlay_file}.dtbo
or
${prefix}dtb/allwinner/overlay/${overlay_prefix}-${overlay_file}.dtbo
in files:
config/bootscripts/boot-sunxi.cmd
config/bootscripts/boot-sun50i-next.cmd

And we have different kernels in our repositories, which place their files in different paths:
boot/dtb/overlay
or
boot/dtb/allwinner/overlay

If the user freezes the kernel, but at the same time updates the BSP package, the download script will change.
Conversely, if he installs only a new core, then get a non-bootable board.

Today I will test the current one and try to make the situation easier and make the script more versatile.

@The-going
Copy link
Contributor Author

Is "current" kernel on both working correctly?

  ___                     __  __ _         _ 
 / __|_ __  __ _ __ ___  |  \/  (_)_ _  __| |
 \__ \ '_ \/ _` / _/ -_) | |\/| | | ' \/ _` |
 |___/ .__/\__,_\__\___|_|_|  |_|_|_||_\__,_|
     |_|              |___|                  
 v24.11 rolling for Banana Pi M3 running Armbian Linux 6.6.60-current-sunxi

 Packages:     Debian stable (bookworm)

leo@bananapim3:~$ armbianmonitor -m
Two CPU clusters are available for monitoring
Stop monitoring using [ctrl]-[c]
Time    CPU_cl0/CPU_cl1  load %cpu %sys %usr %nice %io %irq   Tcpu  C.St.

16:09:22   768/ 576 MHz  0.13   5%   2%   2%   0%   0%   0%  45,3 °C  0/5

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.

[Bug]: incorrect package build for sunxi
2 participants