[PATCH v3 3/6] usb: dwc3: dwc3-st: Update the incorrect DT dwc3 example.

[PATCH v3 3/6] usb: dwc3: dwc3-st: Update the incorrect DT dwc3 example.

[Peter Griffin]

There is a subtle typo phys-names should be phy-names. Using the
current example means you don't have working usb (as you fail to
obtain the phys). Also update the example to use the generic phy
type constants which are now used for miphy28.

Additionally also remove the unnecessary new line in the example.

Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
Acked-by: Rob Herring <robh@kernel.org>
Acked-by: Maxime Coquelin <maxime.coquelin@st.com>
---
 Documentation/devicetree/bindings/usb/dwc3-st.txt | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/Documentation/devicetree/bindings/usb/dwc3-st.txt b/Documentation/devicetree/bindings/usb/dwc3-st.txt
index f9d7025..01c71b1 100644
--- a/Documentation/devicetree/bindings/usb/dwc3-st.txt
+++ b/Documentation/devicetree/bindings/usb/dwc3-st.txt
@@ -49,8 +49,7 @@ st_dwc3: dwc3@8f94000 {
 	st,syscfg	= <&syscfg_core>;
 	resets		= <&powerdown STIH407_USB3_POWERDOWN>,
 			  <&softreset STIH407_MIPHY2_SOFTRESET>;
-	reset-names	= "powerdown",
-			  "softreset";
+	reset-names	= "powerdown", "softreset";
 	#address-cells	= <1>;
 	#size-cells	= <1>;
 	pinctrl-names	= "default";
@@ -62,7 +61,7 @@ st_dwc3: dwc3@8f94000 {
 		reg		= <0x09900000 0x100000>;
 		interrupts	= <GIC_SPI 155 IRQ_TYPE_NONE>;
 		dr_mode		= "host";
-		phys-names      = "usb2-phy", "usb3-phy";
-		phys            = <&usb2_picophy2>, <&phy_port2 MIPHY_TYPE_USB>;
+		phy-names	= "usb2-phy", "usb3-phy";
+		phys		= <&usb2_picophy2>, <&phy_port2 PHY_TYPE_USB3>;
 	};
 };
-- 
1.9.1

[PATCH v3 0/6] Add dwc3 usb3 DT node and various phy cleanup

[Peter Griffin]

Hi Maxime,

This series primarily adds the dwc3 DT node which activates the usb3 controller on
the stih407-b2120, stih410-b2120 and stih418-b2199 platforms.

As part of getting this working I noticed an error with the picophyreset which 
stops the controller working which I've also fixed.

regards,

Peter.

Changes in v3:
- Add miphys8lp options properties for stih418 (Pete)
- Add support for stih418-b2199 (Max)
- Only enable usb3 controller in the board file (Max)
- Fix superflous '\n' (Lee)
- Rebase series on v4.1-rc1 (Pete)

NB: Since v2 some patches were taken into the phy and ahci trees which I've now
removed from the series.

Changes in v2:
- Update commit to explicitly say no ABI breakage (Rob)
- Rebase series on v4.0-rc6 (Pete)
- Collect up Acks (Pete)

Peter Griffin (6):
  ARM: DT: STi: STiH407: Update picophyreset for the usb3 controllers
    usb2 phy
  ARM: DT: STi: STiH407: Add dwc3 usb3 DT node.
  usb: dwc3: dwc3-st: Update the incorrect DT dwc3 example.
  ARM: DT: STi: stihxxx-b2120: Enable USB3 port on stih407-b2120 and
    stih410-b2120
  ARM: DT: STi: STiH418: Add miphy28lp optional oscillator clock
    properties
  ARM: DT: STi: STiH418: Enable USB3 port on stih418-b2199.

 Documentation/devicetree/bindings/usb/dwc3-st.txt |  7 +++---
 arch/arm/boot/dts/stih407-family.dtsi             | 30 ++++++++++++++++++++++-
 arch/arm/boot/dts/stih418-b2199.dts               | 15 ++++++++++++
 arch/arm/boot/dts/stihxxx-b2120.dtsi              |  5 ++++
 4 files changed, 52 insertions(+), 5 deletions(-)

-- 
1.9.1

[PATCH v3 3/6] usb: dwc3: dwc3-st: Update the incorrect DT dwc3 example.

[Peter Griffin]

There is a subtle typo phys-names should be phy-names. Using the
current example means you don't have working usb (as you fail to
obtain the phys). Also update the example to use the generic phy
type constants which are now used for miphy28.

Additionally also remove the unnecessary new line in the example.

Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
Acked-by: Rob Herring <robh@kernel.org>
Acked-by: Maxime Coquelin <maxime.coquelin@st.com>
---
 Documentation/devicetree/bindings/usb/dwc3-st.txt | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/Documentation/devicetree/bindings/usb/dwc3-st.txt b/Documentation/devicetree/bindings/usb/dwc3-st.txt
