[alsa-devel] [PATCH 1/5] ALSA: hda: Remove unused variable

[Takashi Iwai]

At Tue, 1 Jul 2014 08:47:39 -0700,
Dylan Reid wrote:
> 
> On Tue, Jul 1, 2014 at 8:07 AM, Stephen Warren <swarren at wwwdotorg.org> wrote:
> > On 07/01/2014 05:32 AM, Sachin Kamat wrote:
> >> 'status' is not used in the function. Remove it.
> >
> > CCing Dylan Reid for comments/testing since he wrote or upstreamed this
> > code.
> 
> Thanks, I gave this a try on a couple of systems this morning.  HDMI
> audio still works through suspend/resume cycles.
> 
> >
> >> Signed-off-by: Sachin Kamat <sachin.kamat at samsung.com>
> >> Cc: Stephen Warren <swarren at wwwdotorg.org>
> 
> Tested-by: Dylan Reid <dgreid at chromium.org>

Great, I applied the patch now.  Thanks!


Takashi

> 
> >> Cc: Thierry Reding <thierry.reding at gmail.com>
> >> ---
> >> All patches in this series only compile tested.
> >> ---
> >>  sound/pci/hda/hda_tegra.c |    4 ----
> >>  1 file changed, 4 deletions(-)
> >>
> >> diff --git a/sound/pci/hda/hda_tegra.c b/sound/pci/hda/hda_tegra.c
> >> index a366ba9293a8..8cd7b06eecef 100644
> >> --- a/sound/pci/hda/hda_tegra.c
> >> +++ b/sound/pci/hda/hda_tegra.c
> >> @@ -272,13 +272,9 @@ static int hda_tegra_resume(struct device *dev)
> >>       struct snd_card *card = dev_get_drvdata(dev);
> >>       struct azx *chip = card->private_data;
> >>       struct hda_tegra *hda = container_of(chip, struct hda_tegra, chip);
> >> -     int status;
> >>
> >>       hda_tegra_enable_clocks(hda);
> >>
> >> -     /* Read STATESTS before controller reset */
> >> -     status = azx_readw(chip, STATESTS);
> >> -
> >>       hda_tegra_init(hda);
> >>
> >>       azx_init_chip(chip, 1);
> >>
> >
>