Skip to content

Add and enable IPA node#737

Open
rpavanqcom wants to merge 1 commit into
qualcomm-linux:qcom-6.18.yfrom
rpavanqcom:ipa_fw
Open

Add and enable IPA node#737
rpavanqcom wants to merge 1 commit into
qualcomm-linux:qcom-6.18.yfrom
rpavanqcom:ipa_fw

Conversation

@rpavanqcom

Copy link
Copy Markdown

Enable IPA and ensure ipa apps loads the gsi firmware becaus modem doesn't support IPA FW loading.

CRs-Fixed: 4576442

@rpavanqcom rpavanqcom changed the base branch from main to qcom-6.18.y June 18, 2026 08:17
@rpavanqcom rpavanqcom requested review from a team, quic-tingweiz, trsoni and yijiyang June 18, 2026 08:17
@qcomlnxci

Copy link
Copy Markdown

Test Matrix

Test Case lemans-evk monaco-evk qcs615-ride qcs6490-rb3gen2 qcs8300-ride qcs9100-ride-r3 x1e80100-crd
BT_FW_KMD_Service ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
BT_ON_OFF ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
BT_SCAN ✅ Pass ❌ Fail ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
CPUFreq_Validation ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
CPU_affinity ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
DSP_AudioPD ✅ Pass ✅ Pass ⚠️ skip ✅ Pass ✅ Pass ⚠️ skip ◻️
Ethernet ⚠️ skip ✅ Pass ⚠️ skip ⚠️ skip ⚠️ skip ⚠️ skip ◻️
Freq_Scaling ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
GIC ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
IPA ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
Interrupts ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
OpenCV ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
PCIe ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
Probe_Failure_Check ❌ Fail ❌ Fail ❌ Fail ❌ Fail ❌ Fail ❌ Fail ◻️
RMNET ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
UFS_Validation ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
USBHost ❌ Fail ❌ Fail ❌ Fail ❌ Fail ❌ Fail ❌ Fail ◻️
WiFi_Firmware_Driver ❌ Fail ❌ Fail ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
WiFi_OnOff ✅ Pass ❌ Fail ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
adsp_remoteproc ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ❌ Fail ◻️
cdsp_remoteproc ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ❌ Fail ◻️
gpdsp_remoteproc ✅ Pass ✅ Pass ⚠️ skip ⚠️ skip ✅ Pass ❌ Fail ◻️
hotplug ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
irq ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
kaslr ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
pinctrl ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
qcom_hwrng ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
remoteproc ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ❌ Fail ◻️
rngtest ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
shmbridge ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
smmu ❌ Fail ✅ Pass ❌ Fail ✅ Pass ✅ Pass ❌ Fail ◻️
watchdog ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
wpss_remoteproc ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️

@qcomlnxci

Copy link
Copy Markdown

Test Matrix

Test Case lemans-evk monaco-evk qcs615-ride qcs6490-rb3gen2 qcs8300-ride qcs9100-ride-r3 x1e80100-crd
BT_FW_KMD_Service ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
BT_ON_OFF ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
BT_SCAN ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
CPUFreq_Validation ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
CPU_affinity ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
DSP_AudioPD ✅ Pass ✅ Pass ⚠️ skip ✅ Pass ✅ Pass ⚠️ skip ◻️
Ethernet ⚠️ skip ✅ Pass ⚠️ skip ⚠️ skip ⚠️ skip ⚠️ skip ◻️
Freq_Scaling ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
GIC ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
IPA ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
Interrupts ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
OpenCV ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
PCIe ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
Probe_Failure_Check ❌ Fail ❌ Fail ✅ Pass ❌ Fail ❌ Fail ❌ Fail ◻️
RMNET ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
UFS_Validation ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
USBHost ❌ Fail ❌ Fail ❌ Fail ❌ Fail ❌ Fail ❌ Fail ◻️
WiFi_Firmware_Driver ❌ Fail ❌ Fail ❌ Fail ✅ Pass ✅ Pass ✅ Pass ◻️
WiFi_OnOff ✅ Pass ❌ Fail ⚠️ skip ✅ Pass ✅ Pass ✅ Pass ◻️
adsp_remoteproc ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ❌ Fail ◻️
cdsp_remoteproc ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ❌ Fail ◻️
gpdsp_remoteproc ✅ Pass ✅ Pass ⚠️ skip ⚠️ skip ✅ Pass ❌ Fail ◻️
hotplug ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
irq ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
kaslr ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
pinctrl ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
qcom_hwrng ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
remoteproc ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ❌ Fail ◻️
rngtest ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
shmbridge ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
smmu ❌ Fail ✅ Pass ❌ Fail ✅ Pass ✅ Pass ❌ Fail ◻️
watchdog ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
wpss_remoteproc ✅ Pass ✅ Pass ⚠️ skip ✅ Pass ✅ Pass ✅ Pass ◻️

