On Tue, Apr 30, 2024 at 03:25:42PM +0800, Baojun Xu wrote:
Integrate tas2781 hda spi driver configs for HP (Varcolac). Every tas2781 SPI node was added by serial-multi-instantie.c as a SPI device. The code support Realtek as the primary codec.
...
{"TIAS2781", },
Is this for real? I mean you really abused ACPI specifications to get this into the real products on the market?! If so, never do this again.
_This_ one is non-conforming abuse of ACPI specification and has not to be here at all. But, if the above is the case, provide the models of the devices and the excerpt of the DSDT where this broken ID happen, also add a big comment explaining how comes this happened and that you promise to never happen this again.
...
- { "TIAS2781", (unsigned long)&tas2781_hda },
Ditto.
...
+config SND_HDA_SCODEC_TAS2781_SPI
- tristate "Build TAS2781 HD-audio side codec support for SPI Bus"
- depends on SPI_MASTER
- depends on ACPI
No compile test?
- depends on EFI
- depends on SND_SOC
- select CRC32_SARWATE
- help
Say Y or M here to include TAS2781 SPI HD-audio side codec supportin snd-hda-intel driver, such as ALC287.