Re: [patch] rio500 devfs support

2001-07-01 Thread Gregory T. Norris
It's working beautifully here. I'll forward the patch to the maintainer, since I have no idea if he's seen this thread. Cheers! PGP signature

Re: [patch] rio500 devfs support

2001-06-25 Thread Greg KH
On Mon, Jun 25, 2001 at 05:12:01PM -0500, Gregory T. Norris wrote: > I was thinking of doing some similar updates this evening after work. > Darn it... now I have to find something else to do! :-) > > Going by this morning's comments from Richard Gooch, it sounds like the > > if (rio->de

Re: [patch] rio500 devfs support

2001-06-25 Thread Gregory T. Norris
I was thinking of doing some similar updates this evening after work. Darn it... now I have to find something else to do! :-) Going by this morning's comments from Richard Gooch, it sounds like the if (rio->devfs == NULL) dbg("probe_rio: device node registration failed"

Re: [patch] rio500 devfs support

2001-06-25 Thread Greg KH
On Tue, Jun 19, 2001 at 05:52:24PM -0500, Gregory T. Norris wrote: > The attached diff adds devfs support to the rio500 driver, so that > /dev/usb/rio500 gets created automagically. It was generated against > 2.4.5, but probably applies fine against any recent kernel. Comments > are welcome (but

Re: [patch] rio500 devfs support

2001-06-25 Thread Gregory T. Norris
Ok, back to the original version then. Thanx! On Mon, Jun 25, 2001 at 12:16:51AM -0600, Richard Gooch wrote: > No, it's a bad idea to test the error from devfs_register() unless you > *really* have a good reason. Most people who think they have a good > reason actually don't, they're just confus

Re: [patch] rio500 devfs support

2001-06-24 Thread Richard Gooch
Gregory T. Norris writes: > > --qlTNgmc+xy1dBmNv > Content-Type: multipart/mixed; boundary="0F1p//8PRICkK4MW" > Content-Disposition: inline Yuk! MIME! > --0F1p//8PRICkK4MW > Content-Type: text/plain; charset=us-ascii > Content-Disposition: inline > Content-Transfer-Encoding: quoted-printable H

Re: [patch] rio500 devfs support

2001-06-24 Thread Gregory T. Norris
Here's an updated version of the patch - the only real difference is that rio500.c will printk an error message if devfs_register() fails. I left that out originally because devfs logs the error, but it's probably a good idea to indicate which driver made the request. Cheers! On Tue, Jun 19, 20