Re: [PATCH 2/2] arm: boot: dts: am4372: enable dwc3 suspend PHY quirk

2014-11-10 Thread Tony Lindgren
* Felipe Balbi ba...@ti.com [141106 09:40]:
 Whenever Suspend PHY bit is set on AM437x devices,
 USB will not work due to Set EP Configuration command
 always failing.
 
 This was only found after a recent commit 2164a47 (usb:
 dwc3: set SUSPHY bit for all cores, which will be merged
 for v3.19) added a missing *required* step to dwc3
 initialization. Synopsys Databook requires that we enable
 Suspend PHY bit after initialization but that, unfortunately,
 breaks AM437x.
 
 Signed-off-by: Felipe Balbi ba...@ti.com

This should be safe to queue along with the related USB
changes:

Acked-by: Tony Lindgren t...@atomide.com

 ---
  arch/arm/boot/dts/am4372.dtsi | 4 
  1 file changed, 4 insertions(+)
 
 diff --git a/arch/arm/boot/dts/am4372.dtsi b/arch/arm/boot/dts/am4372.dtsi
 index 46660ff..e19068d 100644
 --- a/arch/arm/boot/dts/am4372.dtsi
 +++ b/arch/arm/boot/dts/am4372.dtsi
 @@ -817,6 +817,8 @@
   maximum-speed = high-speed;
   dr_mode = otg;
   status = disabled;
 + snps,dis_u3_susphy_quirk;
 + snps,dis_u2_susphy_quirk;
   };
   };
  
 @@ -839,6 +841,8 @@
   maximum-speed = high-speed;
   dr_mode = otg;
   status = disabled;
 + snps,dis_u3_susphy_quirk;
 + snps,dis_u2_susphy_quirk;
   };
   };
  
 -- 
 2.1.0.GIT
 
 
 ___
 linux-arm-kernel mailing list
 linux-arm-ker...@lists.infradead.org
 http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
 
--
To unsubscribe from this list: send the line unsubscribe linux-usb in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH 2/2] arm: boot: dts: am4372: enable dwc3 suspend PHY quirk

2014-11-06 Thread Felipe Balbi
Whenever Suspend PHY bit is set on AM437x devices,
USB will not work due to Set EP Configuration command
always failing.

This was only found after a recent commit 2164a47 (usb:
dwc3: set SUSPHY bit for all cores, which will be merged
for v3.19) added a missing *required* step to dwc3
initialization. Synopsys Databook requires that we enable
Suspend PHY bit after initialization but that, unfortunately,
breaks AM437x.

Signed-off-by: Felipe Balbi ba...@ti.com
---
 arch/arm/boot/dts/am4372.dtsi | 4 
 1 file changed, 4 insertions(+)

diff --git a/arch/arm/boot/dts/am4372.dtsi b/arch/arm/boot/dts/am4372.dtsi
index 46660ff..e19068d 100644
--- a/arch/arm/boot/dts/am4372.dtsi
+++ b/arch/arm/boot/dts/am4372.dtsi
@@ -817,6 +817,8 @@
maximum-speed = high-speed;
dr_mode = otg;
status = disabled;
+   snps,dis_u3_susphy_quirk;
+   snps,dis_u2_susphy_quirk;
};
};
 
@@ -839,6 +841,8 @@
maximum-speed = high-speed;
dr_mode = otg;
status = disabled;
+   snps,dis_u3_susphy_quirk;
+   snps,dis_u2_susphy_quirk;
};
};
 
-- 
2.1.0.GIT

--
To unsubscribe from this list: send the line unsubscribe linux-usb in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html