From: Tim Waugh <twaugh@redhat.com>
To: Jakob Kemi <jakob.kemi@telia.com>
Cc: Marcelo Tosatti <marcelo@conectiva.com.br>,
lkml <linux-kernel@vger.kernel.org>
Subject: Re: Linux 2.4.17-pre4
Date: Wed, 5 Dec 2001 22:18:12 +0000 [thread overview]
Message-ID: <20011205221812.U14028@redhat.com> (raw)
In-Reply-To: <Pine.LNX.4.21.0112051640570.20575-100000@freak.distro.conectiva> <200112052155.fB5Ltxa26014@d1o849.telia.com>
In-Reply-To: <200112052155.fB5Ltxa26014@d1o849.telia.com>; from jakob.kemi@telia.com on Wed, Dec 05, 2001 at 10:54:26PM +0100
[-- Attachment #1: Type: text/plain, Size: 1178 bytes --]
On Wed, Dec 05, 2001 at 10:54:26PM +0100, Jakob Kemi wrote:
> Since 2.4.14 the ieee1284 parport functions have worked improperly
> for some drivers (CPiA, W9966 and others). Joe
> <joeja@mindspring.com> was quick to find the failure. He said he was
> going to send the patch to you some week ago and maybe he already
> have. If it somehow got lost in an internet black-hole or something
> I try to also send it to you. It's just a one-liner which reverts to
> pre-2.4.14 behavior.
I am the maintainer of this code. Why is this the first time that I
have seen this patch (or even heard of a problem report)? Marcelo,
please don't apply it.
The correct fix is to change:
ctl &= ~(PARPORT_CONTROL_STROBE | PARPORT_CONTROL_INIT);
so that it also turns off AUTOFD as well.
I have several patches queued up for parport (see
http://people.redhat.com/twaugh/patches/). It would be very helpful
to me if you could first send changes to me rather than Marcelo or
Linus, so that merging is not made more difficult. In addition, there
is a ChangeLog file in that directory; changing the code without
updating the ChangeLog makes my attempt to keep it current futile. :-(
Tim.
*/
[-- Attachment #2: Type: application/pgp-signature, Size: 232 bytes --]
next prev parent reply other threads:[~2001-12-05 22:18 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2001-12-05 18:41 Linux 2.4.17-pre4 Marcelo Tosatti
2001-12-05 21:54 ` Jakob Kemi
2001-12-05 22:18 ` Tim Waugh [this message]
2001-12-06 0:22 ` Jakob Kemi
2001-12-06 7:27 ` Tim Waugh
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20011205221812.U14028@redhat.com \
--to=twaugh@redhat.com \
--cc=jakob.kemi@telia.com \
--cc=linux-kernel@vger.kernel.org \
--cc=marcelo@conectiva.com.br \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox