On Sun, Jun 21, 2015 at 6:56 AM, Christoph Hellwig <hch@xxxxxx> wrote: > On Sun, Jun 21, 2015 at 06:31:38AM -0700, Dan Williams wrote: >> > As mentioned last time only arch asm/ headers may include asm-generic >> > headers. >> >> No, not in this case, there's no other way to define readq()/writeq() >> on 32-bit builds. See: > > Oh my god. I think we're both right: no driver should use asm-generic, > but because someone totally messed this abstraction up you have no choice. > > We really should have a linux/*.h header for these that just does the > right thing. > > Btw, what's the reason you're using the hi-lo ordering unlikely everyone > else? IMHO that should be an arch and not a driver choice. If the hardware latches the register on writing the hi or lo bits first then it matters, otherwise it's arbitrary like it is in this case. It's hard to have an arch default because different devices care about different orderings, so it must be a driver choice afaics. -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in