[PATCH v2 1/1] usb: storage: continue probe on "Invalid device"

Marek Vasut marex at denx.de
Thu Nov 3 23:38:16 CET 2022


On 11/3/22 23:23, Marek Vasut wrote:
> On 11/3/22 22:36, Janne Grunau wrote:
>> On 2022-09-28 04:20:52 -0600, Simon Glass wrote:
>>> +Marek Vasut
>>> +Tom Rini
>>>
>>> On Sun, 25 Sept 2022 at 23:07, Janne Grunau <j at jannau.net> wrote:
>>>>
>>>> On 2022-08-10 21:54:22 +0200, Janne Grunau wrote:
>>>>> Fixes a crash during probing of sd card readers without medium 
>>>>> present.
>>>>>
>>>>> Link: https://github.com/AsahiLinux/linux/issues/44
>>>>> Link: https://lists.denx.de/pipermail/u-boot/2022-July/489717.html
>>>>> Signed-off-by: Janne Grunau <j at jannau.net>
>>>>> ---
>>>>> Changes since v1:
>>>>>   - changed unconditiona return to "continue" as proposed by AKASHI 
>>>>> Takahiro
>>>>>
>>>>>   common/usb_storage.c | 1 +
>>>>>   1 file changed, 1 insertion(+)
>>>>>
>>>>> diff --git a/common/usb_storage.c b/common/usb_storage.c
>>>>> index eaa31374ef73..f9204552a683 100644
>>>>> --- a/common/usb_storage.c
>>>>> +++ b/common/usb_storage.c
>>>>> @@ -239,6 +239,7 @@ static int usb_stor_probe_device(struct 
>>>>> usb_device *udev)
>>>>>                        ret = device_unbind(dev);
>>>>>                        if (ret)
>>>>>                                return ret;
>>>>> +                     continue;
>>>>>                }
>>>>>
>>>>>                ret = blk_probe_or_unbind(dev);
>>>>
>>>> ping. Is there anything holding up merging this fix?
>>
>> ping2
>>
>> This fixes a 100% reproducible crash when an USB storage device with
>> "medium not ready" is connected.
> 
> Can you please CC me next time when submitting these kinds of USB fixes ?
> 
> Also, can you tell which device this is ?

Reading through the rest of the links, can you please just fill in 
details of the USB card reader for future reference (like lsusb -vvvn or 
similar). And then send V3 with my:

Reviewed-by: Marek Vasut <marex at denx.de>


More information about the U-Boot mailing list