On Wed, Apr 29, 2009 at 08:43:12AM -0700, Sarah Sharp wrote: > On Tue, Apr 28, 2009 at 09:36:06AM +0200, Oliver Neukum wrote: > > Am Dienstag 28 April 2009 04:52:22 schrieb Sarah Sharp: > > > +/* TODO: copied from ehci.h - can be refactored? */ > > > +/* xHCI spec says all registers are little endian */ > > > +static inline unsigned int xhci_readl(const struct xhci_hcd *xhci, > > > + __u32 __iomem *regs) > > > +{ > > > + return readl(regs); > > > +} > > > +static inline void xhci_writel(const struct xhci_hcd *xhci, > > > + const unsigned int val, __u32 __iomem *regs) > > > +{ > > > + if (!in_interrupt()) > > > + xhci_dbg(xhci, "`MEM_WRITE_DWORD(3'b000, 32'h%0x, 32'h%0x, 4'hf);\n", > > > + (unsigned int) regs, val); > > > + writel(val, regs); > > > +} > > > > No 64 bit version? > > I was told by Greg that 64-bit reads on the PCIe bus had issues on some > platforms. Greg, is that still true? I think so, yes. But have not tried it in a long time, so it might be resolved now. thanks, 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