Re: [PATCH] ARM: multi_v7_defconfig: enable usb3503

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



2015-09-14 17:35 GMT+09:00 Riku Voipio <riku.voipio@xxxxxxxxxx>:
> On 5 June 2015 at 15:45, Arnd Bergmann <arnd@xxxxxxxx> wrote:
>> On Thursday 04 June 2015 10:47:07 Kevin Hilman wrote:
>>>
>>> > But I wonder why is not working, shouldn't the driver defer and
>>> > be probed again once the PHY driver probe succeeds?
>>>
>>> Yeah, I'm not sure why that isn't working, and didn't look into it.
>>>
>>> FWIW, the same problem happens when both are modules.  If you modprobe
>>> usb3503 first, then the phy, it doesn't work.  You have to load the phy
>>> before the usb3503.
>
>> The driver does not try to get a reference to the phy, and it does
>> not return -EPROBE_DEFER in any circumstance, so I assume it just
>> runs into an error condition on the first probe and does not
>> try again.
>
>> I don't really understand why the driver registers both an i2c_driver
>> and a platform_driver, or if that is required, but it may also
>> complicate getting deferred probing to work here.
>
> Is someone looking into fixing it?

Fixing what? The PHY issue? The driver not supporting deferred probe?

As for module vs builtin, this is somehow orthogonal for me.
Although modules are preferred on multi_v7 but in case of
boot-essential drivers this should not be a requirement. I also don't
use initrd for network boot... however my root is on MMC. Regardless
if of that I would expect nfsroot to be working on multi_v7 kernel.

>From my point of view this is the same case as USB_NET_SMSC75XX or
USB_NET_SMSC95XX, so:
1. Reviewed-by: Krzysztof Kozlowski <k.kozlowski@xxxxxxxxxxx>
2. +1 for CONFIG_PHY_SAMSUNG_USB2=y (regardless of fixing issues in the driver)
3. +1 for fixing the PHY driver


Best regards,
Krzysztof
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux