From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bartlomiej Zolnierkiewicz Subject: Re: [RFC]add ACPI hooks for IDE suspend/resume Date: Wed, 7 Dec 2005 15:45:19 +0100 Message-ID: <58cb370e0512070645o78569e82y63483a9ae5511f52@mail.gmail.com> References: <1133849404.3026.10.camel@sli10-mobl.sh.intel.com> <20051206222001.GA14171@srcf.ucam.org> <58cb370e0512070017u606ee22fse207b9a859856dd4@mail.gmail.com> <20051207131454.GA16558@srcf.ucam.org> <58cb370e0512070619k17022317v8e871dc3f9cafb9@mail.gmail.com> <20051207143337.GA16938@srcf.ucam.org> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7BIT Return-path: Received: from nproxy.gmail.com ([64.233.182.203]:15594 "EHLO nproxy.gmail.com") by vger.kernel.org with ESMTP id S1751105AbVLGOpV convert rfc822-to-8bit (ORCPT ); Wed, 7 Dec 2005 09:45:21 -0500 Received: by nproxy.gmail.com with SMTP id x29so117575nfb for ; Wed, 07 Dec 2005 06:45:19 -0800 (PST) In-Reply-To: <20051207143337.GA16938@srcf.ucam.org> Content-Disposition: inline Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Matthew Garrett Cc: Shaohua Li , linux-ide , lkml , pavel , Len Brown , akpm On 12/7/05, Matthew Garrett wrote: > On Wed, Dec 07, 2005 at 03:19:07PM +0100, Bartlomiej Zolnierkiewicz wrote: > > On 12/7/05, Matthew Garrett wrote: > > > On Wed, Dec 07, 2005 at 09:17:31AM +0100, Bartlomiej Zolnierkiewicz wrote: > > > > > > > Isn't ide-io.c:ide_{start,complete}_power_step() enough? > > > > > > No. > > > > Why? :) > > Heh :) The failure is (IIRC, I don't have the serial logs to hand) > before ide_start_power_step() is called. We get to start_request(), and > the system then fails with "bus not ready on wakeup" and "drive not > ready on wakeup". Calling the _GTM and _STM ACPI methods before then > lets the system come back up normally. OK, I understand it now - when using 'ide-generic' host driver for IDE PCI device, resume fails (for obvious reason - IDE PCI device is not re-configured) and this patch fixes it through using ACPI methods. Thanks, Bartlomiej