Re: [PATCH] Remove unused variable 'codec' from sound/soc/codecs/tlv320aic23.c

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

 



At Tue, 28 Oct 2008 12:33:33 +0530,
Arun KS wrote:
> 
> Manish Katiyar wrote:
> > [ Apologies if this is not the proper list for this patch, but I
> > couldn't find any list more appropriate for this]
> >
> > Below patch removes the following warning.
> > sound/soc/codecs/tlv320aic23.c:424: warning: unused variable 'codec'
> 
> CCing Troy
> 
> Troy has got some patch which adds support for more mclk frequencies.
> So that variable will be required.

Please let me know the state of Troy's patch.
If his fix is to be merged to 2.6.28 kernel, then I'll keep the code
as is.  If it's for the next kernel (2.6.29), I'll fix this warning
now.

thanks,

Takashi


> Thanks,
> Arun KS
> >
> >
> > Signed-off-by: Manish Katiyar <mkatiyar@xxxxxxxxx>
> > ---
> >  sound/soc/codecs/tlv320aic23.c |    2 --
> >  1 files changed, 0 insertions(+), 2 deletions(-)
> >
> > diff --git a/sound/soc/codecs/tlv320aic23.c b/sound/soc/codecs/tlv320aic23.c
> > index 44308da..45395d0 100644
> > --- a/sound/soc/codecs/tlv320aic23.c
> > +++ b/sound/soc/codecs/tlv320aic23.c
> > @@ -421,8 +421,6 @@ static int tlv320aic23_set_dai_fmt(struct
> > snd_soc_dai *codec_dai,
> >  static int tlv320aic23_set_dai_sysclk(struct snd_soc_dai *codec_dai,
> >  				      int clk_id, unsigned int freq, int dir)
> >  {
> > -	struct snd_soc_codec *codec = codec_dai->codec;
> > -
> >  	switch (freq) {
> >  	case 12000000:
> >  		return 0;
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@xxxxxxxxxxxxxxxx
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
> 
_______________________________________________
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