@qcomlnxci

Copy link
Copy Markdown

Test Matrix

Test Case lemans-evk monaco-evk qcs615-ride qcs6490-rb3gen2 qcs8300-ride qcs9100-ride-r3 x1e80100-crd
BT_FW_KMD_Service ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
BT_ON_OFF ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
BT_SCAN ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
CPUFreq_Validation ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
CPU_affinity ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
DSP_AudioPD ✅ Pass ✅ Pass ⚠️ skip ✅ Pass ✅ Pass ⚠️ skip ◻️
Ethernet ⚠️ skip ✅ Pass ⚠️ skip ⚠️ skip ⚠️ skip ⚠️ skip ◻️
Freq_Scaling ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
GIC ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
IPA ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
Interrupts ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
OpenCV ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
PCIe ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
Probe_Failure_Check ❌ Fail ❌ Fail ❌ Fail ❌ Fail ❌ Fail ❌ Fail ◻️
RMNET ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
UFS_Validation ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
USBHost ❌ Fail ❌ Fail ❌ Fail ❌ Fail ❌ Fail ❌ Fail ◻️
WiFi_Firmware_Driver ❌ Fail ❌ Fail ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
WiFi_OnOff ✅ Pass ❌ Fail ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
adsp_remoteproc ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ❌ Fail ◻️
cdsp_remoteproc ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ❌ Fail ◻️
gpdsp_remoteproc ✅ Pass ✅ Pass ⚠️ skip ⚠️ skip ✅ Pass ❌ Fail ◻️
hotplug ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
irq ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
kaslr ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
pinctrl ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
qcom_hwrng ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
remoteproc ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ❌ Fail ◻️
rngtest ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
shmbridge ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
smmu ❌ Fail ✅ Pass ❌ Fail ✅ Pass ✅ Pass ❌ Fail ◻️
watchdog ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
wpss_remoteproc ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️

@qlijarvis

Copy link
Copy Markdown

PR #737 — validate-patch

PR: #737

Verdict Issues Detailed Report
⚠️ 4 Full report

Final Summary

  1. Lore link present: Yes — Link provided in commit message with proper format
  2. Lore link matches PR commits: N/A — Cannot verify due to network restrictions (lore.kernel.org inaccessible)
  3. Upstream patch status: In review — FROMLIST prefix indicates patch posted to mailing list but not yet merged
  4. PR present in qcom-next: Not checked — Cannot verify without repository access

Note: This validation is limited by network restrictions. The patch structure, metadata, and SoB chain follow kernel conventions. The FROMLIST prefix correctly indicates this is a patch under upstream review. Full validation would require comparing the diff against the actual upstream submission on lore.kernel.org.

Verdict: ⚠️ — click to expand

🔍 Patch Validation

PR: #737 - arm64: dts: qcom: qcm6490-idp: add and enable IPA node
Upstream commit: https://lore.kernel.org/all/2qlpabzpyvjwd2siwkpcdcjq7geyzj5egu732dg775asn2wtbq@tcbj6r5ti2o5/T/#t
Verdict: ⚠️ PARTIAL

