Skip to content

Drop SoC version suffixes from FIT DTB compatible strings#2159

Open
vkraleti wants to merge 1 commit into
qualcomm-linux:masterfrom
vkraleti:fit-compat-cleanup
Open

Drop SoC version suffixes from FIT DTB compatible strings#2159
vkraleti wants to merge 1 commit into
qualcomm-linux:masterfrom
vkraleti:fit-compat-cleanup

Conversation

@vkraleti
Copy link
Copy Markdown
Contributor

With the new DTB compatible string format, SoC version and board revision are expressed as explicit suffixes. When these suffixes are not present, FIT matching falls back to the best available compatible string. As a result, explicitly listing SoC-versioned compatible entries is optional when there is no difference in DTB or DTBO selection.

To simplify compatible string listings, remove redundant SoC version entries from FIT_DTB_COMPATIBLE and rely on base compatibles for matching.

@ricardosalveti
Copy link
Copy Markdown
Contributor

Not everything is being validated at #2149 yet, but should we try to get that merged first before proposing additional fit-dtb changes?

@vkraleti
Copy link
Copy Markdown
Contributor Author

Sure, this PR can wait.

@lumag
Copy link
Copy Markdown
Contributor

lumag commented May 12, 2026

The #2149 is now merged. Please rebase and merge.

With the new DTB compatible string format, SoC version and board
revision are expressed as explicit suffixes. When these suffixes are
not present, FIT matching falls back to the best available compatible
string. As a result, explicitly listing SoC-versioned compatible entries
is optional when there is no difference in DTB or DTBO selection.

To simplify compatible string listings, remove redundant SoC version
entries from FIT_DTB_COMPATIBLE and rely on base compatibles for
matching.

Signed-off-by: Viswanath Kraleti <viswanath.kraleti@oss.qualcomm.com>
@vkraleti vkraleti force-pushed the fit-compat-cleanup branch from 9e854e1 to a93eab4 Compare May 13, 2026 05:36
@github-actions
Copy link
Copy Markdown

Test run workflow

Test jobs for commit a93eab4

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

@test-reporting-app
Copy link
Copy Markdown

test-reporting-app Bot commented May 13, 2026

Test Results

   99 files  ±0    528 suites  ±0   5h 1m 22s ⏱️ - 4m 33s
   93 tests  - 3     91 ✅ +11   1 💤 ± 0  1 ❌  - 14 
4 995 runs  +7  4 967 ✅ +52  27 💤  - 17  1 ❌  - 28 

For more details on these failures, see this check.

Results for commit a93eab4. ± Comparison against base commit a27cc18.

This pull request removes 3 tests.
3_AudioRecord_Config07 ‑ AudioRecord_Config09
lava ‑ auto-login-action
lava ‑ minimal-boot

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Copy Markdown

Test run workflow

Test jobs for commit a93eab4

qcom-distro
Pass: 199 | Fail: 2 | Total: 214
qcom-distro_linux-qcom-6.18
Pass: 199 | Fail: 0 | Total: 208
nodistro
Pass: 9 | Fail: 0 | Total: 9

@github-actions
Copy link
Copy Markdown

Test run workflow

Test jobs for commit a93eab4

qcom-distro
Pass: 210 | Fail: 7 | Total: 231
qcom-distro_linux-qcom-6.18
Pass: 200 | Fail: 1 | Total: 213
nodistro
Pass: 9 | Fail: 0 | Total: 9

@github-actions
Copy link
Copy Markdown

Test run workflow

Test jobs for commit a93eab4

qcom-distro
Pass: 210 | Fail: 7 | Total: 231
qcom-distro_linux-qcom-6.18
Pass: 200 | Fail: 0 | Total: 213
nodistro
Pass: 9 | Fail: 0 | Total: 9

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.

3 participants