Re: [PATCH] dt-bindings: arm: qcom: Document Eliza CQM and CQS EVK boards

[Abel Vesa]

On 26-05-16 12:30:41, Krzysztof Kozlowski wrote:
> On 16/05/2026 12:27, Abel Vesa wrote:
> > On 26-05-16 11:58:56, Krzysztof Kozlowski wrote:
> >> On Fri, May 15, 2026 at 02:13:31PM +0300, Abel Vesa wrote:
> >>> 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@xxxxxxxxxxxxxxxx>
> >>> ---
> >>>  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
> >>
> >> When we talked privately, I asked to mirror Shikra approach. But you did
> >> not - it's different from what Shikra sent.
> > 
> > Because I think it is wrong to describe the SoM since it can't be used
> > without a base board. Listing SoMs is useless, IMO. But maybe I'm wrong.
> > 
> > So my reason for doing this is basically reducing the list to only
> > describe entire setups: MTP, CQM EVK and CQS EVK. No SoMs.
> 
> That's really unique statement. All or most of other vendors, including
> the most popular for SoMs - NXP, use it. But of course existing practice
> is not the sole reason.
> 
> The reason is that it is the same useful as having SoC compatible. You
> cannot use SoC without base board, then why do you have SoC compatible?
> To allow quirks for the SoC. And here to allow quirks for the SoM, since
> it is a dedicated and specific hardware component.

Yeah, makes sense.