Commit Message

Check Status Note
Subject matches upstream ⚠️ Cannot verify - lore.kernel.org inaccessible due to network restrictions
Body preserves rationale Clear explanation: "Enable IPA and ensure ipa apps loads the gsi firmware because modem doesn't support IPA FW loading"
Fixes tag present/correct N/A Not a fix; new feature enablement
Authorship preserved ⚠️ Original author: Sarat Addepalli; Co-developer: Pavan Kumar M - proper SoB chain present
Backport note (if applicable) N/A FROMLIST prefix indicates patch is from mailing list, not yet merged upstream

Diff

File Status Notes
arch/arm64/boot/dts/qcom/qcm6490-idp.dts ⚠️ Cannot verify against upstream - network restricted; diff structure appears valid

Issues

  • Lore link inaccessible: Network restrictions prevent fetching upstream patch for comparison. The message-ID format appears valid but cannot be verified.
  • FROMLIST prefix: Indicates patch is posted to mailing list but not yet accepted upstream. This is appropriate for patches under review.
  • Authorship chain: Proper dual SoB chain (original author → committer) is present and follows kernel conventions.
  • Commit message grammar: Minor issue - "ipa apps loads" should be "IPA apps load" (subject-verb agreement).

Verdict

Cannot fully validate due to network restrictions preventing upstream comparison; commit structure and metadata appear correct, but diff faithfulness to upstream cannot be verified.

Final Summary

  1. Lore link present: Yes — Link provided in commit message with proper format
  2. Lore link matches PR commits: N/A — Cannot verify due to network restrictions (lore.kernel.org inaccessible)
  3. Upstream patch status: In review — FROMLIST prefix indicates patch posted to mailing list but not yet merged
  4. PR present in qcom-next: Not checked — Cannot verify without repository access

Note: This validation is limited by network restrictions. The patch structure, metadata, and SoB chain follow kernel conventions. The FROMLIST prefix correctly indicates this is a patch under upstream review. Full validation would require comparing the diff against the actual upstream submission on lore.kernel.org.

@qlijarvis

Copy link
Copy Markdown

PR #737 — checker-log-analyzer

PR: #737
Checker run: https://github.com/qualcomm-linux/kernel-config/actions/runs/27816637526

Checker Result Summary
Checker Result Summary
checkpatch No style issues detected
dt-binding-check No binding schema violations
dtb-check Device tree validation passed
sparse-check ⏭️ Skipped (no C code changes)
check-uapi-headers ⏭️ Skipped (no UAPI header changes)
check-patch-compliance Invalid lore.kernel.org link
tag-check N/A Not applicable for qcom-6.18.y
qcom-next-check FROMLIST: prefix present

Detailed report: Full report

Checker analysis — click to expand

🤖 CI Checker Analysis (checker-log-analyzer)

PR: #737 - FROMLIST: arm64: dts: qcom: qcm6490-idp: add and enable IPA node
Source: https://github.com/qualcomm-linux/kernel-config/actions/runs/27816637526

Checker Result Summary
checkpatch No style issues detected
dt-binding-check No binding schema violations
dtb-check Device tree validation passed
sparse-check ⏭️ Skipped (no C code changes)
check-uapi-headers ⏭️ Skipped (no UAPI header changes)
check-patch-compliance Invalid lore.kernel.org link
tag-check N/A Not applicable for qcom-6.18.y
qcom-next-check FROMLIST: prefix present

❌ check-patch-compliance

Root cause: The Link: tag in the commit message contains a malformed lore.kernel.org URL that b4 cannot fetch.

Failure details:

2026-06-19T09:27:08.1258236Z Something seems wrong with the provided link. Please verify it
2026-06-19T09:27:08.1258826Z Try below command to run locally-
2026-06-19T09:27:08.1259913Z b4 am --single-message -C -l -3 https://lore.kernel.org/all/2qlpabzpyvjwd2siwkpcdcjq7geyzj5egu732dg775asn2wtbq@tcbj6r5ti2o5/T/#t

