From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1030888AbbJ3LeI (ORCPT ); Fri, 30 Oct 2015 07:34:08 -0400 Received: from mx2.suse.de ([195.135.220.15]:54512 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1030377AbbJ3LeF (ORCPT ); Fri, 30 Oct 2015 07:34:05 -0400 Date: Fri, 30 Oct 2015 12:34:03 +0100 Message-ID: From: Takashi Iwai To: "Ondrej Zary" Cc: , "Kernel development list" Subject: Re: [PATCH] snd-cs46xx: Fix suspend for all channels In-Reply-To: <1446148344-1209-1-git-send-email-linux@rainbow-software.org> References: <1446148344-1209-1-git-send-email-linux@rainbow-software.org> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI/1.14.6 (Maruoka) FLIM/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL/10.8 Emacs/24.5 (x86_64-suse-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 29 Oct 2015 20:52:24 +0100, Ondrej Zary wrote: > > snd_cs46xx_suspend() is missing snd_pcm_suspend_all calls for pcm_rear, > pcm_center_lfe and pcm_iec958. > > There's also a copy/paste bug in snd_cs46xx_pcm_iec958(), overwriting > pcm_rear in struct snd_cs46xx, leaving pcm_iec958 unused. > > Fix all this to allow all channels to survive suspend/resume. > > Signed-off-by: Ondrej Zary Applied, thanks. Takashi > --- > sound/pci/cs46xx/cs46xx_lib.c | 7 ++++++- > 1 file changed, 6 insertions(+), 1 deletion(-) > > diff --git a/sound/pci/cs46xx/cs46xx_lib.c b/sound/pci/cs46xx/cs46xx_lib.c > index e62c841..2706f27 100644 > --- a/sound/pci/cs46xx/cs46xx_lib.c > +++ b/sound/pci/cs46xx/cs46xx_lib.c > @@ -1864,7 +1864,7 @@ int snd_cs46xx_pcm_iec958(struct snd_cs46xx *chip, int device) > /* global setup */ > pcm->info_flags = 0; > strcpy(pcm->name, "CS46xx - IEC958"); > - chip->pcm_rear = pcm; > + chip->pcm_iec958 = pcm; > > snd_pcm_lib_preallocate_pages_for_all(pcm, SNDRV_DMA_TYPE_DEV, > snd_dma_pci_data(chip->pci), 64*1024, 256*1024); > @@ -3780,6 +3780,11 @@ static int snd_cs46xx_suspend(struct device *dev) > snd_power_change_state(card, SNDRV_CTL_POWER_D3hot); > chip->in_suspend = 1; > snd_pcm_suspend_all(chip->pcm); > +#ifdef CONFIG_SND_CS46XX_NEW_DSP > + snd_pcm_suspend_all(chip->pcm_rear); > + snd_pcm_suspend_all(chip->pcm_center_lfe); > + snd_pcm_suspend_all(chip->pcm_iec958); > +#endif > // chip->ac97_powerdown = snd_cs46xx_codec_read(chip, AC97_POWER_CONTROL); > // chip->ac97_general_purpose = snd_cs46xx_codec_read(chip, BA0_AC97_GENERAL_PURPOSE); > > -- > Ondrej Zary > >