diff mbox series

[U-Boot] net: sh-eth: Fix misaligned cache operation warning

Message ID 20171201045608.23132-1-iwamatsu@nigauri.org
State Accepted
Commit b621534
Delegated to: Joe Hershberger
Headers show
Series [U-Boot] net: sh-eth: Fix misaligned cache operation warning | expand

Commit Message

Nobuhiro Iwamatsu Dec. 1, 2017, 4:56 a.m. UTC
When we using network on board using sh-eth, it prints a lot of
"CACHE: Misaligned operation at range" messages.
This commit fixes this problem.

Signed-off-by: Nobuhiro Iwamatsu <iwamatsu@nigauri.org>
---
 drivers/net/sh_eth.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Joe Hershberger Dec. 5, 2017, 8:47 p.m. UTC | #1
On Thu, Nov 30, 2017 at 10:56 PM, Nobuhiro Iwamatsu
<iwamatsu@nigauri.org> wrote:
> When we using network on board using sh-eth, it prints a lot of
> "CACHE: Misaligned operation at range" messages.
> This commit fixes this problem.
>
> Signed-off-by: Nobuhiro Iwamatsu <iwamatsu@nigauri.org>

Acked-by: Joe Hershberger <joe.hershberger@ni.com>
Joe Hershberger Jan. 22, 2018, 4:54 p.m. UTC | #2
Hi Nobuhiro,

https://patchwork.ozlabs.org/patch/843323/ was applied to http://git.denx.de/?p=u-boot/u-boot-net.git

Thanks!
-Joe
diff mbox series

Patch

diff --git a/drivers/net/sh_eth.c b/drivers/net/sh_eth.c
index cb000dd18e..b7a0f413c6 100644
--- a/drivers/net/sh_eth.c
+++ b/drivers/net/sh_eth.c
@@ -29,7 +29,7 @@ 
 
 #if defined(CONFIG_SH_ETHER_CACHE_WRITEBACK) && !defined(CONFIG_SYS_DCACHE_OFF)
 #define flush_cache_wback(addr, len)    \
-		flush_dcache_range((u32)addr, (u32)(addr + len - 1))
+		flush_dcache_range((u32)addr, (u32)(addr + ALIGN(len, CONFIG_SH_ETHER_ALIGNE_SIZE)))
 #else
 #define flush_cache_wback(...)
 #endif
@@ -205,7 +205,7 @@  static int sh_eth_tx_desc_init(struct sh_eth_dev *eth)
 		goto err;
 	}
 
-	flush_cache_wback((u32)port_info->tx_desc_alloc, alloc_desc_size);
+	flush_cache_wback(port_info->tx_desc_alloc, alloc_desc_size);
 
 	/* Make sure we use a P2 address (non-cacheable) */
 	port_info->tx_desc_base =