The link in the commit message is:

Link: https://lore.kernel.org/all/2qlpabzpyvjwd2siwkpcdcjq7geyzj5egu732dg775asn2wtbq@tcbj6r5ti2o5/T/#t

The /T/#t suffix is invalid for lore.kernel.org message-ID URLs. This appears to be a threading view URL fragment that was incorrectly included in the Link: tag.

Fix:

  1. Remove the /T/#t suffix from the Link: tag. The correct format should be:

    Link: https://lore.kernel.org/all/2qlpabzpyvjwd2siwkpcdcjq7geyzj5egu732dg775asn2wtbq@tcbj6r5ti2o5/
    
  2. Alternatively, verify the actual message-ID from the lore.kernel.org thread and use the canonical format:

    Link: https://lore.kernel.org/all/<message-id>/
    
  3. Update the commit message with git commit --amend and force-push to the PR branch.

Reproduce locally:

# Test the link with b4
b4 am --single-message -C -l -3 https://lore.kernel.org/all/2qlpabzpyvjwd2siwkpcdcjq7geyzj5egu732dg775asn2wtbq@tcbj6r5ti2o5/T/#t

# This will fail. Remove /T/#t and try:
b4 am --single-message -C -l -3 https://lore.kernel.org/all/2qlpabzpyvjwd2siwkpcdcjq7geyzj5egu732dg775asn2wtbq@tcbj6r5ti2o5/

Verdict

1 blocker to fix: The lore.kernel.org link in the commit message must be corrected by removing the /T/#t suffix. Once the link is fixed and the PR is updated, all checkers should pass.

Enable IPA and ensure ipa apps loads the gsi firmware because
modem doesn't support IPA FW loading.

Link: https://lore.kernel.org/all/20260618124700.2001191-1-sarat.addepalli@oss.qualcomm.com/
Signed-off-by: Sarat Addepalli <sarat.addepalli@oss.qualcomm.com>
Signed-off-by: Pavan Kumar M <rpavan@qti.qualcomm.com>
@qlijarvis

Copy link
Copy Markdown

PR #737 — validate-patch

PR: #737

Verdict Issues Detailed Report
⚠️ 9 Full report

Final Summary

  1. Lore link present: Yes — valid lore.kernel.org message-ID link included in commit message

  2. Lore link matches PR commits: Cannot verify — network access restricted; manual comparison required against https://lore.kernel.org/all/20260618124700.2001191-1-sarat.addepalli@oss.qualcomm.com/

  3. Upstream patch status: In review — FROMLIST prefix indicates patch posted to mailing list on 2026-06-18 (1 day ago); too recent to be merged upstream

  4. PR present in qcom-next: Not checked — validation scope limited to commit message and authorship hygiene per constraints

Verdict: ⚠️ — click to expand

🔍 Patch Validation

PR: #737 - arm64: dts: qcom: qcm6490-idp: add and enable IPA node
Upstream commit: https://lore.kernel.org/all/20260618124700.2001191-1-sarat.addepalli@oss.qualcomm.com/
Verdict: ⚠️ PARTIAL

Commit Message

Check Status Note
Subject matches upstream ⚠️ FROMLIST prefix added; core subject preserved
Body preserves rationale ⚠️ Grammar issue: "ipa apps loads" should be "IPA apps load"
Fixes tag present/correct N/A Not a fix; new feature enablement
Authorship preserved ⚠️ Original author (Sarat Addepalli) in SoB chain, but From: changed to Pavan Kumar M
Backport note (if applicable) N/A FROMLIST indicates pending upstream submission

Diff

File Status Notes
arch/arm64/boot/dts/qcom/qcm6490-idp.dts Clean device tree addition; alphabetically ordered after &gpu, before &lpass_rx_macro

Issues

