* Re: hotplug-2001_01_23 and kernel 2.4.1
2001-01-31 22:41 hotplug-2001_01_23 and kernel 2.4.1 Greg KH
@ 2001-02-01 1:06 ` David Brownell
2001-02-01 1:54 ` David Brownell
2001-02-02 0:49 ` Gordon Lack
2 siblings, 0 replies; 4+ messages in thread
From: David Brownell @ 2001-02-01 1:06 UTC (permalink / raw)
To: linux-hotplug
[-- Attachment #1: Type: text/plain, Size: 1953 bytes --]
Try this patch. If it works, it should go to Linus.
It's not so much that things got switched, as that there are
two mechanisms for specifying device/interface type and they
don't strictly overlap (according to my brief investigation).
Some printers use one mechanism (interface typing), others
use another (device typing).
- Dave
----- Original Message -----
From: Greg KH <greg@kroah.com>
To: Gordon Lack <gmlack@freenet.co.uk>
Cc: <linux-hotplug-devel@lists.sourceforge.net>; <linux-usb-devel@lists.sourceforge.net>
Sent: Wednesday, January 31, 2001 2:41 PM
Subject: Re: hotplug-2001_01_23 and kernel 2.4.1
> I think there was a patch to the printer driver to fix this, but I don't
> remember, does anyone on linux-usb-devel or linux-hotplug see this same
> thing (I don't have a USB printer handy at the moment.)
>
> greg k-h
>
> On Wed, Jan 31, 2001 at 10:07:22PM +0000, Gordon Lack wrote:
> > I've just combined these 2.
> >
> > Bascially Ok, but one problem...
> >
> > I have an Epson Stylus 740 printer on USB. The kernel calls out to
> > hotplug with a TYPE os 7/1/2 for this (and hence no INTERFACE value).
> >
> > So, the scripts set the 3 interface values to 1000 (out of range).
> >
> > This doesn't match any of the 3 printer entries in
> > /lib/modules/2.4.1/modules.usbconf. *Those* entries have Interface
> > settings of 7/1/1, 7/1/2 and 7/1/3, but no Device settings.
> >
> > So it looks to me as though something has swaped the meaning of the
> > Interface and Device settings between whatever produces the
> > modules.usbconf files (if this is modutils then I've just upgrade to
> > 2.4.2) and what your hotplug scripts expect.
> >
> > If I swap the two pairs of 3 fields arround in the "while read ..."
> > line of the usb.agent script then all is Ok.
> >
> >
> > I hope this makes sense and is useful....
> >
> > Thanks....
> > Gordon Lack
>
> --
> greg@(kroah|wirex).com
>
[-- Attachment #2: printer-pre3.patch --]
[-- Type: application/octet-stream, Size: 622 bytes --]
--- 2.4.0-test12/drivers/usb/printer.c Tue Dec 12 10:20:37 2000
+++ linux/drivers/usb/printer.c Tue Dec 19 15:10:57 2000
@@ -613,6 +613,9 @@
};
static struct usb_device_id usblp_ids [] = {
+ { bDeviceClass: 7, bDeviceSubClass: 1, bDeviceProtocol: 1},
+ { bDeviceClass: 7, bDeviceSubClass: 1, bDeviceProtocol: 2},
+ { bDeviceClass: 7, bDeviceSubClass: 1, bDeviceProtocol: 3},
{ bInterfaceClass: 7, bInterfaceSubClass: 1, bInterfaceProtocol: 1},
{ bInterfaceClass: 7, bInterfaceSubClass: 1, bInterfaceProtocol: 2},
{ bInterfaceClass: 7, bInterfaceSubClass: 1, bInterfaceProtocol: 3},
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: hotplug-2001_01_23 and kernel 2.4.1
2001-01-31 22:41 hotplug-2001_01_23 and kernel 2.4.1 Greg KH
2001-02-01 1:06 ` David Brownell
2001-02-01 1:54 ` David Brownell
@ 2001-02-02 0:49 ` Gordon Lack
2 siblings, 0 replies; 4+ messages in thread
From: Gordon Lack @ 2001-02-02 0:49 UTC (permalink / raw)
To: linux-hotplug
[-- Attachment #1: Type: text/plain, Size: 559 bytes --]
David Brownell wrote:
>
> > Try this patch. If it works, it should go to Linus.
>
> ... but it won't, it needs to use the new macros that got
> introduced in the 2.4.0 "final" kernel. That one was
> against test12-pre3, which doesn't have them. A working
> version should go to Linus, that much is true! :-)
Well, it seems to work at 2.4.1 once (trivially) converted to the new
macros. \switching on the printer now gets the printer module loaded.
Thanks. Patch against 2.1.4 appended (I've just got the modified and
original file in one dir).
[-- Attachment #2: printer-2.4.1.patch --]
[-- Type: text/plain, Size: 356 bytes --]
--- printer.c.orig Thu Jan 4 21:15:32 2001
+++ printer.c Fri Feb 2 00:24:52 2001
@@ -613,6 +613,9 @@
};
static struct usb_device_id usblp_ids [] = {
+ { USB_DEVICE_INFO(7, 1, 1) },
+ { USB_DEVICE_INFO(7, 1, 2) },
+ { USB_DEVICE_INFO(7, 1, 3) },
{ USB_INTERFACE_INFO(7, 1, 1) },
{ USB_INTERFACE_INFO(7, 1, 2) },
{ USB_INTERFACE_INFO(7, 1, 3) },
^ permalink raw reply [flat|nested] 4+ messages in thread