From mboxrd@z Thu Jan 1 00:00:00 1970 From: Patrizio Bassi Subject: Re: Linux 2.6.16-rc3 Date: Mon, 13 Feb 2006 15:34:28 +0100 Message-ID: <43F098F4.8030906@gmail.com> References: <20060212190520.244fcaec.akpm@osdl.org> <43F07DA3.6080702@gmail.com> <43F08A2B.2030607@gmail.com> Reply-To: patrizio.bassi@gmail.com Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: Received: from wproxy.gmail.com ([64.233.184.202]:35734 "EHLO wproxy.gmail.com") by vger.kernel.org with ESMTP id S1751267AbWBMOen (ORCPT ); Mon, 13 Feb 2006 09:34:43 -0500 Received: by wproxy.gmail.com with SMTP id i20so88380wra for ; Mon, 13 Feb 2006 06:34:42 -0800 (PST) In-Reply-To: Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: Takashi Iwai Cc: Andrew Morton , Linus Torvalds , linux-kernel@vger.kernel.org, Jens Axboe , James Bottomley , "Brown, Len" , "David S. Miller" , Greg KH , linux-acpi@vger.kernel.org, linux-usb-devel@lists.sourceforge.net, "Yu, Luming" , Ben Castricum , sanjoy@mrao.cam.ac.uk, Helge Hafting , "Carlo E. Prelz" , =?ISO-8859-1?Q?Gerrit_Bruchh=E4us?= =?ISO-8859-1?Q?er?= , Nicolas.Mailhot@LaPoste.net, Jaroslav Kysela , =?ISO-8859-1?Q?Bj=F6rn_Nilsson?= , Andrey Borzenkov , "P. Christeas" , ghrt , jinhong hu , Andrew Vasquez Takashi Iwai ha scritto: > At Mon, 13 Feb 2006 14:31:23 +0100, > Patrizio Bassi wrote: > >> Takashi Iwai ha scritto: >> >>> At Mon, 13 Feb 2006 13:37:55 +0100, >>> Patrizio Bassi wrote: >>> >>> >>>> Takashi Iwai ha scritto: >>>> >>>> >>>>> At Sun, 12 Feb 2006 19:05:20 -0800, >>>>> Andrew Morton wrote: >>>>> >>>>> >>>>> >>>>>> - Patrizio Bassi has an alsa suspend >>>>>> regression ("alsa suspend/resume continues to fail for ens1370") >>>>>> >>>>>> >>>>>> >>>>> It's not a "regression". PM didn't work with ens1370 at all in the >>>>> eralier version. >>>>> >>>>> About the problem there, I have no idea now what's wrong. The >>>>> suspend-to-disk works fine if the driver is built as module but not as >>>>> built-in kernel. >>>>> >>>>> >>>>> >>>>> >>>>> >>>> i wrote "regression" because before (ehm...exactly don't know...about >>>> 2.6.14 time) >>>> after suspend i had to restart my distro's mixer values service or i >>>> couldn't hear anything. >>>> and...ok..it was boring but worked. >>>> >>>> >>> You abused the function which wasn't officially supported :) >>> >>> >>> Takashi >>> >>> >>> >> nice i'm an abuser! :) >> >> ok, seriously..that's bad, because before it was not implemented, so ok... >> but now it fails with errors (and make apps not working properly) which >> is worse. >> > > My rough guess is the initialization order, the resume was called too > early. > > What about to put sleep between snd_ensoniq_chip_init() and > snd_ak4531_resume()? Or put more delay in snd_ak4531_resume()? > > > Takashi > > i'm almost sure the problem is not there (or, at least not only) infact i get 0x660 errors (or better a long flood...) while suspending too. there may be a bug or problem during suspending, and these problems affect the normal resume. however sleep is not always trustable...soif you think that's an init problem you may add a boolean value to check if init is completed or not and poll for TRUE value in the resume function. but, as i wrote before, i'm not sure the problem is **only** there. Patrizio