On Fri, Aug 23, 2024 at 02:24:17AM +0000, Ding, Shenghao wrote:
From: Andy Shevchenko andriy.shevchenko@linux.intel.com Sent: Thursday, August 22, 2024 9:37 PM To: Ding, Shenghao shenghao-ding@ti.com On Thu, Aug 22, 2024 at 02:32:02PM +0800, Shenghao Ding wrote:
...
/* pow(10, db/20) * pow(2,30) */ -static const unsigned char tas2563_dvc_table[][4] = { +static const __maybe_unused unsigned char tas2563_dvc_table[][4] = { { 0X00, 0X00, 0X00, 0X00 }, /* -121.5db */ { 0X00, 0X00, 0X03, 0XBC }, /* -121.0db */ { 0X00, 0X00, 0X03, 0XF5 }, /* -120.5db */
As far as I can see in the latest Linux Next the above mentioned table is used solely in sound/soc/codecs/tas2781-i2c.c. Why not moving it to the C file instead?
We have a big plan to implement the digital gain for tas2563 in the side codec driver too, so abstracted this table into a separated header file.
So, this __maybe_unused is a temporary stub, correct? Just don't forget to remove it when the mentioned functionality will be ready.
Thanks for the explanation.