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

BananaPi BPI-M4-Zero: Improve support #7355

Merged
merged 6 commits into from
Oct 9, 2024
Merged

Conversation

pyavitz
Copy link
Collaborator

@pyavitz pyavitz commented Oct 9, 2024

DTS:

  • Removed gpio-keys node (NOTE: add to the M4 Berry DTS)
  • Add vcc1v8 and vcc3v3 regulators
  • Fixup MMC nodes
  • Add bluetooth node (NOTE: BT is still iffy in Armbian)

Wireless firmware:

Arm trusted firmware updated to v2.10.7

The updated linux-6.11.y patches have already been added to the following PR; #7352

Removed gpio-keys node (NOTE: add to the M4 Berry DTS)
Add vcc1v8 and vcc3v3 regulators
Fixup MMC nodes
Add bluetooth node (NOTE: BT is still iffy in Armbian)

Signed-off-by: Patrick Yavitz <pyavitz@gmail.com>
Signed-off-by: Patrick Yavitz <pyavitz@gmail.com>
Signed-off-by: Patrick Yavitz <pyavitz@gmail.com>
Removed gpio-keys node (NOTE: add to the M4 Berry DTS)
Add vcc1v8 and vcc3v3 regulators
Fixup MMC nodes
Add bluetooth node (NOTE: BT is still iffy in Armbian)

Signed-off-by: Patrick Yavitz <pyavitz@gmail.com>
and fixup Makefile

Signed-off-by: Patrick Yavitz <pyavitz@gmail.com>
@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, ... BSP Board Support Packages labels Oct 9, 2024
@igorpecovnik igorpecovnik added Ready to merge Reviewed, tested and ready for merge 11 Milestone: Fourth quarter release labels Oct 9, 2024
@pyavitz pyavitz merged commit fa3eade into armbian:main Oct 9, 2024
@The-going
Copy link
Contributor

Merged !?!

[🔨]   Problem with ->patches.megous/power-supply-axp20x_battery-Add-support-for-reporting-OCV(:1) (+23/-0)[1M] {axp20x_battery.c}<-: Failed to apply patch /home/leo/armbian/patch/kernel/archive/sunxi-6.6/patches.megous/power-supply-axp20x_battery-Add-support-for-reporting-OCV.patch
...
[🔨]   -> 062/426: patches.megous/power-supply-axp20x-battery-Support-POWER_SUPPLY_PROP_CHARGE_BE(:1) (+31/-0)[1M] {axp20x_battery.c}
[🔨]   Problem with ->patches.megous/power-supply-axp20x-battery-Support-POWER_SUPPLY_PROP_CHARGE_BE(:1) (+31/-0)[1M] {axp20x_battery.c}<-: Failed to apply patch /home/leo/armbian/patch/kernel/archive/sunxi-6.6/patches.megous/power-supply-axp20x-battery-Support-POWER_SUPPLY_PROP_CHARGE_BE.patch
....
[🔨]   Failed to apply 2 patches: ->patches.megous/power-supply-axp20x_battery-Add-support-for-reporting-OCV(:1) (+23/-0)[1M] {axp20x_battery.c}<-,->patches.megous/power-supply-axp20x-battery-Support-POWER_SUPPLY_PROP_CHARGE_BE(:1) (+31/-0)[1M] {axp20x_battery.c}<-
[🔨]   Summary: kernel patching: 426 total patches; 424 applied; 30 with problems; 2 failed_apply; 28 needs_rebase; 2 not_mbox

I suspect that very good fixes from this pull request may not be available to the end user.
Fix it #7358

@pyavitz
Copy link
Collaborator Author

pyavitz commented Oct 10, 2024

Merged !?!

[🔨]   Problem with ->patches.megous/power-supply-axp20x_battery-Add-support-for-reporting-OCV(:1) (+23/-0)[1M] {axp20x_battery.c}<-: Failed to apply patch /home/leo/armbian/patch/kernel/archive/sunxi-6.6/patches.megous/power-supply-axp20x_battery-Add-support-for-reporting-OCV.patch
...
[🔨]   -> 062/426: patches.megous/power-supply-axp20x-battery-Support-POWER_SUPPLY_PROP_CHARGE_BE(:1) (+31/-0)[1M] {axp20x_battery.c}
[🔨]   Problem with ->patches.megous/power-supply-axp20x-battery-Support-POWER_SUPPLY_PROP_CHARGE_BE(:1) (+31/-0)[1M] {axp20x_battery.c}<-: Failed to apply patch /home/leo/armbian/patch/kernel/archive/sunxi-6.6/patches.megous/power-supply-axp20x-battery-Support-POWER_SUPPLY_PROP_CHARGE_BE.patch
....
[🔨]   Failed to apply 2 patches: ->patches.megous/power-supply-axp20x_battery-Add-support-for-reporting-OCV(:1) (+23/-0)[1M] {axp20x_battery.c}<-,->patches.megous/power-supply-axp20x-battery-Support-POWER_SUPPLY_PROP_CHARGE_BE(:1) (+31/-0)[1M] {axp20x_battery.c}<-
[🔨]   Summary: kernel patching: 426 total patches; 424 applied; 30 with problems; 2 failed_apply; 28 needs_rebase; 2 not_mbox

I suspect that very good fixes from this pull request may not be available to the end user. Fix it #7358

I had no problems building this on the designated kernel version at the time, which I believe was 6.6.44. So I'm not sure how this relates to this PR.

@The-going
Copy link
Contributor

The-going commented Oct 10, 2024

I had no problems building this on the designated kernel version at the time, which I believe was 6.6.44.

I decided to check the status of this kernel.
There was a question on the forum.
I ran the build and yes it was the v6.6.44 kernel. I saw these messages in the log and did not wait for the build to finish.
In any case, the patches were rebased to the new version and extracted again.

@The-going
Copy link
Contributor

So I'm not sure how this relates to this PR.

You and I are collecting cores using our own methods.
If the assembly in the Armbian framework ends with any error during the release process, the release occurs anyway. The last successfully assembled kernel from the cache is simply being converted to the new version.
I am "delighted" with this technology.
I have no complaints or questions about your work.
If you notice problems with patches next time, just ask me to extract them again.
It's not a problem for me.

@pyavitz
Copy link
Collaborator Author

pyavitz commented Oct 10, 2024

If you notice problems with patches next time, just ask me to extract them again.

Will do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
11 Milestone: Fourth quarter release BSP Board Support Packages Hardware Hardware related like kernel, U-Boot, ... Patches Patches related to 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.

3 participants