Authorship Chain:

  • From: field shows Pavan Kumar M <rpavan@qti.qualcomm.com> (committer)
  • First SoB: Sarat Addepalli <sarat.addepalli@oss.qualcomm.com> (original author per lore link)
  • Second SoB: Pavan Kumar M <rpavan@qti.qualcomm.com> (committer/integrator)

This pattern suggests Pavan is integrating Sarat's upstream patch. However, kernel convention for FROMLIST patches typically preserves the original author in the From: field. The current structure implies Pavan authored the patch, which conflicts with the lore link showing Sarat as the original author.

Commit Message Grammar:

  • "ipa apps loads" → should be "IPA apps load" (subject-verb agreement)
  • "ipa" → should be capitalized as "IPA" for consistency

Technical Content:
The diff adds IPA node configuration with:

  • qcom,gsi-loader = "self" - indicates apps processor loads GSI firmware
  • memory-region = <&ipa_fw_mem> - references firmware memory region
  • firmware-name = "qcom/qcm6490/ipa_fws.mbn" - specifies firmware binary
  • status = "okay" - enables the node

This is a standard device tree enablement pattern for Qualcomm IPA (IP Accelerator) subsystem.

Verdict

Recommended action: Request minor corrections before merge:

  1. Authorship: If this is a faithful backport of Sarat's upstream patch, preserve original authorship by setting From: Sarat Addepalli <sarat.addepalli@oss.qualcomm.com> and keeping Pavan's SoB as integrator.

  2. Grammar: Fix commit message: "Enable IPA and ensure IPA apps load the GSI firmware because modem doesn't support IPA FW loading."

  3. Verification needed: Cannot verify diff faithfulness to upstream due to network restrictions. Manual comparison with lore.kernel.org/all/20260618124700.2001191-1-sarat.addepalli@oss.qualcomm.com/ is required to confirm the diff matches exactly.

Final Summary

  1. Lore link present: Yes — valid lore.kernel.org message-ID link included in commit message

  2. Lore link matches PR commits: Cannot verify — network access restricted; manual comparison required against https://lore.kernel.org/all/20260618124700.2001191-1-sarat.addepalli@oss.qualcomm.com/

  3. Upstream patch status: In review — FROMLIST prefix indicates patch posted to mailing list on 2026-06-18 (1 day ago); too recent to be merged upstream

  4. PR present in qcom-next: Not checked — validation scope limited to commit message and authorship hygiene per constraints

@qlijarvis

Copy link
Copy Markdown

PR #737 — checker-log-analyzer

PR: #737
Checker run: https://github.com/qualcomm-linux/kernel-config/actions/runs/27818044102

Checker Result Summary
Checker Result Summary
checkpatch No style issues detected
dt-binding-check ⏭️ Skipped (no binding changes)
dtb-check Test passed
sparse-check ⏭️ Skipped (no C code changes)
check-uapi-headers ⏭️ Skipped (no UAPI changes)
check-patch-compliance Author mismatch detected
tag-check FROMLIST: prefix present
qcom-next-check FROMLIST: commit compliant

Detailed report: Full report

Checker analysis — click to expand

🤖 CI Checker Analysis (checker-log-analyzer)

PR: #737 - FROMLIST: arm64: dts: qcom: qcm6490-idp: add and enable IPA node
Source: https://github.com/qualcomm-linux/kernel-config/actions/runs/27818044102

Checker Result Summary
checkpatch No style issues detected
dt-binding-check ⏭️ Skipped (no binding changes)
dtb-check Test passed
sparse-check ⏭️ Skipped (no C code changes)
check-uapi-headers ⏭️ Skipped (no UAPI changes)
check-patch-compliance Author mismatch detected
tag-check FROMLIST: prefix present
qcom-next-check FROMLIST: commit compliant

❌ check-patch-compliance

Root cause: The commit author (From: field) does not match the original patch author in the Signed-off-by: chain.

Failure details:

