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


Groups > linux.kernel > #1241978

Re: [RFT 0/3] usb: usb3503: Fix probing on Arndale board (missing phy)

From Krzysztof Kozlowski <k.kozlowski@samsung.com>
Newsgroups linux.kernel
Subject Re: [RFT 0/3] usb: usb3503: Fix probing on Arndale board (missing phy)
Date 2015-10-08 08:10 +0200
Message-ID <qhcyR-3lW-15@gated-at.bofh.it> (permalink)
References <qgKVX-5zL-11@gated-at.bofh.it> <qgXTb-7qA-1@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On 07.10.2015 23:26, Marek Szyprowski wrote:
> Hello,
> 
> On 2015-10-07 02:30, Krzysztof Kozlowski wrote:
>> Introduction
>> ============
>> This patchset tries to fix probing of usb3503 on Arndale board
>> if the Samsung PHY driver is probed later (or built as a module).
>>
>> *The patchset was not tested on Arndale board.*
>> I don't have that board. Please test it and say if the usb3503
>> deferred probe
>> works fine and the issue is solved.
>>
>> The patchset was tested on Odroid U3 board (which is different!)
>> in a simulated environment. It is not sufficient testing.
>>
>>
>> Difference
>> ==========
>> The usb3503 device driver can be used as a I2C device (on Odroid U3)
>> or as a platform device connected through phy (on Arndale). In the second
>> case the necessary phy reference has to be obtained and enabled.
>>
>> For some details please look also at thread [0][1].
>>
>> [0]
>> http://lists.infradead.org/pipermail/linux-arm-kernel/2015-June/348524.html
>>
>> [1]
>> http://lists.infradead.org/pipermail/linux-arm-kernel/2015-June/348875.html
>>
>>

> 
> I'm not sure that this is the correct approach. usb3503 chip is simply
> connected
> to Exynos USB2 phy, so it visible on the USB bus. The real driver that
> controls USB2
> PHY is Exynos EHCI driver and USB3503 should not mess around it.

The ehci node (usb@12110000) has one port configured and it takes one
PHY reference (phy of id 1 - USB host). I can't see driver taking
reference to HSIC0 or HSIC1 phys... Since I cannot diagnose the error I
don't know what is really expected here.

> 
> In my opinion all that is needed in case of Arndale board is forcing
> reset of
> usb3503 chip after successful EHCI and USB2 PHY initialization (for some
> reason
> initialization of usb3503 chip must be done after usb host initialization).
> However I have no idea which driver should trigger this reset. Right now
> I didn't
> find any good solution for additional control for devices which are on
> autoprobed
> bus like usb.

The reset is done at the end of usb3503's probe. The question "why
usb3503 has to be initialized after EHCI and USB PHY" is still valid...

Anyway thanks for feedback! I really appreciate it.

Best regards,
Krzysztof

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

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


Thread

[RFT 0/3] usb: usb3503: Fix probing on Arndale board (missing phy) Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2015-10-07 02:40 +0200
  [RFT 3/3] ARM: dts: Fix usb3503 probe by enabling phy on  exynos5250-arndale Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2015-10-07 02:40 +0200
  Re: [RFT 0/3] usb: usb3503: Fix probing on Arndale board (missing phy) Marek Szyprowski <m.szyprowski@samsung.com> - 2015-10-07 16:30 +0200
    Re: [RFT 0/3] usb: usb3503: Fix probing on Arndale board (missing phy) Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2015-10-08 08:10 +0200
      Re: [RFT 0/3] usb: usb3503: Fix probing on Arndale board (missing phy) Marek Szyprowski <m.szyprowski@samsung.com> - 2015-10-08 08:30 +0200
        Re: [RFT 0/3] usb: usb3503: Fix probing on Arndale board (missing  phy) Javier Martinez Canillas <javier@osg.samsung.com> - 2015-10-08 11:40 +0200
          Re: [RFT 0/3] usb: usb3503: Fix probing on Arndale board (missing phy) Marek Szyprowski <m.szyprowski@samsung.com> - 2015-10-08 12:00 +0200
  Re: [RFT 0/3] usb: usb3503: Fix probing on Arndale board (missing phy) Kevin Hilman <khilman@kernel.org> - 2015-10-09 21:20 +0200
    Re: [RFT 0/3] usb: usb3503: Fix probing on Arndale board (missing  phy) Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2015-10-10 05:10 +0200

csiph-web