GLINK wakelock changes (debug)

[jprakash-qc]

No description provided.

[qlijarvis]

PR #1247 — validate-patch

PR: #1247

Verdict	Issues	Detailed Report
❌	10	Full report

Final Summary

1. Lore link present: No — no lore.kernel.org link found in either commit message
2. Lore link matches PR commits: N/A — no lore link to compare against
3. Upstream patch status: Not upstream — no evidence these patches were posted to any mailing list
4. PR present in qcom-next: Not checked — cannot verify without proper commit attribution

Verdict: ❌ — click to expand

🔍 Patch Validation

PR: #1247 (ucsi wakelock + sysfs control)
Upstream commit: N/A
Verdict: ❌ FAIL

Commit Message

Check	Status	Note
Subject matches upstream	❌	No upstream link to compare against
Body preserves rationale	❌	Commit messages are empty — no description, no rationale
Fixes tag present/correct	❌	No Fixes tag present
Authorship preserved	N/A	No upstream source to verify against
Backport note (if applicable)	N/A	Not a backport

Diff

File	Status	Notes
drivers/usb/typec/ucsi/ucsi_glink.c	⚠️	Changes appear reasonable but lack upstream attribution

Issues

Commit 1: "ucsi wakelock"

• ❌ No lore.kernel.org link in commit message
• ❌ No vendor prefix (UPSTREAM:/FROMLIST:/QCLINUX:/PENDING:/WORKAROUND:)
• ❌ Empty commit body — no description of why wakelock is needed
• ❌ No Signed-off-by chain beyond author
• ⚠️ Adds pm_wakeup_event() with 200ms timeout but comment says "10s gives sufficient margin" — inconsistency between comment and code

Commit 2: "sysfs control"

• ❌ No lore.kernel.org link in commit message
• ❌ No vendor prefix
• ❌ Empty commit body — no description of purpose or use case
• ❌ No Signed-off-by chain beyond author
• ⚠️ Adds sysfs ABI without Documentation/ABI/ entry (required for new sysfs attributes)

Verdict

Do not merge. Both commits lack proper attribution, commit message bodies, and upstream links. The changes appear to be original work that should either:

1. Be posted upstream first (use FROMLIST: prefix + lore link), or
2. Be marked as vendor-only (use QCLINUX: prefix if not suitable for upstream)

Additionally, commit messages must include a description of the problem being solved and the rationale for the solution.

Final Summary

1. Lore link present: No — no lore.kernel.org link found in either commit message
2. Lore link matches PR commits: N/A — no lore link to compare against
3. Upstream patch status: Not upstream — no evidence these patches were posted to any mailing list
4. PR present in qcom-next: Not checked — cannot verify without proper commit attribution

[qlijarvis]

PR #1247 — checker-log-analyzer

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

Checker	Result	Summary
Checker	Result	Summary
checkpatch	❌	2 commits with missing commit descriptions
dt-binding-check	⏭️	No DT binding changes
dtb-check	⏭️	No DTS changes
sparse-check	✅	Passed
check-uapi-headers	✅	No UAPI changes
check-patch-compliance	❌	2 commits missing required prefix
tag-check	❌	2 commits missing subject prefix (mandatory for tech/pmic/misc branch)

Detailed report: Full report

Checker analysis — click to expand

🤖 CI Checker Analysis (checker-log-analyzer)

PR: #1247 - ucsi wakelock and sysfs control
Source: https://github.com/qualcomm-linux/kernel-config/actions/runs/26492992919
Target branch: tech/pmic/misc

Checker	Result	Summary
checkpatch	❌	2 commits with missing commit descriptions
dt-binding-check	⏭️	No DT binding changes
dtb-check	⏭️	No DTS changes
sparse-check	✅	Passed
check-uapi-headers	✅	No UAPI changes
check-patch-compliance	❌	2 commits missing required prefix
tag-check	❌	2 commits missing subject prefix (mandatory for tech/pmic/misc branch)

---

❌ checkpatch

Root cause: Both commits lack commit message body descriptions.

Failure details:

Commit d9a47eb66a8e ("ucsi wakelock")
WARNING: Missing commit description - Add an appropriate one

Commit e392cf124d4b ("sysfs control")
WARNING: Missing commit description - Add an appropriate one

