diff mbox

[v6,01/15] usb: doc: phy-mxs: Add more compatible strings

Message ID 1386897825-6130-2-git-send-email-peter.chen@freescale.com (mailing list archive)
State New, archived
Headers show

Commit Message

Peter Chen Dec. 13, 2013, 1:23 a.m. UTC
Add "fsl,imx6q-usbphy" for imx6dq and imx6dl, add
"fsl,imx6sl-usbphy" for imx6sl.

Signed-off-by: Peter Chen <peter.chen@freescale.com>
---
 Documentation/devicetree/bindings/usb/mxs-phy.txt |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

Comments

Lothar Waßmann Dec. 13, 2013, 7:15 a.m. UTC | #1
Hi,

Peter Chen wrote:
> Add "fsl,imx6q-usbphy" for imx6dq and imx6dl, add
> "fsl,imx6sl-usbphy" for imx6sl.
> 
> Signed-off-by: Peter Chen <peter.chen@freescale.com>
> ---
>  Documentation/devicetree/bindings/usb/mxs-phy.txt |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/usb/mxs-phy.txt b/Documentation/devicetree/bindings/usb/mxs-phy.txt
> index 5835b27..d850e55 100644
> --- a/Documentation/devicetree/bindings/usb/mxs-phy.txt
> +++ b/Documentation/devicetree/bindings/usb/mxs-phy.txt
> @@ -1,7 +1,8 @@
>  * Freescale MXS USB Phy Device
>  
>  Required properties:
> -- compatible: Should be "fsl,imx23-usbphy"
> +- compatible: "fsl,imx23-usbphy" for imx23 and imx28, "fsl,imx6q-usbphy"
> +for imx6dq and imx6dl, "fsl,imx6sl-usbphy" for imx6sl
>
indentation?


Lothar Waßmann
Peter Chen Dec. 13, 2013, 7:24 a.m. UTC | #2
On Fri, Dec 13, 2013 at 3:15 PM, Lothar Waßmann <LW@karo-electronics.de> wrote:
> Hi,
>
> Peter Chen wrote:
>> Add "fsl,imx6q-usbphy" for imx6dq and imx6dl, add
>> "fsl,imx6sl-usbphy" for imx6sl.
>>
>> Signed-off-by: Peter Chen <peter.chen@freescale.com>
>> ---
>>  Documentation/devicetree/bindings/usb/mxs-phy.txt |    3 ++-
>>  1 files changed, 2 insertions(+), 1 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/usb/mxs-phy.txt b/Documentation/devicetree/bindings/usb/mxs-phy.txt
>> index 5835b27..d850e55 100644
>> --- a/Documentation/devicetree/bindings/usb/mxs-phy.txt
>> +++ b/Documentation/devicetree/bindings/usb/mxs-phy.txt
>> @@ -1,7 +1,8 @@
>>  * Freescale MXS USB Phy Device
>>
>>  Required properties:
>> -- compatible: Should be "fsl,imx23-usbphy"
>> +- compatible: "fsl,imx23-usbphy" for imx23 and imx28, "fsl,imx6q-usbphy"
>> +for imx6dq and imx6dl, "fsl,imx6sl-usbphy" for imx6sl
>>
> indentation?
>

How many characters? 2 or 8?

BR,
Peter Chen
Lothar Waßmann Dec. 13, 2013, 7:38 a.m. UTC | #3
Hi,

