Re: [PATCH] ALSA: usb-audio: constify snd_kcontrol_new structures

[Takashi Iwai]

On Tue, 21 Feb 2017 21:03:27 +0100,
Bhumika Goyal wrote:
> 
> Declare snd_kcontrol_new structures as const as they are only passed as
> an argument to the function add_new_ctl. This agrument is of type const,
> so snd_kcontrol_new structures having this property can be made const
> too.
> 
> Signed-off-by: Bhumika Goyal <bhumirks@xxxxxxxxx>

Well, this is again no urgent fix, so I'll postpone this to 4.12.

In general, please try to stop posting this kind of patches during the
merge window.  During the merge window, we concentrate only on the
stuff for the upcoming rc1, and receiving repeatedly non-urgent
cleanup patches is often just annoyance.

Of course, once after the merge window is closed, it's perfectly fine
to take such patches.


thanks,

Takashi


> ---
>  sound/usb/mixer_scarlett.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/sound/usb/mixer_scarlett.c b/sound/usb/mixer_scarlett.c
> index 7438e7c..c33e237 100644
> --- a/sound/usb/mixer_scarlett.c
> +++ b/sound/usb/mixer_scarlett.c
> @@ -477,7 +477,7 @@ static int scarlett_ctl_meter_get(struct snd_kcontrol *kctl,
>  	return 0;
>  }
>  
> -static struct snd_kcontrol_new usb_scarlett_ctl_switch = {
> +static const struct snd_kcontrol_new usb_scarlett_ctl_switch = {
>  	.iface = SNDRV_CTL_ELEM_IFACE_MIXER,
>  	.name = "",
>  	.info = scarlett_ctl_switch_info,
> @@ -487,7 +487,7 @@ static int scarlett_ctl_meter_get(struct snd_kcontrol *kctl,
>  
>  static const DECLARE_TLV_DB_SCALE(db_scale_scarlett_gain, -12800, 100, 0);
>  
> -static struct snd_kcontrol_new usb_scarlett_ctl = {
> +static const struct snd_kcontrol_new usb_scarlett_ctl = {
>  	.iface = SNDRV_CTL_ELEM_IFACE_MIXER,
>  	.access = SNDRV_CTL_ELEM_ACCESS_READWRITE |
>  		  SNDRV_CTL_ELEM_ACCESS_TLV_READ,
> @@ -499,7 +499,7 @@ static int scarlett_ctl_meter_get(struct snd_kcontrol *kctl,
>  	.tlv = { .p = db_scale_scarlett_gain }
>  };
>  
> -static struct snd_kcontrol_new usb_scarlett_ctl_master = {
> +static const struct snd_kcontrol_new usb_scarlett_ctl_master = {
>  	.iface = SNDRV_CTL_ELEM_IFACE_MIXER,
>  	.access = SNDRV_CTL_ELEM_ACCESS_READWRITE |
>  		  SNDRV_CTL_ELEM_ACCESS_TLV_READ,
> @@ -511,7 +511,7 @@ static int scarlett_ctl_meter_get(struct snd_kcontrol *kctl,
>  	.tlv = { .p = db_scale_scarlett_gain }
>  };
>  
> -static struct snd_kcontrol_new usb_scarlett_ctl_enum = {
> +static const struct snd_kcontrol_new usb_scarlett_ctl_enum = {
>  	.iface = SNDRV_CTL_ELEM_IFACE_MIXER,
>  	.name = "",
>  	.info = scarlett_ctl_enum_info,
> @@ -519,7 +519,7 @@ static int scarlett_ctl_meter_get(struct snd_kcontrol *kctl,
>  	.put =  scarlett_ctl_enum_put,
>  };
>  
> -static struct snd_kcontrol_new usb_scarlett_ctl_dynamic_enum = {
> +static const struct snd_kcontrol_new usb_scarlett_ctl_dynamic_enum = {
>  	.iface = SNDRV_CTL_ELEM_IFACE_MIXER,
>  	.name = "",
>  	.info = scarlett_ctl_enum_dynamic_info,
> @@ -527,7 +527,7 @@ static int scarlett_ctl_meter_get(struct snd_kcontrol *kctl,
>  	.put =  scarlett_ctl_enum_put,
>  };
>  
> -static struct snd_kcontrol_new usb_scarlett_ctl_sync = {
> +static const struct snd_kcontrol_new usb_scarlett_ctl_sync = {
>  	.iface = SNDRV_CTL_ELEM_IFACE_MIXER,
>  	.access = SNDRV_CTL_ELEM_ACCESS_READ | SNDRV_CTL_ELEM_ACCESS_VOLATILE,
>  	.name = "",
> -- 
> 1.9.1
> 
>