From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lothar =?UTF-8?B?V2HDn21hbm4=?= Date: Fri, 1 Sep 2017 08:30:49 +0200 Subject: [U-Boot] [UBOOT] [PATCH] cmd: usb: ignore block devices under mass storage device In-Reply-To: <1502430811-21475-1-git-send-email-suneelglinux@gmail.com> References: <1502430811-21475-1-git-send-email-suneelglinux@gmail.com> Message-ID: <20170901083049.32ed8295@karo-electronics.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: quoted-printable To: u-boot@lists.denx.de Hi, On Thu, 10 Aug 2017 22:53:31 -0700 Suneel Garapati wrote: > usb tree and info commands may cause crash otherwise >=20 > Signed-off-by: Suneel Garapati > --- > cmd/usb.c | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) >=20 > diff --git a/cmd/usb.c b/cmd/usb.c > index 992d414..81e1a7b 100644 > --- a/cmd/usb.c > +++ b/cmd/usb.c > @@ -415,7 +415,8 @@ static void usb_show_tree_graph(struct usb_device *de= v, char *pre) > udev =3D dev_get_parent_priv(child); > =20 > /* Ignore emulators, we only want real devices */ > - if (device_get_uclass_id(child) !=3D UCLASS_USB_EMUL) { > + if (device_get_uclass_id(child) !=3D > + (UCLASS_USB_EMUL | UCLASS_BLK)) { > This should most probably be: > + if ((device_get_uclass_id(child) !=3D UCLASS_USB_EMUL) && (device_get_uclass_id(child) !=3D UCLASS_BLK)) { Lothar Wa=C3=9Fmann