lkml.org 
[lkml]   [2002]   [Mar]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [BUG] 2.4.18-pre/rc broke PLIP
    On Mon, 25 Feb 2002, Mikael Pettersson wrote:

    > Someone (sorry I forgot who) reported having problems with
    > PLIP in recent kernels. I've done some testing and can confirm
    > that PLIP worked up to 2.4.17 but broke in 2.4.18-pre/rc.
    > The only thing PLIP does is put "plip0: transmit timeout(1,87)"
    > messages in the kernel log.
    >
    > After a lot of testing I've narrowed it down to the following
    > hunk in the 2.4.18-rc4 patch:
    >
    > --- linux.orig/drivers/parport/parport_pc.c Mon Feb 18 20:18:40 2002
    > +++ linux/drivers/parport/parport_pc.c Mon Jan 14 19:08:50 2002
    > @@ -2212,7 +2233,7 @@
    > }
    > memcpy (ops, &parport_pc_ops, sizeof (struct parport_operations));
    > priv->ctr = 0xc;
    > - priv->ctr_writable = 0xff;
    > + priv->ctr_writable = ~0x10;
    > priv->ecr = 0;
    > priv->fifo_depth = 0;
    > priv->dma_buf = 0;
    >
    > If I back this hunk out, PLIP starts working again.
    > Is this fix sufficient or is there something else that need fixing?

    I'm still looking at this, trying to figure out what they were trying to
    do here... but I note the same code in parport_gsc.c was NOT
    changed. Putting it back makes it work for me, but the code is not long on
    comments, and digging up all the data sheets for various supported chips
    is not a casual job.

    I'm going to browse the code more this week, the bit not set isn't
    documented where I've looked :-(

    Anyway, back to working, perhaps there was a note with the patch giving
    some hint of what it was doing?

    --
    bill davidsen <davidsen@tmr.com> CTO, TMR Associates, Inc
    Programming without software engineering is like sculpting with a chain
    saw. The very talented can produce a work of art, the mediocre wind up with
    a misshapen lump in a pile of rubble, and in neither case does the end
    result have more than a passing resemblance to the original intent.

    -
    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:24    [W:8.541 / U:0.024 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site