Checking commit: FROMLIST: arm64: dts: qcom: qcm6490-idp: add and enable IPA node
Author mismatch:
  Original author: Sarat Addepalli <sarat.addepalli@oss.qualcomm.com>
  Commit author : Pavan Kumar M <rpavan@qti.qualcomm.com>

The patch metadata shows:

This violates the kernel patch compliance rule that requires the commit author to match the first Signed-off-by: line when applying patches from upstream sources (FROMLIST/UPSTREAM/BACKPORT).

Fix:

The commit author must be set to match the original author. Use one of these approaches:

  1. Amend the commit to fix the author:

    git commit --amend --author="Sarat Addepalli <sarat.addepalli@oss.qualcomm.com>"
  2. If cherry-picking or applying from lore, preserve the original author:

    git am --keep-author <patch-file>
    # or
    git cherry-pick -x <commit-sha>
  3. If Pavan Kumar M is co-developing, the correct format is:

    • Keep Sarat Addepalli as the commit author (From: field)
    • Add Pavan Kumar M's Signed-off-by: after the original author's

Reproduce locally:

cd /path/to/kernel
git clone https://github.com/qualcomm-linux/kernel-checkers.git
bash kernel-checkers/check-patch-compliance.sh --kernel-src . --base <base-sha> --head <pr-head-sha>

Verdict

1 blocker to fix: Author mismatch in check-patch-compliance. The commit author must be changed from "Pavan Kumar M" to "Sarat Addepalli" to match the original patch author. All other checkers passed or were correctly skipped.

@qcomlnxci

Copy link
Copy Markdown

Test Matrix

Test Case lemans-evk monaco-evk qcs615-ride qcs6490-rb3gen2 qcs8300-ride qcs9100-ride-r3 x1e80100-crd
BT_FW_KMD_Service ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
BT_ON_OFF ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
BT_SCAN ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
CPUFreq_Validation ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
CPU_affinity ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
DSP_AudioPD ✅ Pass ✅ Pass ⚠️ skip ✅ Pass ✅ Pass ⚠️ skip ◻️
Ethernet ⚠️ skip ✅ Pass ⚠️ skip ⚠️ skip ⚠️ skip ⚠️ skip ◻️
Freq_Scaling ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
GIC ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
IPA ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
Interrupts ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
OpenCV ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
PCIe ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
Probe_Failure_Check ❌ Fail ❌ Fail ❌ Fail ❌ Fail ❌ Fail ❌ Fail ◻️
RMNET ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
UFS_Validation ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
USBHost ❌ Fail ❌ Fail ❌ Fail ❌ Fail ❌ Fail ❌ Fail ◻️
WiFi_Firmware_Driver ❌ Fail ❌ Fail ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
WiFi_OnOff ✅ Pass ❌ Fail ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
adsp_remoteproc ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ❌ Fail ◻️
cdsp_remoteproc ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ❌ Fail ◻️
gpdsp_remoteproc ✅ Pass ✅ Pass ⚠️ skip ⚠️ skip ✅ Pass ❌ Fail ◻️
hotplug ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
irq ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
kaslr ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
pinctrl ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
qcom_hwrng ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
remoteproc ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ❌ Fail ◻️
rngtest ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
shmbridge ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
smmu ❌ Fail ✅ Pass ❌ Fail ✅ Pass ✅ Pass ❌ Fail ◻️
watchdog ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️
wpss_remoteproc ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ✅ Pass ◻️

@qcomlnxci

Copy link
Copy Markdown

Test Matrix

