[Chandan Jha (cjha01)]

Downstream camera DTB is not included in the final image
when building linux-qcom, linux-qcom-next, linux-qcom-rt,
or linux-qcom-next-rt kernels for the shikra EVK boards.
This prevents the downstream camera stack from being
enabled at boot, even though the kernel supports it.
 
Include the downstream camera DTBO so it is packaged into
the final image and downstream camera functionality is
available for the shikra EVK boards.
 
Add multi-DTB support for camx overlay to enable downstream
camera for shikra cqs, cqm and iqs evk boards.

[Chandan Jha (cjha01)]

qli-2.1 pull-request freeze

[Chandan Jha (cjha01)]

qli-2.1 pull-request freeze

[Chandan Jha (cjha01)]

backport wrynose

[github-actions]

Test run workflow

Test jobs for commit 8222193

qcom-distro_linux-qcom-6.18
Pass: 234 | Fail: 1 | Total: 253

nodistro
Pass: 10 | Fail: 0 | Total: 10

qcom-distro
Pass: 337 | Fail: 2 | Total: 367

[test-reporting-app]

Test Results

  117 files    717 suites   7h 17m 8s ⏱️
  195 tests   191 ✅  2 💤 2 ❌
4 613 runs  4 564 ✅ 46 💤 3 ❌

For more details on these failures, see this check.

Results for commit 8222193.

♻️ This comment has been updated with latest results.

[Chandan Jha (cjha01)]

Ricardo Salveti (@ricardosalveti), Dmitry Baryshkov (@lumag), Shikra camera enablement is a key deliverable for the QLI 2.1 milestone, and this change is required to enable camera support.

Since the Shikra DTSO changes have not yet been merged into the qcom-next branch, we have added a conditional check so that the code is compiled only for the QLI 2.x kernel and is excluded from the mainline kernel build.

We hope this approach is acceptable and helps us meet the QLI 2.1 timeline while maintaining mainline compatibility. Could you please review the PR and share your feedback?

[qcomlnxci]

Test Coral run workflow

Test jobs for commit 3907012

• qcomdistro: multimedia image
  Pass: 9 | Fail: 0 | Total: 9
• qcomdistro: multimedia image-prop
  Pass: 41 | Fail: 0 | Total: 41

[Dmitry Baryshkov (lumag)]

You haven't read CONTRIBUTING.md, have you?

[Chandan Jha (cjha01)]

You haven't read CONTRIBUTING.md, have you?
Yes, I understand your concern. Due to project timelines, we were focused on the QLI 2.1 kernel only. I will make sure that future changes are first submitted for the QLI mainline kernel and then extended to QLI 2.x kernel.

[qcomlnxci]

Test Coral run workflow

Test jobs for commit 0ebbb52

• qcomdistro: multimedia image-prop
  Pass: 25 | Fail: 4 | Total: 29
• qcomdistro: multimedia image
  Pass: 7 | Fail: 2 | Total: 9

[Ricardo Salveti (ricardosalveti)]

Drop this comment, we know that LINUX_QCOM_KERNEL_DEVICETREE is for dtbs which are not yet available upstream.

[Chandan Jha (cjha01)]

Dropped this comment. Done.

[Ricardo Salveti (ricardosalveti)]

See https://github.com/qualcomm-linux/meta-qcom/blob/master/CONTRIBUTING.md#commit-messages for the commit message pattern we use here.

[qcomlnxci]

Test Coral run workflow

Test jobs for commit 718a898

• qcomdistro: multimedia image-prop
  Pass: 26 | Fail: 2 | Others: 3 | Total: 31
• qcomdistro: multimedia image
  Pass: 4 | Fail: 0 | Total: 4

[Ricardo Salveti (ricardosalveti)]

Subject and body still say linux-qcom only, but this now covers linux-qcom and linux-qcom-next. The last paragraph ("Add multi-DTB support…") belongs to the second commit. Also use lowercase enable, as lumag suggested.

Please name shikra in the subject, e.g. fit-dtb-compatible: add shikra camx overlay combos. The body repeats the first commit's text; say what the entries do instead: base compatible plus -camx, with the imx577 overlay dropped, as hamoa does.

[qcomlnxci]

Test Coral run workflow

Test jobs for commit 3ecc1a5

• qcomdistro: multimedia image-prop
  Pass: 29 | Fail: 0 | Others: 3 | Total: 32
• qcomdistro: multimedia image
  Pass: 4 | Fail: 0 | Total: 4

[Chandan Jha (cjha01)]

Subject and body still say linux-qcom only, but this now covers linux-qcom and linux-qcom-next. The last paragraph ("Add multi-DTB support…") belongs to the second commit. Also use lowercase enable, as lumag suggested.

Please name shikra in the subject, e.g. fit-dtb-compatible: add shikra camx overlay combos. The body repeats the first commit's text; say what the entries do instead: base compatible plus -camx, with the imx577 overlay dropped, as hamoa does.

Updated the commit text.

[Chandan Jha (cjha01)]

Subject and body still say linux-qcom only, but this now covers linux-qcom and linux-qcom-next. The last paragraph ("Add multi-DTB support…") belongs to the second commit. Also use lowercase enable, as lumag suggested.

Please name shikra in the subject, e.g. fit-dtb-compatible: add shikra camx overlay combos. The body repeats the first commit's text; say what the entries do instead: base compatible plus -camx, with the imx577 overlay dropped, as hamoa does.

I have updated the commit message based on the suggestions. Could you please review it once.

[Dmitry Baryshkov (lumag)]

Please fix your editor settings. Line length is too short.

[Chandan Jha (cjha01)]

Please fix your editor settings. Line length is too short.

Updated. Please review.

[qcomlnxci]

Test Coral run workflow

Test jobs for commit 496b774

• qcomdistro: multimedia image-prop
  Pass: 30 | Fail: 2 | Total: 32
• qcomdistro: multimedia image
  Pass: 4 | Fail: 0 | Total: 4

[qcomlnxci]

Test Coral run workflow

Test jobs for commit 94df16a

• qcomdistro: multimedia image-prop
  Pass: 32 | Fail: 0 | Total: 32
• qcomdistro: multimedia image
  Pass: 4 | Fail: 0 | Total: 4

[qcomlnxci]

Test Coral run workflow

Test jobs for commit 8222193

• qcomdistro: multimedia image
  Pass: 6 | Fail: 0 | Total: 6
• qcomdistro: multimedia image-prop
  Pass: 32 | Fail: 0 | Total: 32

[Chandan Jha (cjha01)]

HI Dmitry Baryshkov (@lumag) , @ricardosalvet, Please review the change and approve it if it meets your expectations.

[Ricardo Salveti (ricardosalveti)]

Known issues.

[quic-yocto-ci]

Successfully created backport PR for wrynose:

• [Backport wrynose] Enable shikra camx DTB compilation for linux-qcom kernels  #3292