lkml.org 
[lkml]   [2014]   [Nov]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 3/3] ARM: dts: vf-colibri: add USB regulators
On 2014-11-11 15:12, Shawn Guo wrote:
> On Tue, Nov 04, 2014 at 02:07:10PM +0100, Stefan Agner wrote:
>> Add structure of USB supply logic. Especially, the USB hosts power
>> enable regulator is needed to control VBUS supply.
>>
>> Signed-off-by: Stefan Agner <stefan@agner.ch>
>> ---
>> arch/arm/boot/dts/vf-colibri-eval-v3.dtsi | 45 ++++++++++++++++++++++++++++++-
>> arch/arm/boot/dts/vf-colibri.dtsi | 12 +++++++++
>> 2 files changed, 56 insertions(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/boot/dts/vf-colibri-eval-v3.dtsi b/arch/arm/boot/dts/vf-colibri-eval-v3.dtsi
>> index 80e8fbc..eb4ca50 100644
>> --- a/arch/arm/boot/dts/vf-colibri-eval-v3.dtsi
>> +++ b/arch/arm/boot/dts/vf-colibri-eval-v3.dtsi
>> @@ -11,6 +11,41 @@
>> chosen {
>> bootargs = "console=ttyLP0,115200";
>> };
>> +
>> + regulators {
>> + compatible = "simple-bus";
>> + #address-cells = <1>;
>> + #size-cells = <0>;
>> +
>> + sys_5v0_reg: regulator@1 {
>> + compatible = "regulator-fixed";
>> + reg = <1>;
>
> The numbering generally starts from 0 instead of 1.
>
>> + regulator-name = "5v0";
>> + regulator-min-microvolt = <5000000>;
>> + regulator-max-microvolt = <5000000>;
>> + regulator-always-on;
>> + };
>> +
>> + usbc_vbus_reg: regulator@2 {
>> + compatible = "regulator-fixed";
>> + reg = <2>;
>> + regulator-name = "usbc_vbus";
>> + regulator-min-microvolt = <5000000>;
>> + regulator-max-microvolt = <5000000>;
>> + vin-supply = <&sys_5v0_reg>;
>> + };
>> +
>> + /* USBH_PEN */
>> + usbh_vbus_reg: regulator@3 {
>> + compatible = "regulator-fixed";
>> + reg = <3>;
>> + regulator-name = "usbh_vbus";
>> + regulator-min-microvolt = <5000000>;
>> + regulator-max-microvolt = <5000000>;
>> + gpio = <&gpio3 19 GPIO_ACTIVE_LOW>;
>> + vin-supply = <&sys_5v0_reg>;
>> + };
>> + };
>> };
>>
>> &bl {
>> @@ -51,4 +86,12 @@
>>
>> &uart2 {
>> status = "okay";
>> -};
>> \ No newline at end of file
>> +};
>> +
>> +&usbdev0 {
>> + vbus-supply = <&usbc_vbus_reg>;
>> +};
>> +
>> +&usbh1 {
>> + vbus-supply = <&usbh_vbus_reg>;
>> +};
>> diff --git a/arch/arm/boot/dts/vf-colibri.dtsi b/arch/arm/boot/dts/vf-colibri.dtsi
>> index 2371760..79ccfe2 100644
>> --- a/arch/arm/boot/dts/vf-colibri.dtsi
>> +++ b/arch/arm/boot/dts/vf-colibri.dtsi
>> @@ -163,5 +163,17 @@
>> VF610_PAD_PTD3__UART2_CTS 0x21a1
>> >;
>> };
>> +
>> + pinctrl_usbh1_ext: gpio_usb_vbus {
>> + fsl,pins = <
>> + VF610_PAD_PTD4__GPIO_83 0x22ed
>
> We used to know the GPIO usage by looking for the phandle reference to
> the node. But it doesn't work for VF610 case now.
>
> I guess this is for usbh_vbus_reg usage? It would be nice to have a
> comment telling the usage of the GPIO.

Yeah its true, its assign is not required for Vybird. However, we could
assign them anyway, it would even enforce correct pin setup from
Startup... This would then make a comment superfluous. What do you
think?


>
>> + >;
>> + };
>> +
>> + pinctrl_usbdev0_ext: gpio_usb_cdet {
>> + fsl,pins = <
>> + VF610_PAD_PTC29__GPIO_102 0x22ed
>
> How is this one used?

This is not yet used. It is more or less the OTG-ID pin, but on 3.3V
level. We use that on the Colibri standard to switch between Host/Device
mode, however a driver which uses that GPIO is still outstanding. I can
also leave that away for now, if you prefer....

--
Stefan

>
> Shawn
>
>> + >;
>> + };
>> };
>> };
>> --
>> 2.1.3
>>



\
 
 \ /
  Last update: 2014-11-12 17:21    [W:0.054 / U:1.144 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site