AW: [PATCH -next] USB: ohci-sm501: fix error return code in ohci_hcd_sm501_drv_probe()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



while you are here ...
maybe you can give labels like err5 some meaningful names ?

re,
 wh
________________________________________
Von: kernel-janitors-owner@xxxxxxxxxxxxxxx <kernel-janitors-owner@xxxxxxxxxxxxxxx> im Auftrag von Alan Stern <stern@xxxxxxxxxxxxxxxxxxx>
Gesendet: Mittwoch, 6. Mai 2020 16:12:08
An: Wei Yongjun
Cc: Greg Kroah-Hartman; Laurentiu Tudor; linux-usb@xxxxxxxxxxxxxxx; kernel-janitors@xxxxxxxxxxxxxxx
Betreff: Re: [PATCH -next] USB: ohci-sm501: fix error return code in ohci_hcd_sm501_drv_probe()

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





[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux