Re: [PATCH v2 3/3] ASoC: rt5677: Remove never used variables

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> On Wed, 2017-07-19 at 12:24 +0100, Mark Brown wrote:
> > On Tue, Jul 18, 2017 at 08:34:19PM +0300, Andy Shevchenko wrote:
> > > There are two variables that have never been used.
> > >
> > > sound/soc/codecs/rt5677.c:785:35: warning: ‘in_vol_tlv’ defined but
> > > not used [-Wunused-const-variable=] static const
> > > DECLARE_TLV_DB_SCALE(in_vol_tlv, -3450, 150, 0);
> > >
> > > sound/soc/codecs/rt5677.c:783:35: warning: ‘out_vol_tlv’ defined but
> > > not used [-Wunused-const-variable=] static const
> > > DECLARE_TLV_DB_SCALE(out_vol_tlv, -4650, 150, 0);
> > >
> > > Simple remove them.
> >
> > Are we sure this doesn't mean something has the wrong TLV or a missing
> > TLV?
> 
> Good question.
> 
> If it so, it should not have been working (properly) at the beginning.
> 
> Guys from Realtek, can you comment on this?
> 
They are seemed redundant definitions, thanks. 

_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel




[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux