ALSA: hda/tas2781: select CRC32 instead of CRC32_SARWATE
Subsystems (2)
PATCH 1/1 ALSA: hda/tas2781: select CRC32 instead of CRC32_SARWATE
From: Eric Biggers <[email protected]>
Fix the kconfig option for the tas2781 HDA driver to select CRC32 rather than CRC32_SARWATE. CRC32_SARWATE is an option from the kconfig 'choice' that selects the specific CRC32 implementation. Selecting a 'choice' option seems to have no effect, but even if it did work, it would be incorrect for a random driver to override the user's choice. CRC32 is the correct option to select for crc32() to be available.
--- sound/pci/hda/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
sound/pci/hda/Kconfig +1 -1
@@ -196,11 +196,11 @@ config SND_HDA_SCODEC_TAS2781_I2C depends on ACPI depends on EFI depends on SND_SOC select SND_SOC_TAS2781_COMLIB select SND_SOC_TAS2781_FMWLIB - select CRC32_SARWATE + select CRC32 help Say Y or M here to include TAS2781 I2C HD-audio side codec support in snd-hda-intel driver, such as ALC287. comment "Set to Y if you want auto-loading the side codec driver"
base-commit: 715ca9dd687f89ddaac8ec8ccb3b5e5a30311a99
Signature · 1 line
2.47.0
Takashi Iwai
AppliedQuoted 10 lines · Eric Biggers
From: Eric Biggers <[email protected]>
Fix the kconfig option for the tas2781 HDA driver to select CRC32 rather than
CRC32_SARWATE.CRC32_SARWATEis an option from the kconfig 'choice' that selects the specific CRC32 implementation. Selecting a 'choice' option seems to have no effect, but even if it did work, it would be incorrect for a random driver to override the user's choice. CRC32 is the correct option to select forcrc32()to be available.
Quoted 3 lines
Fixes 5be27f1e3ec9 ("ALSA: hda/tas2781: Add tas2781 HDA driver")Signed-off-by Eric Biggers <[email protected]>
Thanks, applied now.
Takashi