Re: [PATCH 1/9] dt-bindings: arm: Add support for DSB element

[Tao Zhang]

在 9/8/2022 6:54 PM, Krzysztof Kozlowski 写道:
> On 08/09/2022 10:44, Tao Zhang wrote:
> > Add property "qcom,dsb-elem-size" to support DSB element for TPDA.
> > Specifies the DSB element size supported by each monitor connected
> > to the aggregator on each port. Should be specified in pairs (port,
> > dsb element size).
> >
> > Signed-off-by: Tao Zhang <quic_taozha@xxxxxxxxxxx>
> > ---
> >   Documentation/devicetree/bindings/arm/qcom,coresight-tpda.yaml | 9 +++++++++
> >   1 file changed, 9 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/arm/qcom,coresight-tpda.yaml b/Documentation/devicetree/bindings/arm/qcom,coresight-tpda.yaml
> > index eb9bfc5..1bb3fdf 100644
> > --- a/Documentation/devicetree/bindings/arm/qcom,coresight-tpda.yaml
> > +++ b/Documentation/devicetree/bindings/arm/qcom,coresight-tpda.yaml
> > @@ -40,6 +40,13 @@ properties:
> >       minItems: 1
> >       maxItems: 2
> >   
> > +  qcom,dsb-elem-size:
> > +    description: |
> > +      Specifies the DSB element size supported by each monitor
> > +      connected to the aggregator on each port. Should be specified
> > +      in pairs (port, dsb element size).
> > +    $ref: /schemas/types.yaml#/definitions/uint32-array
> So it is rather uint32-matrix (need to describe the items subschema).
> What about maxItems?
>
> Best regards,
> Krzysztof

Yes, indeed it should be uint32-matrix here. I will update in the next 
release.

The "maxItems" cannot be known explicitly because it depends on how many 
DSB subunit TPDMs are connected to the TPDA input ports.

Usually the number of the items is 1 to several, but there is no limit 
to its maximum value.


Best regards,

Tao