index f9d7025..01c71b1 100644
--- a/Documentation/devicetree/bindings/usb/dwc3-st.txt
+++ b/Documentation/devicetree/bindings/usb/dwc3-st.txt
@@ -49,8 +49,7 @@ st_dwc3: dwc3@8f94000 {
 	st,syscfg	= <&syscfg_core>;
 	resets		= <&powerdown STIH407_USB3_POWERDOWN>,
 			  <&softreset STIH407_MIPHY2_SOFTRESET>;
-	reset-names	= "powerdown",
-			  "softreset";
+	reset-names	= "powerdown", "softreset";
 	#address-cells	= <1>;
 	#size-cells	= <1>;
 	pinctrl-names	= "default";
@@ -62,7 +61,7 @@ st_dwc3: dwc3@8f94000 {
 		reg		= <0x09900000 0x100000>;
 		interrupts	= <GIC_SPI 155 IRQ_TYPE_NONE>;
 		dr_mode		= "host";
-		phys-names      = "usb2-phy", "usb3-phy";
-		phys            = <&usb2_picophy2>, <&phy_port2 MIPHY_TYPE_USB>;
+		phy-names	= "usb2-phy", "usb3-phy";
+		phys		= <&usb2_picophy2>, <&phy_port2 PHY_TYPE_USB3>;
 	};
 };
-- 
1.9.1

Re: [PATCH v3 3/6] usb: dwc3: dwc3-st: Update the incorrect DT dwc3 example.

[Peter Griffin]

Hi Felipe,

On Thu, 30 Apr 2015, Peter Griffin wrote:

> There is a subtle typo phys-names should be phy-names. Using the
> current example means you don't have working usb (as you fail to
> obtain the phys). Also update the example to use the generic phy
> type constants which are now used for miphy28.
> 
> Additionally also remove the unnecessary new line in the example.
> 
> Signed-off-by: Peter Griffin <peter.griffin-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
> Acked-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
> Acked-by: Maxime Coquelin <maxime.coquelin-qxv4g6HH51o@public.gmane.org>

Can you take this dwc3 DT doc update patch via your tree?

regards,

Peter.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH v3 3/6] usb: dwc3: dwc3-st: Update the incorrect DT dwc3 example.

[Felipe Balbi]

[-- Attachment #1: Type: text/plain, Size: 798 bytes --]

On Thu, May 07, 2015 at 04:13:58PM +0100, Peter Griffin wrote:
> Hi Felipe,
> 
> On Thu, 30 Apr 2015, Peter Griffin wrote:
> 
> > There is a subtle typo phys-names should be phy-names. Using the
> > current example means you don't have working usb (as you fail to
> > obtain the phys). Also update the example to use the generic phy
> > type constants which are now used for miphy28.
> > 
> > Additionally also remove the unnecessary new line in the example.
> > 
> > Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
> > Acked-by: Rob Herring <robh@kernel.org>
> > Acked-by: Maxime Coquelin <maxime.coquelin@st.com>
> 
> Can you take this dwc3 DT doc update patch via your tree?

only this patch ? Sure. Just post on linux-usb and I'll fetch it from
there.

-- 
balbi

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

Re: [PATCH v3 3/6] usb: dwc3: dwc3-st: Update the incorrect DT dwc3 example.

[Peter Griffin]

Hi Felipe,

On Thu, 07 May 2015, Felipe Balbi wrote:

> On Thu, May 07, 2015 at 04:13:58PM +0100, Peter Griffin wrote:
> > Hi Felipe,
> > 
> > On Thu, 30 Apr 2015, Peter Griffin wrote:
> > 
> > > There is a subtle typo phys-names should be phy-names. Using the
> > > current example means you don't have working usb (as you fail to
> > > obtain the phys). Also update the example to use the generic phy
> > > type constants which are now used for miphy28.
> > > 
> > > Additionally also remove the unnecessary new line in the example.
> > > 
> > > Signed-off-by: Peter Griffin <peter.griffin-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
> > > Acked-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
> > > Acked-by: Maxime Coquelin <maxime.coquelin-qxv4g6HH51o@public.gmane.org>
> > 
> > Can you take this dwc3 DT doc update patch via your tree?
> 
> only this patch ?

Yes, I think the test will go via Maximes STI tree

> Sure. Just post on linux-usb and I'll fetch it from
> there.

Brilliant, thanks. I've sent it see here https://lkml.org/lkml/2015/5/7/809
with linux-usb on CC (sorry for missing that off the first time round)

regards,

Peter.


--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH v3 3/6] usb: dwc3: dwc3-st: Update the incorrect DT dwc3 example.

[Felipe Balbi]

[-- Attachment #1: Type: text/plain, Size: 1264 bytes --]

On Fri, May 08, 2015 at 11:31:40AM +0100, Peter Griffin wrote:
> Hi Felipe,
> 
> On Thu, 07 May 2015, Felipe Balbi wrote:
> 
> > On Thu, May 07, 2015 at 04:13:58PM +0100, Peter Griffin wrote:
> > > Hi Felipe,
> > > 
> > > On Thu, 30 Apr 2015, Peter Griffin wrote:
> > > 
> > > > There is a subtle typo phys-names should be phy-names. Using the
> > > > current example means you don't have working usb (as you fail to
> > > > obtain the phys). Also update the example to use the generic phy
> > > > type constants which are now used for miphy28.
> > > > 
> > > > Additionally also remove the unnecessary new line in the example.
> > > > 
> > > > Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
> > > > Acked-by: Rob Herring <robh@kernel.org>
> > > > Acked-by: Maxime Coquelin <maxime.coquelin@st.com>
> > > 
> > > Can you take this dwc3 DT doc update patch via your tree?
> > 
> > only this patch ?
> 
> Yes, I think the test will go via Maximes STI tree
> 
> > Sure. Just post on linux-usb and I'll fetch it from
> > there.
> 
> Brilliant, thanks. I've sent it see here https://lkml.org/lkml/2015/5/7/809
> with linux-usb on CC (sorry for missing that off the first time round)

it's in my testing/next now.

-- 
balbi

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]