[Note: I used an old invalid email address for Dmitry in the original post.]
On Wed, 6 Dec 2006, Marcel Holtmann wrote: > Hi Alan, > > > Considering all the discussion whirling around about HID changes, it > > seemed like a good idea to let you know about some proposed patches coming > > up. > > > > They are below. The first addresses a problem that Pete has mentioned in > > the past: Some keyboards sometimes send more data than a single report in > > a packet, leading to EOVERFLOW errors. The patch tries to fix things by > > using a transfer length at least as large as the maxpacket size. I'm not > > sure if this will fully fix the problem, though -- if the device actually > > does stick more than one report in a single packet then the driver will > > need to parse them all, and the patch doesn't try to do that. > > > > The second patch addresses a problem caused by some KVM switches: They > > disrupt the data path between the host and the device (even when not > > switched away). The patch adds a module parameter to usbhid, telling it > > that a KVM is present so it should ignore certain types of errors. This > > essentially restores the behavior we had back in 2.6.16. > > both patches are quite simple. I think we should do the split first and > then apply these two patches on top of it. That's okay with me. I wasn't going to submit them right away. Actually I'm still interested in getting feedback on the patches themselves. In particular, do we need to handle packets containing more than one report? Alan Stern ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ linux-usb-devel@lists.sourceforge.net To unsubscribe, use the last form field at: https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
