Hi Rob,
> From: Rob Herring <r...@kernel.org>
>
> Allow a NULL table to be passed to usb_gadget_get_string for cases
> when a string table may not be populated.
I might be wrong, since I'm not the native speaker, but this
description is a bit misleading.
For me this patch is supposed to prevent from using uninitialized string
table in this function.
>
> Signed-off-by: Rob Herring <r...@kernel.org>
> ---
> drivers/usb/gadget/usbstring.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/drivers/usb/gadget/usbstring.c
> b/drivers/usb/gadget/usbstring.c index de5fa3f..8c3ff64 100644
> --- a/drivers/usb/gadget/usbstring.c
> +++ b/drivers/usb/gadget/usbstring.c
> @@ -108,6 +108,9 @@ usb_gadget_get_string(struct usb_gadget_strings
> *table, int id, u8 *buf) struct usb_string *s;
> int len;
>
> + if (!table)
> + return -EINVAL;
> +
> /* descriptor 0 has the language id */
> if (id == 0) {
> buf[0] = 4;
Despite the problem with parsing commit message :-)
Acked-by: Lukasz Majewski <l.majew...@samsung.com>
--
Best regards,
Lukasz Majewski
Samsung R&D Institute Poland (SRPOL) | Linux Platform Group
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
