On Thu, Aug 22, 2024 at 02:32:02PM +0800, Shenghao Ding wrote: > In case of tas2781, tas2563_dvc_table will be unused, > so mark it as __maybe_unused. ... > /* 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? -- With Best Regards, Andy Shevchenko