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

From: Krzysztof Kozlowski
Date: Tue Dec 28 2021 - 08:08:22 EST


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).

Best regards,
Krzysztof