On Friday, June 05, 2015 at 06:19:16 AM, Badola Nikhil wrote:
-----Original Message----- From: Nikhil Badola [mailto:nikhil.badola@freescale.com] Sent: Tuesday, May 12, 2015 2:58 PM To: u-boot@lists.denx.de Cc: Badola Nikhil-B46172 Subject: [PATCH] drivers:usb:fsl: Return if USB_MAX_CONTROLLER_COUNT
Hi,
please do me a favor next time -- insert a space after each tag, ie. drivers: usb: fsl: ....
is incorrect
Return if USB_MAX_CONTROLLER_COUNT hence the index of the controller to be initialised is incorrect
Signed-off-by: Nikhil Badola nikhil.badola@freescale.com
drivers/usb/host/ehci-fsl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/usb/host/ehci-fsl.c b/drivers/usb/host/ehci-fsl.c index ed83eb4..1973e0b 100644 --- a/drivers/usb/host/ehci-fsl.c +++ b/drivers/usb/host/ehci-fsl.c @@ -76,7 +76,7 @@ int ehci_hcd_init(int index, enum usb_init_type init,
break;default: printf("ERROR: wrong controller index!!\n");
break;
return -1;
Can you please use value from errno.h here , like -EINVAL ?
Thanks!
Best regards, Marek Vasut