lkml.org 
[lkml]   [2003]   [Mar]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    SubjectRe: [PATCH] pcmcia: get initialization ordering right [Was: [PATCH 2.5] : i82365 & platform_bus_type]
    From
    On Tue, Mar 04, 2003 at 11:48:22AM -0600, Patrick Mochel wrote:
    >
    > Surely you're sore that your code has required some modifications since
    > Dominik has started working on PCMCIA, and I'm sure that no harm was
    > intended. It's had some bumps, but IMO, he's done a great job, and the
    > result is a vast improvement. The least you could is give the guy some
    > slack, instead of whining about your own inconveniences.

    I don't mind the changes, changes are usually good. In 2.5.X,
    I had to change my code to accomodate the new PCI interface, the
    removal of global IRQ, the new module interface, the various USB API
    changes and other changes. And actually, your work currently hasn't
    had any impact on the source code I follow (yet).
    What I mind is the lack of basic testing. From your patch, the
    initialisation order mixup and the other obvious bug fix I sent you,
    this code had zero chances of working at all and it's obvious that
    nobody bothered to check if it could work or not for at least two
    kernel releases.
    Yeah, I know that I should not complain because at least the
    code did compile (modulo other minor obvious fixes that are already in
    Linus BK).

    > Thanks,
    >
    > -pat

    Have fun...

    Jean
    -
    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: 2005-03-22 13:33    [W:6.151 / U:0.068 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site