[alsa-devel] Remove depop delay for suspend and resume

[Lin, Mengdong]

Hi Kailang/Takashi,

I revised and submitted the patches. Please have a review.

[PATCH 1/2] ALSA: hda/realtek - Remove depop delay for suspend and resume if applicable
[PATCH 2/2] ALSA: hda - fixup ALC262 to remove depop delay on Intel BayleyBay board

Thanks
Mengdong

> -----Original Message-----
> From: alsa-devel-bounces at alsa-project.org
> [mailto:alsa-devel-bounces at alsa-project.org] On Behalf Of Lin, Mengdong
> Sent: Friday, November 29, 2013 5:10 PM
> To: Takashi Iwai
> Cc: alsa-devel at alsa-project.org; Kailang
> Subject: Re: [alsa-devel] Remove depop delay for suspend and resume
> 
> > -----Original Message-----
> > From: Takashi Iwai [mailto:tiwai at suse.de]
> > Sent: Friday, November 29, 2013 4:23 PM
> 
> 
> > > Thank you for the info! I only tested Baytrail-M platform.
> > >
> > > Is it possible to add a flag and set the flag a in board-specific fixup?
> >
> > Yes, feel free to hack struct alc_spec.
> 
> Okay. I'll modify the patch as you suggested, and add a board fixup for
> Baytrail.
> 
> Thanks
> Mengdong
> 
> > > > > At Fri, 29 Nov 2013 10:48:20 +0800, Kailang wrote:
> > > > >>
> > > > >> Hi Takashi,
> > > > >>
> > > > >> Intel test alc262.
> > > > >> They find the suspend and resume time were > 50ms.
> > > > >> So, I delete the delay for alc_eapd_shutup and alc_resume.
> > > > >> If we want to have depop functions, we will follow like alc283.
> > > > >
> > > > > Which delay value to take is pretty specific to machine.
> > > > > Removing delay will likely cause problem on other machines.
> > > > >
> > > > > So, if we'd remove delays, do it only for tested machines, not
> > > > > do blindly to all machines using the same codec chips.  The pop
> > > > > noise isn't only determined by a codec chip but rather
> > > > > influenced by the whole hardware analog design.
> > > > >
> > > > >
> > > > > thanks, > > > >
> > > > > Takashi