Re: [PATCH v3 1/5] dt-bindings: interconnect: Update property for icc-rpm path tag

From: Leo Yan
Date: Mon Jul 04 2022 - 10:12:27 EST


On Mon, Jul 04, 2022 at 03:50:41PM +0200, Krzysztof Kozlowski wrote:

[...]

> >>> '#interconnect-cells':
> >>> - const: 1
> >>> + description: |
> >>> + Number of interconnect specifier. Value: <1> is one cell in a
> >>> + interconnect specifier for the interconnect node id, <2> requires
> >>> + the interconnect node id and an extra path tag.
> >>> + $ref: /schemas/types.yaml#/definitions/uint32
> >>
> >> Ah, too fast. No ack. This ref is also pointless and not explained in
> >> commit msg.
> >
> > I referred other DT binding file for adding $ref to specify property
> > type [1], so it's confused why it's pointless, anyway, I can drop $ref
> > in next spin.
> >
> > Thanks a lot for the review!
>
> The "arm,prefetch-offset" is not a cells property.

Understand. Will drop $ref for in new patch.

Thanks,
Leo