lkml.org 
[lkml]   [2006]   [Feb]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: 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/

    \
     
     \ /
      Last update: 2006-02-07 07:59    [W:3.345 / U:0.616 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site