On Thu, 11 Jun 2009 10:54:35 +0800 Greg KH <greg@xxxxxxxxx> wrote: > On Wed, Jun 10, 2009 at 12:23:58PM +0800, Alek Du wrote: > > >From 9bab93441d0969fe5f16ad8c8d61737e9eff02ff Mon Sep 17 00:00:00 2001 > > From: Alek Du <alek.du@xxxxxxxxx> > > Date: Wed, 10 Jun 2009 11:53:59 +0800 > > Subject: [PATCH] EHCI: Make ehci_qh structure items all 32bit aligned > > > > Intel Moorestown platform EHCI controller has internal SRAM using for > > caching of QH/QTD pools, but this SRAM has the 32bit alignment access > > limit, all SRAM access must be 32bit aligned. This patch will change > > only a few not u32 items to u32. > > Why not just reorginize the fields to not need any padding and provide > the same functionality? Just increasing the field size does bloat > things a bit, right? > > thanks, > > greg k-h Greg, I guess reorganizing order is tricky? Can compiler grant that? The qh is per EP, it won't waste much. I guess this way should be the simple way. If you look into drivers/usb/host/ehci.h, almost all data members are 32bits. Alek -- 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