Fix: Add a proper commit message body explaining:

• Commit 1 (ucsi wakelock): Why the wakeup lock is needed, what problem it solves (e.g., preventing system suspend during UCSI notification handling), and the rationale for the 200ms timeout.
• Commit 2 (sysfs control): Why the sysfs attribute is needed, what use case it serves, and how it allows runtime configuration of the wakeup timeout.

Reproduce locally:

./scripts/checkpatch.pl --strict --ignore FILE_PATH_CHANGES --git df3ae9703774b70a7b7758b53498a25de9f87174..e392cf124d4bfe60775b0e3a54c0c20030a8dc12

---

❌ check-patch-compliance

Root cause: Both commits lack the required subject-line prefix for upstream traceability.

Failure details:

Checking commit: ucsi wakelock
Commit summary does not start with a required prefix

Checking commit: sysfs control
Commit summary does not start with a required prefix

Fix: Add the appropriate prefix to each commit subject:

Prefix	When to use
FROMLIST:	Patch posted to lore.kernel.org but not yet merged
FROMGIT:	Patch taken from a maintainer git tree
UPSTREAM:	Patch merged into Linus's mainline tree
BACKPORT:	Upstream patch backported with modifications
QCLINUX:	Vendor-only change (note: will still fail this checker)

If these patches are vendor-only and not posted upstream, use QCLINUX: prefix (though the checker will still fail — this is a known limitation). If they are posted or will be posted upstream, use FROMLIST: and add a Link: tag pointing to the lore.kernel.org URL.

Example fix:

git rebase -i df3ae9703774b70a7b7758b53498a25de9f87174
# Mark both commits as 'edit'
git commit --amend -m "QCLINUX: usb: typec: ucsi_glink: Add wakeup lock for notification handling"
git rebase --continue
git commit --amend -m "QCLINUX: usb: typec: ucsi_glink: Add sysfs control for wakeup timeout"
git rebase --continue

---

❌ tag-check

Root cause: Target branch tech/pmic/misc requires all commits to have a subject-line prefix tag, but both commits are missing it.

Failure details:

Commit d9a47eb66a8e ("ucsi wakelock"): subject line does not start with a required prefix.
Commit e392cf124d4b ("sysfs control"): subject line does not start with a required prefix.

All branches except qcom-next and qcom-next-staging require one of:
FROMLIST: / FROMGIT: / UPSTREAM: / BACKPORT: / QCLINUX: / PENDING: / WORKAROUND:

Fix: Same as check-patch-compliance — add the appropriate prefix to both commit subjects. Choose based on the patch origin:

• Vendor-only → QCLINUX:
• Posted to lore → FROMLIST:
• In maintainer tree → FROMGIT:
• Merged upstream → UPSTREAM:

---

⚠️ Additional Issue: Undocumented sysfs attribute

Root cause: Commit 2 adds a new sysfs attribute wakeup_timeout_ms but does not document it in Documentation/ABI/.

Details:
The patch adds DEVICE_ATTR_RW(wakeup_timeout_ms) which creates /sys/devices/.../wakeup_timeout_ms. All new sysfs attributes must be documented per kernel ABI policy.

Fix: Add a documentation file:

# Create Documentation/ABI/testing/sysfs-driver-ucsi-glink
cat > Documentation/ABI/testing/sysfs-driver-ucsi-glink <<EOF
What:		/sys/bus/platform/drivers/pmic_glink_ucsi/*/wakeup_timeout_ms
Date:		May 2026
KernelVersion:	6.15
Contact:	Jishnu Prakash <jishnu.prakash@oss.qualcomm.com>
Description:
		Wakeup timeout in milliseconds for UCSI notification handling.
		When a UCSI notification is received, the driver holds a wakeup
		lock for this duration to prevent system suspend while the
		notification chain completes. Default is 200ms.
EOF

Include this file in commit 2 or as a separate commit in the series.

---

Verdict

3 blockers must be fixed before merge:

1. Add commit message bodies to both commits explaining the changes.
2. Add subject-line prefix (QCLINUX: or FROMLIST:) to both commits.
3. Document the new sysfs attribute in Documentation/ABI/testing/.

Once fixed, re-run the checkers to verify all issues are resolved.

[jprakash-qc]

DNM, this is for testing first