From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Tue, 01 Mar 2016 00:05:08 +0100 Subject: [U-Boot] [PATCH v2 26/32] dm: usb: Unbind old block devices when shutting down USB In-Reply-To: <1456784765-10788-27-git-send-email-sjg@chromium.org> References: <1456784765-10788-2-git-send-email-sjg@chromium.org> <1456784765-10788-27-git-send-email-sjg@chromium.org> Message-ID: <56D4CEA4.5020203@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 02/29/2016 11:25 PM, Simon Glass wrote: > We don't want old block devices hanging around since they can still appear > visible in some way. OK > Possibly we should unbind all devices which don't cause problems by being > unbound. Most likely we can remove everything except USB controllers, hubs > and emulators. We can consider that later. Can you please expand on this bit of the commit message ? It's not very clear. Thanks! > Signed-off-by: Simon Glass > --- > > Changes in v2: None > > drivers/usb/host/usb-uclass.c | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/drivers/usb/host/usb-uclass.c b/drivers/usb/host/usb-uclass.c > index 50538e0..69c9a50 100644 > --- a/drivers/usb/host/usb-uclass.c > +++ b/drivers/usb/host/usb-uclass.c > @@ -159,7 +159,11 @@ int usb_stop(void) > if (ret && !err) > err = ret; > } > - > +#ifdef CONFIG_BLK > + ret = blk_unbind_all(IF_TYPE_USB); > + if (ret && !err) > + err = ret; > +#endif > #ifdef CONFIG_SANDBOX > struct udevice *dev; > > -- Best regards, Marek Vasut