Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1514730

Re: [PATCH v3 3/5] dt/bindings: Add a new property to DA8xx USB PHY

Show all headers | View raw

On 11/03/2016 05:53 PM, David Lechner wrote:
> On 11/03/2016 10:26 AM, Alexandre Bailon wrote:
>> The USB PHY is able to operate in OTG, host or peripheral.
>> Some board may be wired to work act only as host or peripheral.
>> In such case, the dr_mode property of controller must be set to
>> host or peripheral. But doing that will also configure the PHY
>> in host or peripheral mode whereas OTG is able to detect which
>> role the USB controller should take.
>> The PHY's host or peripheral mode are actually only useful when
>> hardware doesn't allow OTG to detect it's role.
>>
>> Add the usb20_force_mode property to force the PHY to operate
>> in host or peripheral mode.
> 
> Device tree describes the hardware, not the configuration, so this is
> not acceptable.
I think that is really hardware (and board) dependent.
We will only need to set it for unusual hardware that doesn't
let the otg phy automatically find the it's role.
What do you think I should do?
> 
> Besides, this setting should not be fixed to one value anyway.
Actually, a bool is enough. If we need to force the phy in a specific
mode, we can reuse dr_mode of controller.
> 
>> When usb20_force_mode is used, dr_mode should also be configured
>> to host or peripheral.
>> The controller uses dr_mode to configure itself, but the phy use
>> it to get the mode to use to configure the PHY mode.
>>
>> Signed-off-by: Alexandre Bailon <abailon@baylibre.com>
>> ---
>>  Documentation/devicetree/bindings/phy/phy-da8xx-usb.txt | 5 +++++
>>  1 file changed, 5 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/phy/phy-da8xx-usb.txt
>> b/Documentation/devicetree/bindings/phy/phy-da8xx-usb.txt
>> index c26478b..9fc87fb 100644
>> --- a/Documentation/devicetree/bindings/phy/phy-da8xx-usb.txt
>> +++ b/Documentation/devicetree/bindings/phy/phy-da8xx-usb.txt
>> @@ -4,6 +4,11 @@ Required properties:
>>   - compatible: must be "ti,da830-usb-phy".
>>   - #phy-cells: must be 1.
>>
>> +Optional properties:
>> +- usb20-force-mode: Force the phy to operate in same mode than the
>> USB OTG controller.
>> +            It should only be defined if the hardware is not capable
>> correctly
>> +            detect the role of USB by using VBUS and ID pin.
>> +
>>  This device controls the PHY for both the USB 1.1 OHCI and USB 2.0 OTG
>>  controllers on DA8xx SoCs. Consumers of this device should use index
>> 0 for
>>  the USB 2.0 phy device and index 1 for the USB 1.1 phy device.
>>
> 

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v3 0/5] usb: musb: da8xx: Fix few issues Alexandre Bailon <abailon@baylibre.com> - 2016-11-03 16:30 +0100
  [PATCH v3 5/5] usb: musb: da8xx: Remove set_mode callback Alexandre Bailon <abailon@baylibre.com> - 2016-11-03 16:30 +0100
    Re: [PATCH v3 5/5] usb: musb: da8xx: Remove set_mode callback David Lechner <david@lechnology.com> - 2016-11-03 18:20 +0100
      Re: [PATCH v3 5/5] usb: musb: da8xx: Remove set_mode callback Bin Liu <b-liu@ti.com> - 2016-11-03 18:30 +0100
        Re: [PATCH v3 5/5] usb: musb: da8xx: Remove set_mode callback Alexandre Bailon <abailon@baylibre.com> - 2016-11-03 18:50 +0100
  [PATCH v3 1/5] usb: musb: da8xx: Call earlier clk_prepare_enable() Alexandre Bailon <abailon@baylibre.com> - 2016-11-03 16:30 +0100
  [PATCH v3 2/5] phy: da8xx-usb: Configure CFGCHIP2 to support OTG workaround Alexandre Bailon <abailon@baylibre.com> - 2016-11-03 16:30 +0100
    Re: [PATCH v3 2/5] phy: da8xx-usb: Configure CFGCHIP2 to support OTG  workaround David Lechner <david@lechnology.com> - 2016-11-03 18:10 +0100
  [PATCH v3 3/5] dt/bindings: Add a new property to DA8xx USB PHY Alexandre Bailon <abailon@baylibre.com> - 2016-11-03 16:30 +0100
    Re: [PATCH v3 3/5] dt/bindings: Add a new property to DA8xx USB PHY Kishon Vijay Abraham I <kishon@ti.com> - 2016-11-03 17:40 +0100
      Re: [PATCH v3 3/5] dt/bindings: Add a new property to DA8xx USB PHY Alexandre Bailon <abailon@baylibre.com> - 2016-11-03 18:30 +0100
        Re: [PATCH v3 3/5] dt/bindings: Add a new property to DA8xx USB PHY Kishon Vijay Abraham I <kishon@ti.com> - 2016-11-03 19:00 +0100
          Re: [PATCH v3 3/5] dt/bindings: Add a new property to DA8xx USB PHY Alexandre Bailon <abailon@baylibre.com> - 2016-11-04 12:10 +0100
    Re: [PATCH v3 3/5] dt/bindings: Add a new property to DA8xx USB PHY David Lechner <david@lechnology.com> - 2016-11-03 18:00 +0100
      Re: [PATCH v3 3/5] dt/bindings: Add a new property to DA8xx USB PHY Alexandre Bailon <abailon@baylibre.com> - 2016-11-03 18:40 +0100
        Re: [PATCH v3 3/5] dt/bindings: Add a new property to DA8xx USB PHY David Lechner <david@lechnology.com> - 2016-11-03 19:00 +0100
  [PATCH v3 4/5] phy: da8xx-usb: Use usb20_force_mode property to configure the phy mode Alexandre Bailon <abailon@baylibre.com> - 2016-11-03 16:30 +0100

csiph-web