Skip to content

Commit e82bc51

Browse files
rfvirgilbroonie
authored andcommitted
ALSA: hda: cs35l56: Fix filename string field layout
Change the filename field layout to: cs35l56-rev[-s]-dsp1-misc[-sub].[wmfw|bin] This is to keep the same firmware file naming scheme as the CS35L56 ASoC driver. This is not a compatibility break because no firmware files have been published. The original field layout matched the ASoC driver, but the way the ASoC driver used the wm_adsp driver config to form this filename was bugged. Fixing the ASoC driver to use the correct wm_adsp config strings means that the 's' flag (to indicate a secured part) has to move to somewhere after the first '-'. Signed-off-by: Richard Fitzgerald <rf@opensource.cirrus.com> Fixes: 73cfbfa ("ALSA: hda/cs35l56: Add driver for Cirrus Logic CS35L56 amplifier") Link: https://msgid.link/r/20240129162737.497-17-rf@opensource.cirrus.com Signed-off-by: Mark Brown <broonie@kernel.org>
1 parent 77c6072 commit e82bc51

File tree

1 file changed

+9
-6
lines changed

1 file changed

+9
-6
lines changed

sound/pci/hda/cs35l56_hda.c

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -405,16 +405,19 @@ static int cs35l56_hda_request_firmware_file(struct cs35l56_hda *cs35l56,
405405
int ret = 0;
406406

407407
if (system_name && amp_name)
408-
*filename = kasprintf(GFP_KERNEL, "%scs35l56%s-%02x-dsp1-misc-%s-%s.%s", dir,
409-
cs35l56->base.secured ? "s" : "", cs35l56->base.rev,
408+
*filename = kasprintf(GFP_KERNEL, "%scs35l56-%02x%s-dsp1-misc-%s-%s.%s", dir,
409+
cs35l56->base.rev,
410+
cs35l56->base.secured ? "-s" : "",
410411
system_name, amp_name, filetype);
411412
else if (system_name)
412-
*filename = kasprintf(GFP_KERNEL, "%scs35l56%s-%02x-dsp1-misc-%s.%s", dir,
413-
cs35l56->base.secured ? "s" : "", cs35l56->base.rev,
413+
*filename = kasprintf(GFP_KERNEL, "%scs35l56-%02x%s-dsp1-misc-%s.%s", dir,
414+
cs35l56->base.rev,
415+
cs35l56->base.secured ? "-s" : "",
414416
system_name, filetype);
415417
else
416-
*filename = kasprintf(GFP_KERNEL, "%scs35l56%s-%02x-dsp1-misc.%s", dir,
417-
cs35l56->base.secured ? "s" : "", cs35l56->base.rev,
418+
*filename = kasprintf(GFP_KERNEL, "%scs35l56-%02x%s-dsp1-misc.%s", dir,
419+
cs35l56->base.rev,
420+
cs35l56->base.secured ? "-s" : "",
418421
filetype);
419422

420423
if (!*filename)

0 commit comments

Comments
 (0)