Skip to content

Conversation

@ssadhula
Copy link
Contributor

In qcom-multimedia-proprietary-image, qcom-adreno is present, but the EGL_adreno.json is not prioritized. So, rename EGL_adreno.json to prioritize adreno over mesa.

@lumag
Copy link
Contributor

lumag commented Dec 22, 2025

Which configuration do we end up with otherwise? Software driver inside mesa?

@github-actions
Copy link

Test run workflow

Test jobs for commit 85f6904

@test-reporting-app
Copy link

test-reporting-app bot commented Dec 22, 2025

Test Results

 14 files  + 2   28 suites  +4   35m 16s ⏱️ + 4m 44s
 50 tests + 5   50 ✅ + 5  0 💤 ±0  0 ❌ ±0 
304 runs  +40  304 ✅ +40  0 💤 ±0  0 ❌ ±0 

Results for commit 5554128. ± Comparison against base commit 9fa944d.

♻️ This comment has been updated with latest results.

@ssadhula
Copy link
Contributor Author

Which configuration do we end up with otherwise? Software driver inside mesa?

Yes, it will use 50_mesa.json

@lumag
Copy link
Contributor

lumag commented Dec 23, 2025

Which configuration do we end up with otherwise? Software driver inside mesa?

Yes, it will use 50_mesa.json

Please update the commit message explaining that (and mentioning software implementation). There is no need to mention images or anything else. Otherwise it's not clear why we should prioritize Adreno over any other driver.

50_mesa.json is given preference over EGL_adreno.json, due to
json being loaded in strcmp() order.

Rename EGL_adreno.json to 10_EGL_adreno.json, to prioritize adreno
over 50_mesa.json.

Signed-off-by: Sai Sumanth Sadhula <ssadhula@qti.qualcomm.com>
@github-actions
Copy link

Test run workflow

Test jobs for commit 5554128

@github-actions
Copy link

Test run workflow

Test jobs for commit 5554128

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.

2 participants