]> www.infradead.org Git - users/jedix/linux-maple.git/commitdiff
ASoC: wm_adsp: Remove unused control callback structure
authorCharles Keepax <ckeepax@opensource.cirrus.com>
Thu, 11 Feb 2021 17:21:06 +0000 (17:21 +0000)
committerMark Brown <broonie@kernel.org>
Thu, 11 Feb 2021 17:31:57 +0000 (17:31 +0000)
This callback structure has never been used and it is not clear why it
was added in the first place. Remove it to clear up the code a little.

Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Link: https://lore.kernel.org/r/20210211172106.16258-1-ckeepax@opensource.cirrus.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/wm_adsp.c

index 262c2db4f81ced6f691ab705e73fc1958ec22790..070ca7d8c661003dec4dba893ef21a40e945c58e 100644 (file)
@@ -595,13 +595,6 @@ static const struct {
        [WM_ADSP_FW_MISC] =     { .file = "misc" },
 };
 
-struct wm_coeff_ctl_ops {
-       int (*xget)(struct snd_kcontrol *kcontrol,
-                   struct snd_ctl_elem_value *ucontrol);
-       int (*xput)(struct snd_kcontrol *kcontrol,
-                   struct snd_ctl_elem_value *ucontrol);
-};
-
 struct wm_coeff_ctl {
        const char *name;
        const char *fw_name;
@@ -609,7 +602,6 @@ struct wm_coeff_ctl {
        const char *subname;
        unsigned int subname_len;
        struct wm_adsp_alg_region alg_region;
-       struct wm_coeff_ctl_ops ops;
        struct wm_adsp *dsp;
        unsigned int enabled:1;
        struct list_head list;
@@ -1497,8 +1489,6 @@ static int wm_adsp_create_control(struct wm_adsp *dsp,
        }
        ctl->enabled = 1;
        ctl->set = 0;
-       ctl->ops.xget = wm_coeff_get;
-       ctl->ops.xput = wm_coeff_put;
        ctl->dsp = dsp;
 
        ctl->flags = flags;