Re: [PATCH] PM: Introduce PM_EVENT_HIBERNATE (was: Re: i915 hibernation patch (was: Re: 2.6.25-rc2 System no longer ...))

2008-02-24 Thread Jeff Chua
On Sun, Feb 24, 2008 at 2:43 AM, Linus Torvalds <[EMAIL PROTECTED]> wrote: > > > On Sat, 23 Feb 2008, Rafael J. Wysocki wrote: > > > > > Thanks for testing. Below is the final version of the patch with a > > changelog > > etc. > > Thanks, applied. > > With this, I also find that I dislike

Re: [PATCH] PM: Introduce PM_EVENT_HIBERNATE (was: Re: i915 hibernation patch (was: Re: 2.6.25-rc2 System no longer ...))

2008-02-24 Thread Rafael J. Wysocki
On Sunday, 24 of February 2008, Rafael J. Wysocki wrote: > On Sunday, 24 of February 2008, Mirco Tischler wrote: > > > > On Sa, 2008-02-23 at 19:13 +0100, Rafael J. Wysocki wrote: > > > Index: linux-2.6/drivers/usb/host/u132-hcd.c > > >

Re: [PATCH] PM: Introduce PM_EVENT_HIBERNATE (was: Re: i915 hibernation patch (was: Re: 2.6.25-rc2 System no longer ...))

2008-02-24 Thread Rafael J. Wysocki
On Sunday, 24 of February 2008, Mirco Tischler wrote: > > On Sa, 2008-02-23 at 19:13 +0100, Rafael J. Wysocki wrote: > > Index: linux-2.6/drivers/usb/host/u132-hcd.c > > === > > --- linux-2.6.orig/drivers/usb/host/u132-hcd.c > > +++

Re: [PATCH] PM: Introduce PM_EVENT_HIBERNATE (was: Re: i915 hibernation patch (was: Re: 2.6.25-rc2 System no longer ...))

2008-02-24 Thread Pavel Machek
Hi! > Thanks, applied. > > With this, I also find that I dislike the use of suspend/resume for > freezing for STD a lot less. It's still too easy to get confused, but at > least now drivers always have total knowledge about what is really going > on. I'd not like this interface as a driver

Re: [PATCH] PM: Introduce PM_EVENT_HIBERNATE (was: Re: i915 hibernation patch (was: Re: 2.6.25-rc2 System no longer ...))

2008-02-24 Thread Pavel Machek
Hi! Thanks, applied. With this, I also find that I dislike the use of suspend/resume for freezing for STD a lot less. It's still too easy to get confused, but at least now drivers always have total knowledge about what is really going on. I'd not like this interface as a driver writer,

Re: [PATCH] PM: Introduce PM_EVENT_HIBERNATE (was: Re: i915 hibernation patch (was: Re: 2.6.25-rc2 System no longer ...))

2008-02-24 Thread Rafael J. Wysocki
On Sunday, 24 of February 2008, Mirco Tischler wrote: On Sa, 2008-02-23 at 19:13 +0100, Rafael J. Wysocki wrote: Index: linux-2.6/drivers/usb/host/u132-hcd.c === --- linux-2.6.orig/drivers/usb/host/u132-hcd.c +++

Re: [PATCH] PM: Introduce PM_EVENT_HIBERNATE (was: Re: i915 hibernation patch (was: Re: 2.6.25-rc2 System no longer ...))

2008-02-24 Thread Rafael J. Wysocki
On Sunday, 24 of February 2008, Rafael J. Wysocki wrote: On Sunday, 24 of February 2008, Mirco Tischler wrote: On Sa, 2008-02-23 at 19:13 +0100, Rafael J. Wysocki wrote: Index: linux-2.6/drivers/usb/host/u132-hcd.c ===

Re: [PATCH] PM: Introduce PM_EVENT_HIBERNATE (was: Re: i915 hibernation patch (was: Re: 2.6.25-rc2 System no longer ...))

2008-02-24 Thread Jeff Chua
On Sun, Feb 24, 2008 at 2:43 AM, Linus Torvalds [EMAIL PROTECTED] wrote: On Sat, 23 Feb 2008, Rafael J. Wysocki wrote: Thanks for testing. Below is the final version of the patch with a changelog etc. Thanks, applied. With this, I also find that I dislike the use of

Re: [PATCH] PM: Introduce PM_EVENT_HIBERNATE (was: Re: i915 hibernation patch (was: Re: 2.6.25-rc2 System no longer ...))

2008-02-23 Thread Mirco Tischler
On Sa, 2008-02-23 at 19:13 +0100, Rafael J. Wysocki wrote: > Index: linux-2.6/drivers/usb/host/u132-hcd.c > === > --- linux-2.6.orig/drivers/usb/host/u132-hcd.c > +++ linux-2.6/drivers/usb/host/u132-hcd.c > @@ -3214,14 +3214,19 @@

Re: [PATCH] PM: Introduce PM_EVENT_HIBERNATE (was: Re: i915 hibernation patch (was: Re: 2.6.25-rc2 System no longer ...))

2008-02-23 Thread Linus Torvalds
On Sat, 23 Feb 2008, Rafael J. Wysocki wrote: > > Thanks for testing. Below is the final version of the patch with a changelog > etc. Thanks, applied. With this, I also find that I dislike the use of suspend/resume for freezing for STD a lot less. It's still too easy to get confused, but

[PATCH] PM: Introduce PM_EVENT_HIBERNATE (was: Re: i915 hibernation patch (was: Re: 2.6.25-rc2 System no longer ...))

2008-02-23 Thread Rafael J. Wysocki
On Saturday, 23 of February 2008, Jeff Chua wrote: > On Sat, Feb 23, 2008 at 10:07 AM, Linus Torvalds > <[EMAIL PROTECTED]> wrote: > > > On Sat, 23 Feb 2008, Rafael J. Wysocki wrote: > > > OK, please have a look at the modified patch below. > > > > All right, I'm fine with it. Now we just need

[PATCH] PM: Introduce PM_EVENT_HIBERNATE (was: Re: i915 hibernation patch (was: Re: 2.6.25-rc2 System no longer ...))

2008-02-23 Thread Rafael J. Wysocki
On Saturday, 23 of February 2008, Jeff Chua wrote: On Sat, Feb 23, 2008 at 10:07 AM, Linus Torvalds [EMAIL PROTECTED] wrote: On Sat, 23 Feb 2008, Rafael J. Wysocki wrote: OK, please have a look at the modified patch below. All right, I'm fine with it. Now we just need to confirm

Re: [PATCH] PM: Introduce PM_EVENT_HIBERNATE (was: Re: i915 hibernation patch (was: Re: 2.6.25-rc2 System no longer ...))

2008-02-23 Thread Linus Torvalds
On Sat, 23 Feb 2008, Rafael J. Wysocki wrote: Thanks for testing. Below is the final version of the patch with a changelog etc. Thanks, applied. With this, I also find that I dislike the use of suspend/resume for freezing for STD a lot less. It's still too easy to get confused, but at

Re: [PATCH] PM: Introduce PM_EVENT_HIBERNATE (was: Re: i915 hibernation patch (was: Re: 2.6.25-rc2 System no longer ...))

2008-02-23 Thread Mirco Tischler
On Sa, 2008-02-23 at 19:13 +0100, Rafael J. Wysocki wrote: Index: linux-2.6/drivers/usb/host/u132-hcd.c === --- linux-2.6.orig/drivers/usb/host/u132-hcd.c +++ linux-2.6/drivers/usb/host/u132-hcd.c @@ -3214,14 +3214,19 @@ static