asihpi: clean up a couple of build warnings
diff mbox

Message ID 20171016121921.12722-1-colin.king@canonical.com
State New
Headers show

Commit Message

Colin King Oct. 16, 2017, 12:19 p.m. UTC
From: Colin Ian King <colin.king@canonical.com>

Variable dpcm is never used and hence can be removed. Variable
runtime is being assigned but is never read, so the assignment
is redundant and can be removed. Cleans up two clang warnings:

Value stored to 'dpcm' is never read
Value stored to 'runtime' is never read

Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
 sound/pci/asihpi/asihpi.c | 3 ---
 1 file changed, 3 deletions(-)

Comments

Takashi Iwai Oct. 16, 2017, 12:48 p.m. UTC | #1
On Mon, 16 Oct 2017 14:19:21 +0200,
Colin King wrote:
> 
> From: Colin Ian King <colin.king@canonical.com>
> 
> Variable dpcm is never used and hence can be removed. Variable
> runtime is being assigned but is never read, so the assignment
> is redundant and can be removed. Cleans up two clang warnings:
> 
> Value stored to 'dpcm' is never read
> Value stored to 'runtime' is never read
> 
> Signed-off-by: Colin Ian King <colin.king@canonical.com>

Applied (with the subject fix) now, thanks.


Takashi

Patch
diff mbox

diff --git a/sound/pci/asihpi/asihpi.c b/sound/pci/asihpi/asihpi.c
index 614322171c79..720361455c60 100644
--- a/sound/pci/asihpi/asihpi.c
+++ b/sound/pci/asihpi/asihpi.c
@@ -573,10 +573,8 @@  static void snd_card_asihpi_pcm_int_start(struct snd_pcm_substream *substream)
 
 static void snd_card_asihpi_pcm_int_stop(struct snd_pcm_substream *substream)
 {
-	struct snd_card_asihpi_pcm *dpcm;
 	struct snd_card_asihpi *card;
 
-	dpcm = (struct snd_card_asihpi_pcm *)substream->runtime->private_data;
 	card = snd_pcm_substream_chip(substream);
 
 	hpi_handle_error(hpi_adapter_set_property(card->hpi->adapter->index,
@@ -863,7 +861,6 @@  static void snd_card_asihpi_timer_function(struct timer_list *t)
 
 	snd_pcm_group_for_each_entry(s, substream) {
 		struct snd_card_asihpi_pcm *ds = s->runtime->private_data;
-		runtime = s->runtime;
 
 		/* don't link Cap and Play */
 		if (substream->stream != s->stream)