Skip to content

firmware-qcom: make split packages installable#2557

Open
lumag wants to merge 1 commit into
qualcomm-linux:masterfrom
lumag:fix-fw
Open

firmware-qcom: make split packages installable#2557
lumag wants to merge 1 commit into
qualcomm-linux:masterfrom
lumag:fix-fw

Conversation

@lumag

@lumag lumag commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

Since the commit 77e6f45 ("firmware-qcom-nhlos: make sure that meta firmware package is empty") the main package in the split firmware doesn't have any files and thus is skipped by OE. All split firmware packages depend on PN, which can't be fond. Drop the manual dependency, making all split packages installable again.

Closes #2504

vkraleti
vkraleti previously approved these changes Jun 20, 2026
Since the commit 77e6f45 ("firmware-qcom-nhlos: make sure that meta
firmware package is empty") the main package in the split firmware
doesn't have any files and thus is skipped by OE. All split firmware
packages depend on PN, which can't be fond. Drop the manual dependency,
making all split packages installable again.

Closes qualcomm-linux#2504

Fixes: 77e6f45 ("firmware-qcom-nhlos: make sure that meta firmware package is empty")
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
@github-actions

Copy link
Copy Markdown

Test run workflow

Test jobs for commit 39f4e4b

nodistro
Pass: 9 | Fail: 0 | Total: 9
qcom-distro_linux-qcom-6.18
Pass: 7 | Fail: 1 | Total: 8
qcom-distro
Pass: 9 | Fail: 1 | Total: 10

@test-reporting-app

test-reporting-app Bot commented Jun 21, 2026

Copy link
Copy Markdown

Test Results

  105 files  ± 0    634 suites  ±0   5h 43m 17s ⏱️ - 1h 46m 47s
  129 tests  -  2    107 ✅  -  5   0 💤 ±0  22 ❌ +3 
6 021 runs  +13  5 904 ✅ +12  86 💤 +7  31 ❌  - 6 

For more details on these failures, see this check.

Results for commit 39f4e4b. ± Comparison against base commit f584593.

This pull request removes 5 and adds 3 tests. Note that renamed tests count towards both.
lava ‑ 6_OpenCV
lava ‑ download-retry
lava ‑ downloads
lava ‑ lava-test-retry
lava ‑ lava-test-shell
2_Interrupts ‑ irq
lava ‑ deploy-flasher
lava ‑ deploy-flasher-retry

♻️ This comment has been updated with latest results.

@ricardosalveti

Copy link
Copy Markdown
Contributor

Is the license now part of the individual packages?

@koenkooi koenkooi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nit: fondfound

@lumag

lumag commented Jun 22, 2026

Copy link
Copy Markdown
Contributor Author

Is the license now part of the individual packages?

No, but we have a plenty of packages for which there is no binding license. For example, for the RBn devices Thundercomm doesn't define any licence for the files downloaded via the SDK manager.

@github-actions

Copy link
Copy Markdown

Test run workflow

Test jobs for commit 39f4e4b

nodistro
Pass: 9 | Fail: 0 | Total: 9
qcom-distro_linux-qcom-6.18
Pass: 212 | Fail: 0 | Total: 250
qcom-distro
Pass: 9 | Fail: 1 | Total: 10

@github-actions

Copy link
Copy Markdown

Test run workflow

Test jobs for commit 39f4e4b

nodistro
Pass: 9 | Fail: 0 | Total: 9
qcom-distro_linux-qcom-6.18
Pass: 212 | Fail: 0 | Total: 250
qcom-distro
Pass: 230 | Fail: 6 | Total: 284

@ricardosalveti

Copy link
Copy Markdown
Contributor

Is the license now part of the individual packages?

No, but we have a plenty of packages for which there is no binding license. For example, for the RBn devices Thundercomm doesn't define any licence for the files downloaded via the SDK manager.

So the license will basically stay as CLOSED, right? Which is not really valid, but not sure we can do much here if we want to keep these recipes around.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

sm8550-hdk firmware files generate empty package which blocks installation

4 participants