Skip to content

Commit 312c04c

Browse files
Stefan Bindingtiwai
authored andcommitted
ALSA: hda: cs35l41: Stop creating ALSA Controls for firmware coefficients
When the CS35L41 loads its firmware, it has a number of controls to affect its behaviour. Currently, these controls are exposed as ALSA Controls. These controls were never intended to be exposed to users but the firmware doesn't mark them hidden, so make the driver ignore them. Any changes in the coefficients handled by these controls needs to be matched to the individual system by SSID, which is already handled using the tuning file, when firmware is loaded, so UCM should not be setting these controls anyway. Signed-off-by: Stefan Binding <[email protected]> Link: https://patch.msgid.link/[email protected] Signed-off-by: Takashi Iwai <[email protected]>
1 parent 34e1b1b commit 312c04c

File tree

1 file changed

+0
-13
lines changed

1 file changed

+0
-13
lines changed

sound/pci/hda/cs35l41_hda.c

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -133,17 +133,6 @@ static const struct reg_sequence cs35l41_hda_mute[] = {
133133
{ CS35L41_AMP_DIG_VOL_CTRL, 0x0000A678 }, // AMP_HPF_PCM_EN = 1, AMP_VOL_PCM Mute
134134
};
135135

136-
static void cs35l41_add_controls(struct cs35l41_hda *cs35l41)
137-
{
138-
struct hda_cs_dsp_ctl_info info;
139-
140-
info.device_name = cs35l41->amp_name;
141-
info.fw_type = cs35l41->firmware_type;
142-
info.card = cs35l41->codec->card;
143-
144-
hda_cs_dsp_add_controls(&cs35l41->cs_dsp, &info);
145-
}
146-
147136
static const struct cs_dsp_client_ops client_ops = {
148137
.control_remove = hda_cs_dsp_control_remove,
149138
};
@@ -603,8 +592,6 @@ static int cs35l41_init_dsp(struct cs35l41_hda *cs35l41)
603592
if (ret)
604593
goto err;
605594

606-
cs35l41_add_controls(cs35l41);
607-
608595
cs35l41_hda_apply_calibration(cs35l41);
609596

610597
err:

0 commit comments

Comments
 (0)