Devicetree
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: arm: qcom: Document Eliza CQM and CQS EVK boards
@ 2026-05-15 11:13 Abel Vesa
  2026-05-15 11:29 ` sashiko-bot
  0 siblings, 1 reply; 2+ messages in thread
From: Abel Vesa @ 2026-05-15 11:13 UTC (permalink / raw)
  To: Bjorn Andersson, Konrad Dybcio, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley
  Cc: linux-arm-msm, devicetree, linux-kernel, Abel Vesa

Document the compatible strings for the Qualcomm Eliza CQM and CQS EVK
boards. Both boards are built from a base board paired with a SoM which
is populated with either CQ7790M or CQ7790S (Eliza variants), PMICs,
LPDDR, eMMC and UFS.

Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
---
 Documentation/devicetree/bindings/arm/qcom.yaml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
index 2741c07e9f41..28a18254b4b7 100644
--- a/Documentation/devicetree/bindings/arm/qcom.yaml
+++ b/Documentation/devicetree/bindings/arm/qcom.yaml
@@ -63,6 +63,8 @@ properties:
 
       - items:
           - enum:
+              - qcom,eliza-cqm-evk
+              - qcom,eliza-cqs-evk
               - qcom,eliza-mtp
           - const: qcom,eliza
 

---
base-commit: e98d21c170b01ddef366f023bbfcf6b31509fa83
change-id: 20260514-eliza-bindings-evk-1f05f1c699d5

Best regards,
--  
Abel Vesa <abel.vesa@oss.qualcomm.com>


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] dt-bindings: arm: qcom: Document Eliza CQM and CQS EVK boards
  2026-05-15 11:13 [PATCH] dt-bindings: arm: qcom: Document Eliza CQM and CQS EVK boards Abel Vesa
@ 2026-05-15 11:29 ` sashiko-bot
  0 siblings, 0 replies; 2+ messages in thread
From: sashiko-bot @ 2026-05-15 11:29 UTC (permalink / raw)
  To: Abel Vesa; +Cc: robh, krzk+dt, devicetree, conor+dt

Thank you for your contribution! Sashiko AI review found 2 potential issue(s) to consider:
- [Medium] The compatible strings strictly enforce a 2-element list that omits the intermediate System-on-Module (SoM) and the specific SoC variant compatibles.
- [Medium] The compatible strings and commit message introduce typographical errors by using "CQM", "CQS", "CQ7790M", and "CQ7790S" instead of Qualcomm's standard "QCM" and "QCS" product naming convention.
--

commit 5133e80c24b03ea2551cf3393bae586847e425a8
Author: Abel Vesa <abel.vesa@oss.qualcomm.com>

dt-bindings: arm: qcom: Document Eliza CQM and CQS EVK boards

This commit documents the compatible strings for the Qualcomm Eliza CQM and
CQS EVK boards, which are built from a base board paired with a SoM that is
populated with either CQ7790M or CQ7790S, PMICs, LPDDR, eMMC and UFS.

> dt-bindings: arm: qcom: Document Eliza CQM and CQS EVK boards
>
> Document the compatible strings for the Qualcomm Eliza CQM and CQS EVK
> boards. Both boards are built from a base board paired with a SoM which
> is populated with either CQ7790M or CQ7790S (Eliza variants), PMICs,
> LPDDR, eMMC and UFS.

This isn't a bug, but there appear to be typos in the product names here.
Should CQM, CQS, CQ7790M, and CQ7790S be QCM, QCS, QCM7790, and QCS7790 to
match standard Qualcomm naming conventions?

> diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
> index 2741c07e9f416..28a18254b4b75 100644
> --- a/Documentation/devicetree/bindings/arm/qcom.yaml
> +++ b/Documentation/devicetree/bindings/arm/qcom.yaml
> @@ -63,6 +63,8 @@ properties:
>  
>        - items:
>            - enum:
> +              - qcom,eliza-cqm-evk
> +              - qcom,eliza-cqs-evk

If the typo mentioned above is confirmed, will these compatible strings
permanently bake the misspelled cqm and cqs names into the device tree ABI?

Additionally, does this 2-element list correctly represent the hardware
hierarchy? The commit message notes these EVK boards are composed of a base
board paired with a SoM populated with specific SoC variants. Placing these
compatibles in this block restricts the list to exactly two elements, which
omits the intermediate SoM and SoC variant compatibles. This might cause
validation to fail on a device tree that attempts to include the SoM
compatible string.

>                - qcom,eliza-mtp
>            - const: qcom,eliza

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260515-eliza-bindings-evk-v1-1-8deb81ed86dd@oss.qualcomm.com?part=1

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-05-15 11:29 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-05-15 11:13 [PATCH] dt-bindings: arm: qcom: Document Eliza CQM and CQS EVK boards Abel Vesa
2026-05-15 11:29 ` sashiko-bot

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox