All of lore.kernel.org
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: George Gugulea <gugulea@gmail.com>
Cc: "alsa-devel@alsa-project.org" <alsa-devel@alsa-project.org>,
	Kailang <kailang@realtek.com>
Subject: Re: No sound after suspend-to-ram
Date: Tue, 27 Oct 2015 07:50:14 +0100	[thread overview]
Message-ID: <s5hio5s955l.wl-tiwai@suse.de> (raw)
In-Reply-To: <CALcwt_D=n2mp-X8LZr1HeFPGM35ODaEJyRt3k=gj65gK1hDX4g@mail.gmail.com>

On Mon, 26 Oct 2015 22:51:18 +0100,
George Gugulea wrote:
> 
> Thanks Kailang, I applied the patch and it works.
> I attached again the diffs, before and after PM (with `echo 1 >
> /sys/modules/snd_hda_codec/parameters/dump_coef`) so you can see the
> coefs have the same values.

OK, I queued the patch now.  Not sure whether this will be in 4.3
final, but certainly included in 4.4-rc1.


thanks,

Takashi

> 
> 
> On Mon, Oct 26, 2015 at 1:39 PM, Takashi Iwai <tiwai@suse.de> wrote:
> > On Mon, 26 Oct 2015 08:45:02 +0100,
> > Kailang wrote:
> >>
> >>
> >> Attach is the patch.
> >
> > Looks good to me.
> > I'm going to queue it once when George confirms that it works indeed.
> >
> >
> > thanks,
> >
> > Takashi
> >
> >>
> >> > -----Original Message-----
> >> > From: Takashi Iwai [mailto:tiwai@suse.de]
> >> > Sent: Monday, October 26, 2015 3:13 PM
> >> > To: Kailang
> >> > Cc: George Gugulea; alsa-devel@alsa-project.org
> >> > Subject: Re: [alsa-devel] No sound after suspend-to-ram
> >> >
> >> > On Mon, 26 Oct 2015 08:00:20 +0100,
> >> > Kailang wrote:
> >> > >
> >> > > Hi Takashi,
> >> > >
> >> > >         [ALC275_FIXUP_DELL_XPS] = {
> >> > >                 .type = HDA_FIXUP_VERBS,
> >> > >                 .v.verbs = (const struct hda_verb[]) {
> >> > >                         /* Enables internal speaker */
> >> > >                         {0x20, AC_VERB_SET_COEF_INDEX, 0x1f},
> >> > >                         {0x20, AC_VERB_SET_PROC_COEF, 0x00c0},
> >> > >                         {0x20, AC_VERB_SET_COEF_INDEX, 0x30},
> >> > >                         {0x20, AC_VERB_SET_PROC_COEF, 0x00b1},
> >> > >                         {}
> >> > >                 }
> >> > >         },
> >> > >
> >> > > Could I put code as upper? Was it enter to initial state?
> >> > > I think it should be execute atfter resume back. Right?
> >> >
> >> > Yes, these are performed at resume.  But the COEF stuff needs
> >> > a bit care since it's written in multiple places.  If you can
> >> > cook up the patch, I can check it hda-emu, too.
> >> >
> >> >
> >> > thanks,
> >> >
> >> > Takashi
> >> >
> >> > ------Please consider the environment before printing this e-mail.
> >> >
> >> [2 0000-dell-xps-speaker-no-sound.patch <application/octet-stream (base64)>]
> >>
> 
> 
> 
> -- 
> gg
> [2 alsa-info.txt.4.3-rc6.fix.before <application/octet-stream (base64)>]
> 
> --- alsa-info.txt.4.3-rc6.fix.before	2015-10-26 23:23:04.626388315 +0200
> +++ alsa-info.txt.4.3-rc6.fix.after	2015-10-26 23:27:32.191807203 +0200
> @@ -3,7 +3,7 @@
>  !!ALSA Information Script v 0.4.64
>  !!################################
>  
> -!!Script ran on: Mon Oct 26 21:23:02 UTC 2015
> +!!Script ran on: Mon Oct 26 21:27:31 UTC 2015
>  
>  
>  !!Linux Distribution
> @@ -142,7 +142,7 @@
>      ControlAmp: chs=3, dir=Out, idx=0, ofs=0
>    Device: name="ALC3260 Analog", type="Audio", device=0
>    Amp-Out caps: ofs=0x3f, nsteps=0x40, stepsize=0x03, mute=0
> -  Amp-Out vals:  [0x1f 0x1f]
> +  Amp-Out vals:  [0x24 0x24]
>    Converter: stream=5, channel=0
>    PCM:
>      rates [0x560]: 44100 48000 96000 192000
> @@ -154,7 +154,7 @@
>    Control: name="Speaker Playback Volume", index=0, device=0
>      ControlAmp: chs=3, dir=Out, idx=0, ofs=0
>    Amp-Out caps: ofs=0x3f, nsteps=0x40, stepsize=0x03, mute=0
> -  Amp-Out vals:  [0x1f 0x1f]
> +  Amp-Out vals:  [0x24 0x24]
>    Converter: stream=5, channel=0
>    PCM:
>      rates [0x560]: 44100 48000 96000 192000
> @@ -241,14 +241,14 @@
>  Node 0x11 [Audio Input] wcaps 0x10051b: Stereo Amp-In
>    Amp-In caps: ofs=0x11, nsteps=0x2e, stepsize=0x03, mute=1
>    Amp-In vals:  [0x10 0x10]
> -  Converter: stream=1, channel=0
> +  Converter: stream=0, channel=0
>    SDI-Select: 0
>    PCM:
>      rates [0x160]: 44100 48000 96000
>      bits [0xe]: 16 20 24
>      formats [0x1]: PCM
>    Power states:  D0 D1 D2 D3 EPSS
> -  Power: setting=D0, actual=D0
> +  Power: setting=D3, actual=D3
>    Connection: 1
>       0x22
>  Node 0x12 [Pin Complex] wcaps 0x40040b: Stereo Amp-In
> @@ -263,7 +263,7 @@
>      Misc = NO_PRESENCE
>    Pin-ctls: 0x20: IN
>    Power states:  D0 D1 D2 D3 EPSS
> -  Power: setting=D0, actual=D0
> +  Power: setting=D3, actual=D3
>  Node 0x13 [Pin Complex] wcaps 0x40040b: Stereo Amp-In
>    Amp-In caps: ofs=0x00, nsteps=0x03, stepsize=0x27, mute=0
>    Amp-In vals:  [0x00 0x00]
> @@ -532,7 +532,7 @@
>  crw-rw----+ 1 root audio 116,  6 Oct 26 23:22 /dev/snd/hwC0D0
>  crw-rw----+ 1 root audio 116,  7 Oct 26 23:22 /dev/snd/hwC0D3
>  crw-rw----+ 1 root audio 116,  4 Oct 26 23:22 /dev/snd/pcmC0D0c
> -crw-rw----+ 1 root audio 116,  3 Oct 26 23:22 /dev/snd/pcmC0D0p
> +crw-rw----+ 1 root audio 116,  3 Oct 26 23:27 /dev/snd/pcmC0D0p
>  crw-rw----+ 1 root audio 116,  5 Oct 26 23:22 /dev/snd/pcmC0D3p
>  crw-rw----+ 1 root audio 116,  1 Oct 26 23:22 /dev/snd/seq
>  crw-rw----+ 1 root audio 116, 33 Oct 26 23:22 /dev/snd/timer
> @@ -551,7 +551,7 @@
>  
>  **** List of PLAYBACK Hardware Devices ****
>  card 0: PCH [HDA Intel PCH], device 0: ALC3260 Analog [ALC3260 Analog]
> -  Subdevices: 1/1
> +  Subdevices: 0/1
>    Subdevice #0: subdevice #0
>  card 0: PCH [HDA Intel PCH], device 3: HDMI 0 [HDMI 0]
>    Subdevices: 1/1
> @@ -578,7 +578,7 @@
>    Capabilities: pvolume pvolume-joined pswitch pswitch-joined
>    Playback channels: Mono
>    Limits: Playback 0 - 64
> -  Mono: Playback 32 [50%] [-32.00dB] [on]
> +  Mono: Playback 37 [58%] [-27.00dB] [on]
>  Simple mixer control 'Headphone',0
>    Capabilities: pvolume pswitch
>    Playback channels: Front Left - Front Right
> @@ -804,7 +804,7 @@
>  	control.12 {
>  		iface MIXER
>  		name 'Master Playback Volume'
> -		value 32
> +		value 37
>  		comment {
>  			access 'read write'
>  			type INTEGER
> @@ -812,7 +812,7 @@
>  			range '0 - 64'
>  			dbmin -6400
>  			dbmax 0
> -			dbvalue.0 -3200
> +			dbvalue.0 -2700
>  		}
>  	}
>  	control.13 {
> @@ -895,8 +895,8 @@
>  	control.20 {
>  		iface PCM
>  		name 'Playback Channel Map'
> -		value.0 0
> -		value.1 0
> +		value.0 3
> +		value.1 4
>  		comment {
>  			access read
>  			type INTEGER
> @@ -1021,6 +1021,8 @@
>  !!------------------
>  
>  Module
> +alx
> +mdio
>  vmnet
>  vmw_vsock_vmci_transport
>  vsock
> @@ -1136,11 +1138,9 @@
>  rtsx_pci_sdmmc
>  psmouse
>  ahci
> -alx
>  sdhci_pci
>  libahci
>  rtsx_pci
> -mdio
>  sdhci
>  floppy
>  

  reply	other threads:[~2015-10-27  6:50 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-22  8:38 No sound after suspend-to-ram George Gugulea
2015-10-22 15:15 ` Takashi Iwai
2015-10-22 21:23   ` George Gugulea
2015-10-23  7:30     ` Takashi Iwai
2015-10-23  7:34       ` Takashi Iwai
2015-10-23  8:28       ` Kailang
2015-10-23 22:01         ` George Gugulea
2015-10-24  7:38           ` Takashi Iwai
2015-10-26  3:20             ` Kailang
2015-10-26  7:00             ` Kailang
2015-10-26  7:13               ` Takashi Iwai
     [not found]                 ` <6FAB7C47BCF00940BB0999A99BE3547A17A237D4@RTITMBSV09.realtek.com.tw>
2015-10-26  9:00                   ` George Gugulea
2015-10-26 11:39                   ` Takashi Iwai
2015-10-26 21:51                     ` George Gugulea
2015-10-27  6:50                       ` Takashi Iwai [this message]
  -- strict thread matches above, loose matches on Subject: below --
2015-10-21 17:29 George Gugulea

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=s5hio5s955l.wl-tiwai@suse.de \
    --to=tiwai@suse.de \
    --cc=alsa-devel@alsa-project.org \
    --cc=gugulea@gmail.com \
    --cc=kailang@realtek.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.