OSDN Git Service

FDDI: defza: Include linux/io-64-nonatomic-lo-hi.h
authorPaul Burton <paul.burton@mips.com>
Thu, 20 Jun 2019 22:13:58 +0000 (22:13 +0000)
committerPaul Burton <paul.burton@mips.com>
Mon, 24 Jun 2019 21:14:49 +0000 (14:14 -0700)
commit262e4c3893aca09d1749790880d6b9bdb047c0b9
tree417e1ebf9396a372c3d0df903a007fef3989a5a2
parent65eb3e4c142603b6b75618c4588d7748079ff3b6
FDDI: defza: Include linux/io-64-nonatomic-lo-hi.h

Currently arch/mips/include/asm/io.h provides 64b memory accessor
functions such as readq & writeq even on MIPS32 platforms where those
accessors cannot actually perform a 64b memory access. They instead
BUG(). This is unfortunate for drivers which either #ifdef on the
presence of these accessors, or can function with non-atomic
implementations of them found in either linux/io-64-nonatomic-lo-hi.h or
linux/io-64-nonatomic-hi-lo.h. As such we're preparing to remove the
definitions of these 64b accessor functions for MIPS32 kernels.

In preparation for this, include linux/io-64-nonatomic-lo-hi.h in
defza.c in order to provide a non-atomic implementation of the
readq_relaxed & writeq_relaxed functions that are used by this code. In
practice this will have no runtime effect, since use of the 64b accessor
functions is conditional upon sizeof(unsigned long) == 8, ie. upon
CONFIG_64BIT=y. This means the calls to these non-atomic readq & writeq
implementations will be optimized out anyway, but we need their
definitions to keep the compiler happy.

For 64bit kernels using this code this change should also have no effect
because asm/io.h will continue to provide the definitions of
readq_relaxed & writeq_relaxed, which linux/io-64-nonatomic-lo-hi.h
checks for before defining itself.

Signed-off-by: Paul Burton <paul.burton@mips.com>
Acked-by: Maciej W. Rozycki <macro@linux-mips.org>
Acked-by: David S. Miller <davem@davemloft.net>
Cc: Serge Semin <Sergey.Semin@t-platforms.ru>
Cc: netdev@vger.kernel.org
Cc: linux-mips@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
drivers/net/fddi/defza.c