Re: [PATCH v4 05/39] dt-bindings: crypto: add sam9x7 in Atmel TDES

Tudor Ambarus
2024-02-26T09:24:34Z

On 23.02.2024 19:24, Varshini Rajendran wrote:
> Add DT bindings for atmel TDES.
>
> Signed-off-by: Varshini Rajendran <varshini.rajendran@microchip.com>
> Acked-by: Nicolas Ferre <nicolas.ferre@microchip.com>

This would have deserved a better commit message, I (we) spent a lot of
time deciding whether this is the correct approach.
https://lore.kernel.org/linux-arm-kernel/342de8f3-852f-9bfa-39c4-4d820f349305@linaro.org/

Anyway:
Reviewed-by: Tudor Ambarus <tudor.ambarus@linaro.org>

> ---
> Changes in v4:
> - Updated Acked-by tag
> ---
> .../devicetree/bindings/crypto/atmel,at91sam9g46-tdes.yaml | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/crypto/atmel,at91sam9g46-tdes.yaml b/Documentation/devicetree/bindings/crypto/atmel,at91sam9g46-tdes.yaml
> index 3d6ed24b1b00..6a441f79efea 100644
> --- a/Documentation/devicetree/bindings/crypto/atmel,at91sam9g46-tdes.yaml
> +++ b/Documentation/devicetree/bindings/crypto/atmel,at91sam9g46-tdes.yaml
> @@ -12,7 +12,11 @@ maintainers:
>
> properties:
> compatible:
> - const: atmel,at91sam9g46-tdes
> + oneOf:
> + - const: atmel,at91sam9g46-tdes
> + - items:
> + - const: microchip,sam9x7-tdes
> + - const: atmel,at91sam9g46-tdes
>
> reg:
> maxItems: 1