On Wed, Nov 21, 2012 at 9:04 PM, Sebastian Andrzej Siewior
<sebastian@xxxxxxxxxxxxx> wrote:
> On Wed, Nov 21, 2012 at 08:39:27PM +0800, Ming Lei wrote:
>> diff --git a/drivers/usb/musb/musb_core.c b/drivers/usb/musb/musb_core.c
>> index 774d815..ef57a60 100644
>> --- a/drivers/usb/musb/musb_core.c
>> +++ b/drivers/usb/musb/musb_core.c
>> @@ -1881,8 +1881,11 @@ musb_init_controller(struct device *dev, int nIrq, void __iomem *ctrl)
>> * isp1504, non-OTG, etc) mostly hooking up through ULPI.
>> */
>> status = musb_platform_init(musb);
>> - if (status < 0)
>> + if (status < 0) {
>> + /* try to defer probe if trasceiver is not ready */
>> + status = -EPROBE_DEFER;
>
> Hmm. You basically turn every possible error into EPROBE_DEFER. What about
> replacing the ENODEV with EPROBE_DEFER? I would make a little larger patch but
> -ENOMEM or -EINVAL remains -ENOMEM or -EINVAL.
Hmm, I should have returned -EPROBE_DEFER just for -ENODEV.
Will do it in v1.
Thanks,
--
Ming Lei
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
