Re: [U-Boot] [PATCH] cmd: usb: check if_type before using this device

18 Jun 2016


      On 17 June 2016 at 00:18, Peng Fan van.freenix@gmail.com wrote:
...
For legacy usb storage driver, USB_MAX_STOR_DEV is defined as 7.
If we only have one usb disk on board, `usb dev 0` is ok.
But if `usb dev 1`, still ok, then `usb read xxx` will trigger
system fault and reboot.
So check if_type before using this device.
Signed-off-by: Peng Fan peng.fan@nxp.com
Cc: Simon Glass sjg@chromium.org
Cc: Hans de Goede hdegoede@redhat.com
Cc: Bin Meng bmeng.cn@gmail.com
Cc: Marek Vasut marex@denx.de
Cc: Stefan Roese sr@denx.de
Cc: Marcel Ziswiler marcel.ziswiler@toradex.com
Cc: Peng Fan peng.fan@nxp.com
Cc: Stephen Warren swarren@nvidia.com

cmd/usb.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
Reviewed-by: Simon Glass sjg@chromium.org