Re: [BUG] 2.4.18-pre/rc broke PLIP

From: Bill Davidsen (davidsen@prodigy.com)
Date: Tue Mar 05 2002 - 11:36:59 EST


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/



This archive was generated by hypermail 2b29 : Thu Mar 07 2002 - 21:00:43 EST