[PATCH 1/4] usb: phy: Fix deferred probing
Olof Johansson
olof at lixom.net
Tue Jan 6 20:41:34 PST 2015
Hi,
On Tue, Jan 6, 2015 at 7:45 AM, Maxime Ripard
<maxime.ripard at free-electrons.com> wrote:
> Commit 1290a958d48e ("usb: phy: propagate __of_usb_find_phy()'s error on
> failure") actually broke the deferred probing mechanism, since it now returns
> EPROBE_DEFER only when the try_module_get call fails, but not when the phy
> lookup does.
>
> All the other similar functions seem to return ENODEV when try_module_get
> fails, and the error code of either __usb_find_phy or __of_usb_find_phy
> otherwise.
>
> In order to have a consistent behaviour, and a meaningful EPROBE_DEFER, always
> return EPROBE_DEFER when __(of_)usb_find_phy fails to look up the requested
> phy, that will be propagated by the caller, and ENODEV if try_module_get fails.
>
Fixes: 1290a958d48e ("usb: phy: propagate __of_usb_find_phy()'s error
on failure")
> Signed-off-by: Maxime Ripard <maxime.ripard at free-electrons.com>
This also fixes USB on tegra/trimslice here.
Tested-by: Olof Johansson <olof at lixom.net>
Given that the regression went in during 3.19 merge window, this
should go in as a fix during -rc, please.
-Olof
More information about the linux-arm-kernel
mailing list