RE: [PATCH] usb: gadget: fsl_qe_udc: constify qe_ep0_desc

From: Leo Li
Date: Wed Aug 02 2017 - 14:55:06 EST




> -----Original Message-----
> From: Julia Lawall [mailto:Julia.Lawall@xxxxxxx]
> Sent: Wednesday, August 02, 2017 10:29 AM
> To: Leo Li <leoyang.li@xxxxxxx>
> Cc: kernel-janitors@xxxxxxxxxxxxxxx; Felipe Balbi <balbi@xxxxxxxxxx>; Greg
> Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>; linux-usb@xxxxxxxxxxxxxxx;
> linuxppc-dev@xxxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; Bhumika Goyal
> <bhumirks@xxxxxxxxx>
> Subject: [PATCH] usb: gadget: fsl_qe_udc: constify qe_ep0_desc
>
> qe_ep0_desc is only passed as the second argument to qe_ep_init, which is
> const, so qe_ep0_desc can be const too.
>
> Done with the help of Coccinelle.
>
> Signed-off-by: Julia Lawall <Julia.Lawall@xxxxxxx>

Acked-by: Li Yang <leoyang.li@xxxxxxx>

>
> ---
> I got a lot of warnings when compiling this file, but none seemed to be related
> to the change.
>
> drivers/usb/gadget/udc/fsl_qe_udc.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/usb/gadget/udc/fsl_qe_udc.c
> b/drivers/usb/gadget/udc/fsl_qe_udc.c
> index 303328ce..a3e72d6 100644
> --- a/drivers/usb/gadget/udc/fsl_qe_udc.c
> +++ b/drivers/usb/gadget/udc/fsl_qe_udc.c
> @@ -62,7 +62,7 @@
> "ep3",
> };
>
> -static struct usb_endpoint_descriptor qe_ep0_desc = {
> +static const struct usb_endpoint_descriptor qe_ep0_desc = {
> .bLength = USB_DT_ENDPOINT_SIZE,
> .bDescriptorType = USB_DT_ENDPOINT,
>