Re: [PATCH] mmc: vub300: constify usb_device_id

From: Ulf Hansson
Date: Mon Aug 21 2017 - 08:33:23 EST


On 9 August 2017 at 19:50, Arvind Yadav <arvind.yadav.cs@xxxxxxxxx> wrote:
> usb_device_id are not supposed to change at runtime. All functions
> working with usb_device_id provided by <linux/usb.h> work with
> const usb_device_id. So mark the non-const structs as const.
>
> Signed-off-by: Arvind Yadav <arvind.yadav.cs@xxxxxxxxx>

Thanks, applied for next!

Kind regards
Uffe

> ---
> drivers/mmc/host/vub300.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/mmc/host/vub300.c b/drivers/mmc/host/vub300.c
> index fbeea1a..6c99102 100644
> --- a/drivers/mmc/host/vub300.c
> +++ b/drivers/mmc/host/vub300.c
> @@ -266,7 +266,7 @@ MODULE_PARM_DESC(firmware_rom_wait_states,
> #define ELAN_VENDOR_ID 0x2201
> #define VUB300_VENDOR_ID 0x0424
> #define VUB300_PRODUCT_ID 0x012C
> -static struct usb_device_id vub300_table[] = {
> +static const struct usb_device_id vub300_table[] = {
> {USB_DEVICE(ELAN_VENDOR_ID, VUB300_PRODUCT_ID)},
> {USB_DEVICE(VUB300_VENDOR_ID, VUB300_PRODUCT_ID)},
> {} /* Terminating entry */
> --
> 2.7.4
>