On Fri, Jan 21, 2005 at 07:49:08AM +0000, David Woodhouse wrote: > On Thu, 2005-01-20 at 16:08 -0800, Greg KH wrote: > > Doh, sorry for missing this one. I've applied your patch to my trees, > > and will show up in the next -mm release. > > Actually I think John's problem was that the usb core code has now > _stopped_ doing this byteswapping, and he has a lsusb which is hacked to > expect it. So if you apply my patch you're preserving the userspace ABI > by reverting to the extremely stupid behaviour of byteswapping _some_ of > the fields in the descriptor we pass to userspace.
Well, we should be byteswapping all of the fields that need to be swapped, right? I'm guessing that userspace is expecting the fields to be in cpu endian, correct? But if you want, I'll gladly revert your patch, as I don't have a ppc box to test this out on. thanks, greg k-h - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/