On Sun, Aug 14, 2016 at 10:47 AM, Tom Rini <tr...@konsulko.com> wrote: > On Sun, Aug 14, 2016 at 03:03:16PM +0000, Karl Beldan wrote: > >> check_cache_range() warns that the top boundaries are not properly >> aligned while flushing and invalidating the buffers and make these >> operations to fail. >> ATM the RX bottom boundaries are aligned by design with EMAC_RXBUF_SIZE, >> properly aligned with ARCH_DMA_MINALIGN, however the top ones are not. >> >> This gets rid of the warnings: >> CACHE: Misaligned operation at range >> >> Signed-off-by: Karl Beldan <karl.beldan+...@gmail.com> >> --- >> drivers/net/davinci_emac.c | 6 ++++-- >> 1 file changed, 4 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/net/davinci_emac.c b/drivers/net/davinci_emac.c >> index 947bfab..55461b0 100644 >> --- a/drivers/net/davinci_emac.c >> +++ b/drivers/net/davinci_emac.c >> @@ -632,7 +632,8 @@ static int davinci_eth_send_packet (struct eth_device >> *dev, >> EMAC_CPPI_EOP_BIT); >> >> flush_dcache_range((unsigned long)packet, >> - (unsigned long)packet + length); >> + round_up((unsigned long)packet + length, >> + ARCH_DMA_MINALIGN)); > > It's preferred to use: > (unsigned long)packet + ALIGN(length, PKTALIGN)); > here instead of ARCH_DMA_MINALIGN.
Correct, please resend. Thanks, -Joe _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
