[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <e46c3681-01e8-cb16-2ee8-5947bdd6b62c@st.com>
Date: Mon, 28 Aug 2017 08:13:27 +0000
From: Amelie DELAUNAY <amelie.delaunay@...com>
To: Rob Herring <robh@...nel.org>
CC: Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Mark Rutland <mark.rutland@....com>,
Russell King <linux@...linux.org.uk>,
Maxime Coquelin <mcoquelin.stm32@...il.com>,
Alexandre TORGUE <alexandre.torgue@...com>,
"John Youn" <johnyoun@...opsys.com>,
"linux-usb@...r.kernel.org" <linux-usb@...r.kernel.org>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>,
Benjamin GAIGNARD <benjamin.gaignard@...com>
Subject: Re: [PATCH 1/7] dt-bindings: usb: Document the STM32F7xx DWC2 USB OTG
HS core binding
On 08/22/2017 04:10 AM, Rob Herring wrote:
> On Thu, Aug 17, 2017 at 11:33:00AM +0200, Amelie Delaunay wrote:
>> This patch adds binding documentation for DWC2 controller in HS mode found
>> on STMicroelectronics STM32F7xx SoC.
>>
>> Signed-off-by: Amelie Delaunay <amelie.delaunay@...com>
>> ---
>> Documentation/devicetree/bindings/usb/dwc2.txt | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/usb/dwc2.txt b/Documentation/devicetree/bindings/usb/dwc2.txt
>> index fcf199b..e64d903 100644
>> --- a/Documentation/devicetree/bindings/usb/dwc2.txt
>> +++ b/Documentation/devicetree/bindings/usb/dwc2.txt
>> @@ -19,6 +19,8 @@ Required properties:
>> configured in FS mode;
>> - "st,stm32f4x9-hsotg": The DWC2 USB HS controller instance in STM32F4x9 SoCs
>> configured in HS mode;
>> + - "st,stm32f7xx-hsotg": The DWC2 USB HS controller instance in STM32F7xx SoCs
>> + configured in HS mode;
>
> Don't use wildcards in compatible strings (not sure how the previous one
> snuck in). Just "stm32f7" (i.e. just drop the xx) is probably specific
> enough though.
OK to drop the "xx", I used them only to match the previous one.
>
>> - reg : Should contain 1 register range (address and length)
>> - interrupts : Should contain 1 interrupt
>> - clocks: clock provider specifier
>> --
>> 2.7.4
>>
Powered by blists - more mailing lists