8000 aic8800: update firmware package URL by RadxaYuntian · Pull Request #6768 · armbian/build · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

aic8800: update firmware package URL #6768

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
Merged
merged 1 commit into from
Jun 20, 2024
Merged

aic8800: update firmware package URL #6768

me 8000 rged 1 commit into from
Jun 20, 2024

Conversation

RadxaYuntian
Copy link
Collaborator

Description

Firmware package was incorrectly marked as machine dependent.
This was fixed in radxa-pkg/aic8800#12, causing the file name to change.

First reported in #6761

Signed-off-by: ZHANG Yuntian yt@radxa.com

How Has This Been Tested?

Same change was tested by @CT1IQI: #6761 (comment)

There is a seperate runtime error though.

Checklist:

Please delete options that are not relevant.

  • 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

@RadxaYuntian
aic8800: update firmware package URL
b7bdf1c 
Firmware package was incorrectly marked as machine dependent.
This was fixed in radxa-pkg/aic8800#12, causing the file name to change.

Signed-off-by: ZHANG Yuntian <yt@radxa.com>
@github-actions github-actions bot added size/small PR with less then 50 lines Framework Framework components labels Jun 20, 2024
@EvilOlaf EvilOlaf merged commit e0762bb into armbian:main Jun 20, 2024
@RadxaYuntian RadxaYuntian deleted the aic8800_firmware branch June 20, 2024 09:28
ColorfulRhino pushed a commit that referenced this pull request Jun 20, 2024
@RadxaYuntian @ColorfulRhino
aic8800: update firmware package URL (#6768)
08a3f1c 
Firmware package was incorrectly marked as machine dependent.
This was fixed in radxa-pkg/aic8800#12, causing the file name to change.

Signed-off-by: ZHANG Yuntian <yt@radxa.com>
@andyshrk
Copy link
Contributor

Hi @amazingfate and @RadxaYuntian

the aic8800_dkms_file_name for sdio should be aic8800-sdio-dkms_${latest_version}all.deb ？
the extension set it to =aic8800-usb-dkms${latest_version}_all.deb。

With the aic8800_dkms_file_name fixed，

There was an error when I compile Armbian with AIC8800_TYPE="sdio"：

2024-06-22 09:02:20 (2.78 MB/s) - ‘/tmp/aic8800-firmware_3.0+git20240327.3561b08f-1_all.deb’ saved [1348600/1348600]
   
--> (356) INFO: Install aic8800 packages, will build kernel module in chroot [ radxa-aic8800 ]
--> (356) COMMAND: mkdir -pv /opt/WorkSpace/Armbian/.tmp/rootfs-1af8bc42-d4c5-4178-95d0-ba4eefe318f6/var/cache/apt /opt/WorkSpace/Armbian/.tmp/rootfs-1af8bc42-d4c5-4178-95d0-ba4eefe318f6/var/lib/apt/lists
--> (356) COMMAND: mount --bind /opt/WorkSpace/Armbian/cache/aptcache/noble-arm64 /opt/WorkSpace/Armbian/.tmp/rootfs-1af8bc42-d4c5-4178-95d0-ba4eefe318f6/var/cache/apt
--> (356) COMMAND: mount --bind /opt/WorkSpace/Armbian/cache/aptcache/lists/noble-arm64 /opt/WorkSpace/Armbian/.tmp/rootfs-1af8bc42-d4c5-4178-95d0-ba4eefe318f6/var/lib/apt/lists
--> (356) COMMAND: env -i DEBIAN_FRONTEND=noninteractive apt-get -y -qq -o Dpkg::Use-Pty=0 -o APT::Get::List-Cleanup=0 --no-install-recommends install /tmp/aic8800-sdio-dkms_3.0+git20240327.3561b08f-1_all.deb /tmp/aic8800-firmware_3.0+git20240327.3561b08f-1_all.deb
   debconf: delaying package configuration, since apt-utils is not installed
   Selecting previously unselected package aic8800-firmware.
   (Reading database ... 117374 files and directories currently installed.)
   Preparing to unpack .../aic8800-firmware_3.0+git20240327.3561b08f-1_all.deb ...
   Unpacking aic8800-firmware (3.0+git20240327.3561b08f-1) ...
   Selecting previously unselected package aic8800-sdio-dkms.
   Preparing to unpack .../aic8800-sdio-dkms_3.0+git20240327.3561b08f-1_all.deb ...
   Unpacking aic8800-sdio-dkms (3.0+git20240327.3561b08f-1) ...
   Setting up aic8800-firmware (3.0+git20240327.3561b08f-1) ...
   Setting up aic8800-sdio-dkms (3.0+git20240327.3561b08f-1) ...
   Loading new aic8800-sdio-3.0+git20240327.3561b08f-1 DKMS files...
   It is likely that 6.5.0-35-generic belongs to a chroot's host
   Building for 6.10.0-rc4-edge-rockchip-rk3588
   Building initial module for 6.10.0-rc4-edge-rockchip-rk3588
   Error! Bad return status for module build on kernel: 6.10.0-rc4-edge-rockchip-rk3588 (aarch64)
   Consult /var/lib/dkms/aic8800-sdio/3.0+git20240327.3561b08f-1/build/make.log for more information.
   dpkg: error processing package aic8800-sdio-dkms (--configure):
    installed aic8800-sdio-dkms package post-installation script subprocess returned error exit status 10
   Errors were encountered while processing:
    aic8800-sdio-dkms
   E: Sub-process /usr/bin/dpkg returned an error code (1)
   -->--> command failed with error code 100 after 116 seconds
--> (472) WARNING: stacktrace for failed command [ exit code 100:chroot /opt/WorkSpace/Armbian/.tmp/rootfs-1af8bc42-d4c5-4178-95d0-ba4eefe318f6 /usr/bin/env bash -e -o pipefail -c env -i DEBIAN_FRONTEND=noninteractive apt-get -y -qq -o Dpkg::Use-Pty=0 -o APT::Get::List-Cleanup=0 --no-install-recommends install /tmp/aic8800-sdio-dkms_3.0+git20240327.3561b08f-1_all.deb /tmp/aic8800-firmware_3.0+git20240327.3561b08f-1_all.deb

Hope to get some suggestions from you。

@amazingfate
Copy link
Contributor

I only checked usb package. I will check sdio later.

Dangku pushed a commit to BP 8543 I-SINOVOIP/armbian-build that referenced this pull request Jan 16, 2025
@RadxaYuntian @Dangku
aic8800: update firmware package URL (armbian#6768)
31b3e89 
Firmware package was incorrectly marked as machine dependent.
This was fixed in radxa-pkg/aic8800#12, causing the file name to change.

Signed-off-by: ZHANG Yuntian <yt@radxa.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@amazingfate amazingfate amazingfate approved these changes

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

Assignees
No one assigned
Labels
Framework Framework components size/small PR with less then 50 lines
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@RadxaYuntian @andyshrk @amazingfate @EvilOlaf
0