On Fri, Aug 23, 2024 at 02:24:17AM +0000, Ding, Shenghao wrote:
> > From: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> > Sent: Thursday, August 22, 2024 9:37 PM
> > To: Ding, Shenghao <shenghao-ding@xxxxxx>
> > 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.
--
With Best Regards,
Andy Shevchenko
