Re: [PATCH 2/8] dt-bindings: rng: atmel,at91-trng: convert Atmel TRNG to dtschema

[Nicolas Ferre]

On 28/12/2021 at 14:07, Krzysztof Kozlowski wrote:
> On 28/12/2021 13:54, Rob Herring wrote:
> > On Mon, 27 Dec 2021 19:32:45 +0100, Krzysztof Kozlowski wrote:
> > > Convert the Atmel TRNG bindings to DT schema.
> > >
> > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxxxxx>
> > > ---
> > >   .../bindings/rng/atmel,at91-trng.yaml         | 46 +++++++++++++++++++
> > >   .../devicetree/bindings/rng/atmel-trng.txt    | 16 -------
> > >   2 files changed, 46 insertions(+), 16 deletions(-)
> > >   create mode 100644 Documentation/devicetree/bindings/rng/atmel,at91-trng.yaml
> > >   delete mode 100644 Documentation/devicetree/bindings/rng/atmel-trng.txt
> >
> > Running 'make dtbs_check' with the schema in this patch gives the
> > following warnings. Consider if they are expected or the schema is
> > incorrect. These may not be new warnings.
> >
> > Note that it is not yet a requirement to have 0 warnings for dtbs_check.
> > This will change in the future.
> >
> > Full log is available here: https://patchwork.ozlabs.org/patch/1573492
> >
> >
> > rng@e2010000: compatible:0: 'microchip,sama7g5-trng' is not one of ['atmel,at91sam9g45-trng', 'microchip,sam9x60-trng']
> >        arch/arm/boot/dts/at91-sama7g5ek.dt.yaml
> >
> > rng@e2010000: compatible: Additional items are not allowed ('atmel,at91sam9g45-trng' was unexpected)
> >        arch/arm/boot/dts/at91-sama7g5ek.dt.yaml
> >
> > rng@e2010000: compatible: ['microchip,sama7g5-trng', 'atmel,at91sam9g45-trng'] is too long
> >        arch/arm/boot/dts/at91-sama7g5ek.dt.yaml
>
> This patch is only converting the existing bindings, so new compatible
> is not added. The issue reported here is being fixed in the next patch
> (3/8).

Thanks Krzysztof!
Acked-by: Nicolas Ferre <nicolas.ferre@xxxxxxxxxxxxx>

Best regards,
  Nicolas


--
Nicolas Ferre