On Wed, 6 May 2020, Wei Yongjun wrote:
> Fix to return a negative error code from the error handling
> case instead of 0, as done elsewhere in this function.
>
> Fixes: 7d9e6f5aebe8 ("usb: host: ohci-sm501: init genalloc for local memory")
> Signed-off-by: Wei Yongjun <weiyongjun1@xxxxxxxxxx>
> ---
> drivers/usb/host/ohci-sm501.c | 7 ++++---
> 1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/usb/host/ohci-sm501.c b/drivers/usb/host/ohci-sm501.c
> index c158cda9e4b9..cff965240327 100644
> --- a/drivers/usb/host/ohci-sm501.c
> +++ b/drivers/usb/host/ohci-sm501.c
> @@ -157,9 +157,10 @@ static int ohci_hcd_sm501_drv_probe(struct platform_device *pdev)
> * the call to usb_hcd_setup_local_mem() below does just that.
> */
>
> - if (usb_hcd_setup_local_mem(hcd, mem->start,
> - mem->start - mem->parent->start,
> - resource_size(mem)) < 0)
> + retval = usb_hcd_setup_local_mem(hcd, mem->start,
> + mem->start - mem->parent->start,
> + resource_size(mem));
> + if (retval < 0)
> goto err5;
> retval = usb_add_hcd(hcd, irq, IRQF_SHARED);
> if (retval)
Acked-by: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx>
Thanks for noticing and fixing this.
Alan Stern