Test Case lemans-evk monaco-evk qcs615-ride qcs6490-rb3gen2 qcs8300-ride qcs9100-ride-r3 x1e80100-crd
BT_FW_KMD_Service ✅ Pass ✅ Pass ◻️ ✅ Pass ✅ Pass ✅ Pass ◻️
BT_ON_OFF ✅ Pass ✅ Pass ◻️ ✅ Pass ✅ Pass ✅ Pass ◻️
BT_SCAN ✅ Pass ✅ Pass ◻️ ✅ Pass ✅ Pass ✅ Pass ◻️
CPUFreq_Validation ✅ Pass ✅ Pass ◻️ ✅ Pass ✅ Pass ✅ Pass ◻️
CPU_affinity ✅ Pass ✅ Pass ◻️ ✅ Pass ✅ Pass ✅ Pass ◻️
DSP_AudioPD ✅ Pass ✅ Pass ◻️ ✅ Pass ✅ Pass ⚠️ skip ◻️
Ethernet ⚠️ skip ✅ Pass ◻️ ⚠️ skip ⚠️ skip ⚠️ skip ◻️
Freq_Scaling ✅ Pass ✅ Pass ◻️ ✅ Pass ✅ Pass ✅ Pass ◻️
GIC ✅ Pass ✅ Pass ◻️ ✅ Pass ✅ Pass ✅ Pass ◻️
IPA ✅ Pass ✅ Pass ◻️ ✅ Pass ✅ Pass ✅ Pass ◻️
Interrupts ✅ Pass ✅ Pass ◻️ ✅ Pass ✅ Pass ✅ Pass ◻️
OpenCV ✅ Pass ✅ Pass ◻️ ✅ Pass ✅ Pass ✅ Pass ◻️
PCIe ✅ Pass ✅ Pass ◻️ ✅ Pass ✅ Pass ✅ Pass ◻️
Probe_Failure_Check ❌ Fail ❌ Fail ◻️ ❌ Fail ❌ Fail ❌ Fail ◻️
RMNET ✅ Pass ✅ Pass ◻️ ✅ Pass ✅ Pass ✅ Pass ◻️
UFS_Validation ✅ Pass ✅ Pass ◻️ ✅ Pass ✅ Pass ✅ Pass ◻️
USBHost ❌ Fail ❌ Fail ◻️ ❌ Fail ❌ Fail ❌ Fail ◻️
WiFi_Firmware_Driver ❌ Fail ❌ Fail ◻️ ✅ Pass ✅ Pass ✅ Pass ◻️
WiFi_OnOff ✅ Pass ❌ Fail ◻️ ✅ Pass ✅ Pass ✅ Pass ◻️
adsp_remoteproc ✅ Pass ✅ Pass ◻️ ✅ Pass ✅ Pass ❌ Fail ◻️
cdsp_remoteproc ✅ Pass ✅ Pass ◻️ ✅ Pass ✅ Pass ❌ Fail ◻️
gpdsp_remoteproc ✅ Pass ✅ Pass ◻️ ⚠️ skip ✅ Pass ❌ Fail ◻️
hotplug ✅ Pass ✅ Pass ◻️ ✅ Pass ✅ Pass ✅ Pass ◻️
irq ✅ Pass ✅ Pass ◻️ ✅ Pass ✅ Pass ✅ Pass ◻️
kaslr ✅ Pass ✅ Pass ◻️ ✅ Pass ✅ Pass ✅ Pass ◻️
pinctrl ✅ Pass ✅ Pass ◻️ ✅ Pass ✅ Pass ✅ Pass ◻️
qcom_hwrng ✅ Pass ✅ Pass ◻️ ✅ Pass ✅ Pass ✅ Pass ◻️
remoteproc ✅ Pass ✅ Pass ◻️ ✅ Pass ✅ Pass ❌ Fail ◻️
rngtest ✅ Pass ✅ Pass ◻️ ✅ Pass ✅ Pass ✅ Pass ◻️
shmbridge ✅ Pass ✅ Pass ◻️ ✅ Pass ✅ Pass ✅ Pass ◻️
smmu ❌ Fail ✅ Pass ◻️ ✅ Pass ✅ Pass ❌ Fail ◻️
watchdog ✅ Pass ✅ Pass ◻️ ✅ Pass ✅ Pass ✅ Pass ◻️
wpss_remoteproc ✅ Pass ✅ Pass ◻️ ✅ Pass ✅ Pass ✅ Pass ◻️

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.

4 participants