]> git.dujemihanovic.xyz Git - u-boot.git/commitdiff
net: dc2114x: Use standard I/O accessors
authorMarek Vasut <marek.vasut+renesas@gmail.com>
Wed, 8 Jul 2020 04:31:54 +0000 (06:31 +0200)
committerMarek Vasut <marek.vasut+renesas@gmail.com>
Sat, 25 Jul 2020 09:24:02 +0000 (11:24 +0200)
The current dc21x4x driver accesses its memory mapped registers directly
instead of using the standard I/O accessors. This can cause problems on
some systems as the accesses can get out of order. So convert the direct
volatile dereferences to use the normal in/out macros.

Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>
Cc: Joe Hershberger <joe.hershberger@ni.com>
Cc: Ramon Fried <rfried.dev@gmail.com>
drivers/net/dc2114x.c

index dcbd818581eee93443ece07be2cb747f8cf62363..9cafa3b23dcf7c6cd582b3e9e49a66ea7cb0e6f6 100644 (file)
@@ -1,6 +1,7 @@
 // SPDX-License-Identifier: GPL-2.0+
 
 #include <common.h>
+#include <asm/io.h>
 #include <env.h>
 #include <malloc.h>
 #include <net.h>
@@ -104,12 +105,12 @@ static char tx_ring_size;
 
 static u32 dc2114x_inl(struct eth_device *dev, u32 addr)
 {
-       return le32_to_cpu(*(volatile u32 *)(addr + dev->iobase));
+       return le32_to_cpu(readl(dev->iobase + addr));
 }
 
 static void dc2114x_outl(struct eth_device *dev, u32 command, u32 addr)
 {
-       *(volatile u32 *)(addr + dev->iobase) = cpu_to_le32(command);
+       writel(cpu_to_le32(command), dev->iobase + addr);
 }
 
 static void reset_de4x5(struct eth_device *dev)