Re: [PATCH] dt-bindings: arm: qcom: add ipq8064 board variants

[Rob Herring <robh@kernel.org>]

On Thu, Nov 20, 2025 at 04:31:47PM +0100, Christian Marangi wrote:
> On Thu, Nov 20, 2025 at 04:25:37PM +0100, Konrad Dybcio wrote:
> > On 11/20/25 2:54 PM, Christian Marangi wrote:
> > > Document QCOM ipq8064 board variants ipq8062, ipq8065, ipq8066,
> > > ipq8068, ipq8069 now matched by the QCOM cpufreq nvmem driver if
> > > socinfo can't derive the variant from SMEM.
> > > 
> > > Suggested-by: Rob Herring <robh@kernel.org>
> > > Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
> > > ---
> > >  .../devicetree/bindings/arm/qcom.yaml         | 20 +++++++++++++++++++
> > >  1 file changed, 20 insertions(+)
> > > 
> > > diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
> > > index 18b5ed044f9f..0eb1619fede8 100644
> > > --- a/Documentation/devicetree/bindings/arm/qcom.yaml
> > > +++ b/Documentation/devicetree/bindings/arm/qcom.yaml
> > > @@ -299,12 +299,32 @@ properties:
> > >                - qcom,ipq5424-rdp466
> > >            - const: qcom,ipq5424
> > >  
> > > +      - items:
> > > +          - const: qcom,ipq8062
> > > +          - const: qcom,ipq8064
> > 
> > Since 'items' requires that all items are present (and in this order),
> > we would normally have a board name go first.. but I suppose this is
> > some sort of a fix to the issue that sparked this (posting the link
> > for others to have more context)
> > 
> > But since these SoCs do exist, I wouldn't say this is necessarily
> > wrong..
> >
> 
> Well we can see this as a ""template"" for device that might be added
> using the ipq8062 or ipq8065 compatible.
> 
> When device with that variant will be added we would have to just add an
> enum with the real device name on top of it (as first element).
> 
> Honestly I should have added these compatible long time ago as on
> OpenWrt we have tons of device that are ipq8062 or ipq8065 with the
> compatible structure
> 
> "device,name", "qcom,ipq8065", "qcom,ipq8064".

If you don't you have any boards yet, you can do:

items:
  - description: ...
  - const: qcom,ipq8065
  - const: qcom,ipq8064

Just to prevent skipping a board compatible.

But you said you have tons of devices, so...

Rob