Re: HID issues. [ with miniPATCH ]

[Bj|rn A. <d3a...@dt...>]

Quoting Vojtech Pavlik <vo...@su...>:
> On Sat, Jan 26, 2002 at 08:13:26PM +0100, Bj|rn Augustsson wrote:
> > Quoting Vojtech Pavlik <vo...@su...>:
> > > 
> > > Maybe update, there was a serious rewrite of that part.
> > 
> > I'm trying! :)
> > 
> > I haven't gotten anything since that to build and boot, that's all...
> 
> Use 2.5.2-dj6 if you can, that's probably the easiest (and working as
> well).

Good advice, it turns out. A few hours later, I have a working (?)
kernel, by getting dj6 and then replacing _only_ the
linux/drivers/usb/hid* files with the stuff from the CVS. 

Another question!

How come you set the output interrupt pipe to be a bulk pipe? That seems 
obviously wrong, and it wasn't that way in my patch. 
Do you know something I don't?  
(It had a "/* FIXME should we use sndint here? */" after it, which was 
removed by 1.39)

I've attached a mini-patch as well, that fixes another thing. We only
want to send outbound output reports over the output pipe - feature
reports still go over the control pipe.

/August.
-- 
Wrong on most accounts.  const Foo *foo; and Foo const *foo; mean the same: foo
being a pointer to const Foo.  const Foo const *foo; would mean the same but is
illegal (double const).  You are confusing this with Foo * const foo; and const
Foo * const foo; respectively. -David Kastrup, comp.os.linux.development.system