[PATCH v2] arm: koelsch: make USB0 perform Host/Function switching

2015-07-02 Thread Phil Edworthy 
Both USB Host (pci0) and Function (USBHS) drivers are enabled.
The USB PHY driver determines which IP block should be connected
based on vbus and id signals read via gpios.

Note that switch SW5 and SW6 on Koelsch board needs to be set to
position 3 for this to work.

---

Not for upstream until the following patches have been accepted:
usb: renesas_usbhs: Allow an OTG PHY driver to provide VBUS
phy: rcar-gen2 usb: Add Host/Function switching for USB0
Hence, not signed off.

v2:
  - Added -gpio to dts prop names of GPIO pins.
---
 arch/arm/boot/dts/r8a7791-koelsch.dts | 7 ---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/arch/arm/boot/dts/r8a7791-koelsch.dts 
b/arch/arm/boot/dts/r8a7791-koelsch.dts
index cffe33f..1bb34d0 100644
--- a/arch/arm/boot/dts/r8a7791-koelsch.dts
+++ b/arch/arm/boot/dts/r8a7791-koelsch.dts
@@ -615,7 +615,6 @@
 
 pci0 {
status = okay;
-   pinctrl-0 = usb0_pins;
pinctrl-names = default;
 };
 
@@ -627,13 +626,15 @@
 
 hsusb {
status = okay;
-   pinctrl-0 = usb0_pins;
pinctrl-names = default;
-   renesas,enable-gpio = gpio5 31 GPIO_ACTIVE_HIGH;
 };
 
 usbphy {
status = okay;
+   renesas,pwr-gpio = gpio2 4 GPIO_ACTIVE_HIGH;
+   renesas,id-gpio = gpio5 31 GPIO_ACTIVE_HIGH;
+   renesas,vbus-gpio = gpio7 24 GPIO_ACTIVE_HIGH;
+   renesas,vbus-pwr-gpio = gpio7 23 GPIO_ACTIVE_HIGH;
 };
 
 pcie_bus_clk {
-- 
1.9.1

--
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

Re: [PATCH v2] arm: koelsch: make USB0 perform Host/Function switching

2015-07-02 Thread Sergei Shtylyov 

Hello.

On 7/2/2015 11:14 AM, Phil Edworthy wrote:


Both USB Host (pci0) and Function (USBHS) drivers are enabled.
The USB PHY driver determines which IP block should be connected
based on vbus and id signals read via gpios.



Note that switch SW5 and SW6 on Koelsch board needs to be set to
position 3 for this to work.


[...]


diff --git a/arch/arm/boot/dts/r8a7791-koelsch.dts 
b/arch/arm/boot/dts/r8a7791-koelsch.dts
index cffe33f..1bb34d0 100644
--- a/arch/arm/boot/dts/r8a7791-koelsch.dts
+++ b/arch/arm/boot/dts/r8a7791-koelsch.dts
@@ -615,7 +615,6 @@

  pci0 {
status = okay;
-   pinctrl-0 = usb0_pins;
pinctrl-names = default;


   As you're removing pinctrl-0 prop, you also should remove pinctrl-names.


  };

@@ -627,13 +626,15 @@

  hsusb {
status = okay;
-   pinctrl-0 = usb0_pins;


   Likewise.

[...]

WBR, Sergei

--
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

RE: [PATCH v2] arm: koelsch: make USB0 perform Host/Function switching

2015-07-02 Thread Phil Edworthy 
Hi Sergei.

On 02 July 2015 12:32, Sergei wrote:
 Hello.
 
 On 7/2/2015 11:14 AM, Phil Edworthy wrote:
 
  Both USB Host (pci0) and Function (USBHS) drivers are enabled.
  The USB PHY driver determines which IP block should be connected
  based on vbus and id signals read via gpios.
 
  Note that switch SW5 and SW6 on Koelsch board needs to be set to
  position 3 for this to work.
 
 [...]
 
  diff --git a/arch/arm/boot/dts/r8a7791-koelsch.dts
 b/arch/arm/boot/dts/r8a7791-koelsch.dts
  index cffe33f..1bb34d0 100644
  --- a/arch/arm/boot/dts/r8a7791-koelsch.dts
  +++ b/arch/arm/boot/dts/r8a7791-koelsch.dts
  @@ -615,7 +615,6 @@
 
pci0 {
  status = okay;
  -   pinctrl-0 = usb0_pins;
  pinctrl-names = default;
 
 As you're removing pinctrl-0 prop, you also should remove 
 pinctrl-names.
Ok.

};
 
  @@ -627,13 +626,15 @@
 
hsusb {
  status = okay;
  -   pinctrl-0 = usb0_pins;
 
 Likewise.
Sure.

 [...]
 
 WBR, Sergei

Thanks
Phil
--
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

Re: [PATCH v2] arm: koelsch: make USB0 perform Host/Function switching

2015-07-02 Thread Simon Horman 
Hi Phil,

when you re-spin this patch could you change the prefix to the following?

ARM: shmobile: koelsch:

Thanks
--
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