Skip to content

CAMX: Add Qualcomm Camera X (CamX) module for Purwa - #3169

Open
Ganesh Khose (gkhose-qipl) wants to merge 2 commits into
qualcomm-linux:masterfrom
gkhose-qipl:camx_fw
Open

CAMX: Add Qualcomm Camera X (CamX) module for Purwa#3169
Ganesh Khose (gkhose-qipl) wants to merge 2 commits into
qualcomm-linux:masterfrom
gkhose-qipl:camx_fw

Conversation

@gkhose-qipl

@gkhose-qipl Ganesh Khose (gkhose-qipl) commented Sep 16, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

CAMX: Add Qualcomm Camera X (CamX) module for Purwa

Depends on:- qualcomm-linux/meta-qcom-distro#483

@lumag

Dmitry Baryshkov (lumag) commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

The qcom-distro change needs to be landed first.

@ricardosalveti

Ricardo Salveti (ricardosalveti) commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Where is the distro change?

@gkhose-qipl

Ganesh Khose (gkhose-qipl) commented Sep 17, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Where is the distro change?

It's binary compatibility with hamoa, so distro changes are not needed because hamoa is enabled. for "incompatible licenses" will raise distro change

@gkhose-qipl

Ganesh Khose (gkhose-qipl) commented Sep 17, 2026

Copy link
Copy Markdown
Contributor Author

Where is the distro change?

It's binary compatibility with hamoa, so distro changes are not needed because hamoa is enabled. for "incompatible licenses" will raise distro change

qualcomm-linux/meta-qcom-distro#483

@github-actions

github-actions Bot commented Sep 17, 2026 •
edited
Loading

Copy link
Copy Markdown

Test run workflow

Test jobs for commit 7330689

qcom-distro
Pass: 12 | Fail: 0 | Total: 12 qcom-distro_linux-qcom-6.18
Pass: 233 | Fail: 1 | Total: 259 nodistro
Pass: 10 | Fail: 0 | Total: 10

@test-reporting-app

test-reporting-app Bot commented Sep 17, 2026 •
edited
Loading

Copy link
Copy Markdown

Test Results

   68 files  ±0    319 suites  ±0   3h 45m 53s ⏱️ - 2h 4m 26s
  149 tests  - 4    146 ✅ +2   1 💤 +1  2 ❌  - 7 
2 100 runs   - 3  2 073 ✅ +4  25 💤 +1  2 ❌  - 8 

For more details on these failures, see this check.

Results for commit 7330689. ± Comparison against base commit 7a8b9c7.

This pull request removes 4 tests. 
lava ‑ auto-login-action
lava ‑ lava-test-retry
lava ‑ lava-test-shell
lava ‑ minimal-boot

♻️ This comment has been updated with latest results.

ricardosalveti
Ricardo Salveti (ricardosalveti) reviewed Sep 17, 2026
View reviewed changes
Comment thread recipes-bsp/firmware/camxfirmware-hamoa_1.0.6.bb Outdated
# Create symlinks under x1p42100 to satisfy platform-specific
# lookup paths and avoid binary duplication for Purwa
install -d ${D}${FW_QCOM_BASE_PATH}/x1p42100
ln -sf ../${FW_QCOM_NAME}/CAMERA_ICP.mbn ${D}${FW_QCOM_BASE_PATH}/x1p42100/

@ricardosalveti Ricardo Salveti (ricardosalveti) Sep 17, 2026

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.

This is missing the compression handling that camxfirmware-lemans has (${@fw_compr_file_suffix(d)}).

ricardosalveti
Ricardo Salveti (ricardosalveti) reviewed Sep 17, 2026
View reviewed changes
Comment thread recipes-bsp/firmware/camxfirmware-hamoa_1.0.6.bb
SRC_URI[sha256sum] = "7201797be68d34ca826f6b2530590ab5f1bb3fb5c347a63259bee61a9b16036d"
PBT_BUILD_DATE = "260911"
PBT_BRANCH = "master"
SRC_URI = "https://qartifactory-edge.qualcomm.com/artifactory/qsc_releases/software/chip/component/camx.qclinux.0.0/${PBT_BUILD_DATE}/prebuilt_yocto_${PBT_BRANCH}/${BPN}_${PV}_armv8-2a.tar.gz"

@ricardosalveti Ricardo Salveti (ricardosalveti) Sep 17, 2026

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.

Your commit is saying 'add camera firmware' while updating it for hamoa as well, reflect that in the commit message.

@qcomlnxci

qcomlnxci commented Sep 18, 2026

Copy link
Copy Markdown

Test Coral run workflow

Test jobs for commit eb56c93

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

@gkhose-qipl
camxfirmware-hamoa: Update camera firmware for Hamoa and Purwa
1ed00b8 
Update the Qualcomm camera firmware recipe to support camera enablement
on Purwa while updating the existing Hamoa firmware configuration.

Change SRC_URI to use the prebuilt_yocto_master artifact path, which is
the current location of the published tarball.

Create the required firmware symlink under the Purwa-specific x1p42100
path to avoid duplicating the firmware binary. Resolve the firmware
compression suffix when creating the symlink so it supports both
uncompressed and compressed firmware files, including zstd compression.

Signed-off-by: Ganesh Khose <gkhose@qti.qualcomm.com>
@gkhose-qipl
packagegroup-purwa-iot-evk:Include camxfirmware in firmware packagegroup
7330689 
Add camxfirmware-purwa, firmware required to support Qualcomm Camera X
(CamX) on the board.

Signed-off-by: Ganesh Khose <gkhose@qti.qualcomm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vkraleti Viswanath Kraleti (vkraleti) Awaiting requested review from vkraleti vkraleti is a code owner
@ndechesne Nicolas Dechesne (ndechesne) Awaiting requested review from ndechesne ndechesne is a code owner
@anujm1 Anuj Mittal (anujm1) Awaiting requested review from anujm1 anujm1 is a code owner
@ricardosalveti Ricardo Salveti (ricardosalveti) Awaiting requested review from ricardosalveti ricardosalveti is a code owner
@koenkooi Koen Kooi (koenkooi) Awaiting requested review from koenkooi koenkooi is a code owner
@lumag Dmitry Baryshkov (lumag) Awaiting requested review from lumag lumag is a code owner

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

backport wrynose

Projects

None yet

Milestone

qli-2.1 pull-request freeze

Development

Successfully merging this pull request may close these issues.

6 participants

@gkhose-qipl @lumag @ricardosalveti @qcomlnxci @koenkooi @vkraleti