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

Dylan Reid dgreid at google.com
Tue Jul 1 17:47:39 CEST 2014


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>

>> 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);
>>
>


More information about the Alsa-devel mailing list