UPSTREAM: usb: gadget: f_uac2: constify snd_pcm_ops structures

ArvindYadavCs · numbqq · commit cd0305d471c0 · 2019-08-26T16:36:29.000+08:00
snd_pcm_ops are not supposed to change at runtime. All functions working with snd_pcm_ops provided by <sound/pcm.h> work with const snd_pcm_ops. So mark the non-const structs as const. Change-Id: I0efbb6c4563dbced2154008914d0c7f6fbb8f134 Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com> Signed-off-by: Felipe Balbi <felipe.balbi@linux.intel.com> Signed-off-by: William Wu <william.wu@rock-chips.com> (cherry picked from commit b765ff1)
diff --git a/drivers/usb/gadget/function/u_audio.c b/drivers/usb/gadget/function/u_audio.c
@@ -354,7 +354,7 @@ static int uac_pcm_null(struct snd_pcm_substream *substream)
 	return 0;
 }
 
-static struct snd_pcm_ops uac_pcm_ops = {
+static const struct snd_pcm_ops uac_pcm_ops = {
 	.open = uac_pcm_open,
 	.close = uac_pcm_null,
 	.ioctl = snd_pcm_lib_ioctl,

Original file line number	Diff line number	Diff line change
`@@ -354,7 +354,7 @@ static int uac_pcm_null(struct snd_pcm_substream *substream)`
`354`	`354`	`return 0;`
`355`	`355`	`}`
`356`	`356`
`357`		`-static struct snd_pcm_ops uac_pcm_ops = {`
	`357`	`+static const struct snd_pcm_ops uac_pcm_ops = {`
`358`	`358`	`.open = uac_pcm_open,`
`359`	`359`	`.close = uac_pcm_null,`
`360`	`360`	`.ioctl = snd_pcm_lib_ioctl,`