On Wed, Aug 09, 2023 at 04:53:48PM +0800, Yang Yingliang wrote: > The resource is checked in probe function, so there is > no need do this check in remove function. > > Signed-off-by: Yang Yingliang <yangyingliang@xxxxxxxxxx> > --- > drivers/usb/host/ohci-sm501.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/usb/host/ohci-sm501.c b/drivers/usb/host/ohci-sm501.c > index 0468eeb4fcfd..4b39e9d6f33a 100644 > --- a/drivers/usb/host/ohci-sm501.c > +++ b/drivers/usb/host/ohci-sm501.c > @@ -195,8 +195,7 @@ static void ohci_hcd_sm501_drv_remove(struct platform_device *pdev) > release_mem_region(hcd->rsrc_start, hcd->rsrc_len); > usb_put_hcd(hcd); > mem = platform_get_resource(pdev, IORESOURCE_MEM, 1); > - if (mem) > - release_mem_region(mem->start, resource_size(mem)); > + release_mem_region(mem->start, resource_size(mem)); > > /* mask interrupts and disable power */ Acked-by: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx>
