On Wed, Mar 23, 2011 at 07:48:56AM +1100, Benjamin Herrenschmidt wrote: > On Tue, 2011-03-22 at 07:07 -0700, Greg KH wrote: > > On Tue, Mar 22, 2011 at 05:26:01PM +1100, Benjamin Herrenschmidt wrote: > > > Hi Greg ! > > > > > > I recently noticed that the output from lsusb was utterly wrong on PPC > > > for 16-bit fields (like bad endianess :) > > > > > > Digging a bit I noticed a couple of things: > > > > > > - My distro (ubuntu, yeah I know how much you love them) variant of > > > lsusb does: > > > > > > if (__BYTE_ORDER == __LITTLE_ENDIAN) > > > #define le16_to_cpu(x) (x) > > > #elif (__BYTE_ORDER == __BIG_ENDIAN) > > > #define le16_to_cpu(x) bswap_16(x) > > > #else > > > #error missing BYTE_ORDER > > > #endif > > > > What version of usbutils is this? > > Ubuntu says 0.87-4. Once you've "fixed" it properly upstream, I can > point them (and debian while at it) to it and get them to "backport" the > fix. Oh that's a very old version, I think we fixed it a while ago, but can't find the exact commit. The problem is that for version 001 we switched to use libusb1 so they can't just grab the latest version unless they also have that library in their distro. Go file a bug, it's not upstream's problem :) best of luck, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
