Re: [PATCH] usb: phy: never defer probe in non-OF case
On Fri, Jan 16, 2015 at 04:10:31PM +0300, Sergei Shtylyov wrote: > Hello. > > On 1/16/2015 11:30 AM, Heikki Krogerus wrote: > > >In practice failure to find phy when requested in non-OF > >case means it will never become available, so > >__usb_find_phy() must return -ENODEV and not -EPROBE_DEFER. > > >This fixes a regression caused by commit > >9c9d82492b73991e8e13a6c0af06e44816c31438, where the USB > >Please also specify that commit's summary in parens. I'll fix when applying. -- balbi signature.asc Description: Digital signature
Re: [PATCH] usb: phy: never defer probe in non-OF case
Hello. On 1/16/2015 11:30 AM, Heikki Krogerus wrote: In practice failure to find phy when requested in non-OF case means it will never become available, so __usb_find_phy() must return -ENODEV and not -EPROBE_DEFER. This fixes a regression caused by commit 9c9d82492b73991e8e13a6c0af06e44816c31438, where the USB Please also specify that commit's summary in parens. controller driver is left infinitely into deferred probe when there are no phys. Signed-off-by: Heikki Krogerus [...] WBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH] usb: phy: never defer probe in non-OF case
In practice failure to find phy when requested in non-OF case means it will never become available, so __usb_find_phy() must return -ENODEV and not -EPROBE_DEFER. This fixes a regression caused by commit 9c9d82492b73991e8e13a6c0af06e44816c31438, where the USB controller driver is left infinitely into deferred probe when there are no phys. Signed-off-by: Heikki Krogerus --- drivers/usb/phy/phy.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/phy/phy.c b/drivers/usb/phy/phy.c index ccfdfb2..2f9735b 100644 --- a/drivers/usb/phy/phy.c +++ b/drivers/usb/phy/phy.c @@ -34,7 +34,7 @@ static struct usb_phy *__usb_find_phy(struct list_head *list, return phy; } - return ERR_PTR(-EPROBE_DEFER); + return ERR_PTR(-ENODEV); } static struct usb_phy *__usb_find_phy_dev(struct device *dev, -- 2.1.4 -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html