Messages in this thread | | | Date | Tue, 07 Feb 2006 01:57:07 -0500 | From | Jeff Garzik <> | Subject | Re: e100 oops on resume |
| |
Jesse Brandeburg wrote: > On 1/28/06, Mattia Dongili <malattia@linux.it> wrote: > >>On Thu, Jan 26, 2006 at 08:02:37PM +0100, Stefan Seyfried wrote: >> >>>On Wed, Jan 25, 2006 at 04:28:48PM -0800, Jesse Brandeburg wrote: >>> >>> >>>>Okay I reproduced the issue on 2.6.15.1 (with S1 sleep) and was able >>>>to show that my patch that just removes e100_init_hw works okay for >>>>me. Let me know how it goes for you, I think this is a good fix. >>> >>>worked for me in the Compaq Armada e500 and reportedly also fixed the >>>SONY that originally uncovered it. >> >>confirmed here too. The patch fixes S3 resume on this Sony (GR7/K) >>running 2.6.16-rc1-mm3. > > > excellent news! thanks for testing. > > Jeff, could you please apply to 2.6.16-rcX > > Jesse
SIGH. In your last patch submission you had it right, but Intel has yet again regressed in patch submission form.
Your fixes will be expedited if they can be applied by script, and then quickly whisked upstream to Linus/Andrew. This one had to be applied by hand (so yes, its applied) for several reasons:
* Unreviewable in mail reader, due to MIME type application/octet-stream.
* In general, never use MIME (attachments), they decrease the audience that can easily review your patch.
* Your patch's description and signed-off-by were buried inside the octet-stream attachment.
* Please review http://linux.yyz.us/patch-format.html (I probably should add MIME admonitions to that)
Jeff
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |