Ordered I/O accessors
Get series with b4
b4 am '002d01cb2e55$bf328c90$3d97a5b0$@[email protected]'Versions newest first
- v3 Latest0 of 3 patches 0 repliesNew
Patches in v3 No applies recorded yet
None of the 3 patches the cover letter announces are in this archive.
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v3 yet.
Cover letter [PATCH v3 0/3] Ordered I/O accessors
Hi Russell, Rabin,
Quoted 10 lines · Russell King - ARM Linux
Quoted 9 lines · Catalin Marinas
Quoted 5 lines
This version replaces the do { ... } while (0) constructs in the final patch with ({ ... }) to avoid errors in drivers putting brackets around the write*() accessors. To avoid the write*() macro having a non-void type (and possibly the compiler generating code to read a register), the first patch explicitly casts the __raw_write*() accessors to (void).
Ok. Can you please put this in the patch system? I think we need to get this in prior to 2.6.35 being released.
Catalin's away at the moment and I'm not sure if he's picking up mail.
Quoted 6 lines
ux500 is using
writel()insideuncompress.h, so this series triggers a build failure there:
arch/arm/boot/compressed/misc.o: In function `putc':
arch/arm/mach-ux500/include/mach/uncompress.h:41:
undefined reference to `outer_cache'
There's also an issue with the iowrite* macros, which need a (void) cast in the same way as the write* guys.
Will