On Tue, May 06, 2025 at 07:37:01AM +0000, Ding, Shenghao wrote:
One of the key things about the define that's used for the regmap API version of this is that it's not visible in Kconfig, you have to edit the kernel source due to the whole directly exposing the registers things. It'd be too easy for a distro or something to just turn the define on by mistake otherwise. That's why it's a define in the code instead. The code itself looks fine.
I need to drop the following setting in Kconfig, right? +if SND_SOC_TAS2781_I2C
+config SND_SOC_TAS2781_I2C_ACOUST ......................... +endif ## SND_SOC_TAS2781_I2C
Yes.
As to REGMAP API, when enabling SND_SOC_TAS2781_COMLIB_I2C will select REGMAP_I2C. And enabling SND_SOC_TAS2781_I2C will select SND_SOC_TAS2781_COMLIB_I2C. So whether SND_SOC_TAS2781_I2C_ACOUST is enabled or not, REGMAP_I2C is selected.
Yes, the selecting of regmap is no problem - I was referring to the regmap debugfs interface for writing to registers here.