Peter Chen wrote:
> On Fri, Dec 13, 2013 at 3:15 PM, Lothar Waßmann <LW@karo-electronics.de> wrote:
> > Hi,
> >
> > Peter Chen wrote:
> >> Add "fsl,imx6q-usbphy" for imx6dq and imx6dl, add
> >> "fsl,imx6sl-usbphy" for imx6sl.
> >>
> >> Signed-off-by: Peter Chen <peter.chen@freescale.com>
> >> ---
> >>  Documentation/devicetree/bindings/usb/mxs-phy.txt |    3 ++-
> >>  1 files changed, 2 insertions(+), 1 deletions(-)
> >>
> >> diff --git a/Documentation/devicetree/bindings/usb/mxs-phy.txt b/Documentation/devicetree/bindings/usb/mxs-phy.txt
> >> index 5835b27..d850e55 100644
> >> --- a/Documentation/devicetree/bindings/usb/mxs-phy.txt
> >> +++ b/Documentation/devicetree/bindings/usb/mxs-phy.txt
> >> @@ -1,7 +1,8 @@
> >>  * Freescale MXS USB Phy Device
> >>
> >>  Required properties:
> >> -- compatible: Should be "fsl,imx23-usbphy"
> >> +- compatible: "fsl,imx23-usbphy" for imx23 and imx28, "fsl,imx6q-usbphy"
> >> +for imx6dq and imx6dl, "fsl,imx6sl-usbphy" for imx6sl
> >>
> > indentation?
> >
> 
> How many characters? 2 or 8?
>
I would align the line with the first character in the previous line:
- compatible: "fsl,imx23-usbphy" for imx23 and imx28, "fsl,imx6q-usbphy"
  for imx6dq and imx6dl, "fsl,imx6sl-usbphy" for imx6sl


Lothar Waßmann
Peter Chen Dec. 13, 2013, 7:58 a.m. UTC | #4
On Fri, Dec 13, 2013 at 3:38 PM, Lothar Waßmann <LW@karo-electronics.de> wrote:
> Hi,
>
> Peter Chen wrote:
>> On Fri, Dec 13, 2013 at 3:15 PM, Lothar Waßmann <LW@karo-electronics.de> wrote:
>> > Hi,
>> >
>> > Peter Chen wrote:
>> >> Add "fsl,imx6q-usbphy" for imx6dq and imx6dl, add
>> >> "fsl,imx6sl-usbphy" for imx6sl.
>> >>
>> >> Signed-off-by: Peter Chen <peter.chen@freescale.com>
>> >> ---
>> >>  Documentation/devicetree/bindings/usb/mxs-phy.txt |    3 ++-
>> >>  1 files changed, 2 insertions(+), 1 deletions(-)
>> >>
>> >> diff --git a/Documentation/devicetree/bindings/usb/mxs-phy.txt b/Documentation/devicetree/bindings/usb/mxs-phy.txt
>> >> index 5835b27..d850e55 100644
>> >> --- a/Documentation/devicetree/bindings/usb/mxs-phy.txt
>> >> +++ b/Documentation/devicetree/bindings/usb/mxs-phy.txt
>> >> @@ -1,7 +1,8 @@
>> >>  * Freescale MXS USB Phy Device
>> >>
>> >>  Required properties:
>> >> -- compatible: Should be "fsl,imx23-usbphy"
>> >> +- compatible: "fsl,imx23-usbphy" for imx23 and imx28, "fsl,imx6q-usbphy"
>> >> +for imx6dq and imx6dl, "fsl,imx6sl-usbphy" for imx6sl
>> >>
>> > indentation?
>> >
>>
>> How many characters? 2 or 8?
>>
> I would align the line with the first character in the previous line:
> - compatible: "fsl,imx23-usbphy" for imx23 and imx28, "fsl,imx6q-usbphy"
>   for imx6dq and imx6dl, "fsl,imx6sl-usbphy" for imx6sl
>

Thanks, will change.
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/usb/mxs-phy.txt b/Documentation/devicetree/bindings/usb/mxs-phy.txt
index 5835b27..d850e55 100644
--- a/Documentation/devicetree/bindings/usb/mxs-phy.txt
+++ b/Documentation/devicetree/bindings/usb/mxs-phy.txt
@@ -1,7 +1,8 @@ 
 * Freescale MXS USB Phy Device
 
 Required properties:
-- compatible: Should be "fsl,imx23-usbphy"
+- compatible: "fsl,imx23-usbphy" for imx23 and imx28, "fsl,imx6q-usbphy"
+for imx6dq and imx6dl, "fsl,imx6sl-usbphy" for imx6sl
 - reg: Should contain registers location and length
 - interrupts: Should contain phy interrupt