mtd: diskonchip: use inline functions for DocRead/DocWrite
authorArnd Bergmann <arnd@arndb.de>
Tue, 6 Nov 2012 21:55:27 +0000 (22:55 +0100)
committerArtem Bityutskiy <artem.bityutskiy@linux.intel.com>
Fri, 16 Nov 2012 08:57:38 +0000 (10:57 +0200)
commitd611d41b46c96195b9a168a21992782458826e07
tree6b3acda5c25d5f424b07c85351e66da3dc6dddb0
parentca796f85901880c1247e19053d70b640f996813e
mtd: diskonchip: use inline functions for DocRead/DocWrite

The diskonchip drivers traditionally use home-grown macros for
doing MMIO accesses, which cause a lot of warnings, at least
on ARM machines:

drivers/mtd/devices/doc2000.c: In function 'doc_write':
drivers/mtd/devices/doc2000.c:854:5: warning: value computed is not used [-Wunused-value]
drivers/mtd/devices/doc2000.c: In function 'doc_erase':
drivers/mtd/devices/doc2000.c:1123:5: warning: value computed is not used [-Wunused-value
drivers/mtd/nand/diskonchip.c: In function 'doc2000_read_byte':
drivers/mtd/nand/diskonchip.c:318:3: warning: value computed is not used [-Wunused-value]

A nicer solution is to use the architecture-defined I/O accessors.
Here, we use the __raw_readl/__raw_writel style, instead of the
proper readl/writel ones, in order to preserve the odd semantics
of the existing macros that have their own barrier implementation
and no byte swap. It would be nice to fix this properly and use
the correct accessors as well as make the word size independent
from the architecture, but I guess the hardware is obsolete
enough that we should better not mess the driver an more than
necessary.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
include/linux/mtd/doc2000.h