[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <f042b194-659a-b128-7f5a-572150bf532f@linaro.org>
Date: Fri, 28 Jul 2023 18:42:53 +0200
From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To: Varshini Rajendran <varshini.rajendran@...rochip.com>,
gregkh@...uxfoundation.org, robh+dt@...nel.org,
krzysztof.kozlowski+dt@...aro.org, conor+dt@...nel.org,
nicolas.ferre@...rochip.com, alexandre.belloni@...tlin.com,
claudiu.beznea@...rochip.com, linux-usb@...r.kernel.org,
devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3 27/50] dt-bindings: usb: atmel: add sam9x7
On 28/07/2023 12:27, Varshini Rajendran wrote:
> Add sam9x7 bindings.
>
> Signed-off-by: Varshini Rajendran <varshini.rajendran@...rochip.com>
> ---
> Documentation/devicetree/bindings/usb/atmel-usb.txt | 11 +++++++----
> 1 file changed, 7 insertions(+), 4 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/usb/atmel-usb.txt b/Documentation/devicetree/bindings/usb/atmel-usb.txt
> index 12183ef47ee4..82bd0dbc2a48 100644
> --- a/Documentation/devicetree/bindings/usb/atmel-usb.txt
> +++ b/Documentation/devicetree/bindings/usb/atmel-usb.txt
> @@ -3,8 +3,9 @@ Atmel SOC USB controllers
> OHCI
>
> Required properties:
> - - compatible: Should be "atmel,at91rm9200-ohci" for USB controllers
> - used in host mode.
> + - compatible: Should be "atmel,at91rm9200-ohci" or
> + "microchip,sam9x7-ohci", "atmel,at91rm9200-ohci"
> + for USB controllers used in host mode.
> - reg: Address and length of the register set for the device
> - interrupts: Should contain ohci interrupt
> - clocks: Should reference the peripheral, host and system clocks
> @@ -30,8 +31,9 @@ usb0: ohci@...000 {
> EHCI
>
> Required properties:
> - - compatible: Should be "atmel,at91sam9g45-ehci" for USB controllers
> - used in host mode.
> + - compatible: Should be "atmel,at91sam9g45-ehci" or
> + "microchip,sam9x7-ehci", "atmel,at91sam9g45-ehci"
This confuses me. Isn't microchip,sam9x7-ehci already described in EHCI
bindings?
Best regards,
Krzysztof
Powered by blists - more mailing lists