From mboxrd@z Thu Jan  1 00:00:00 1970
From: Fabio Estevam <festevam@gmail.com>
Date: Wed, 3 Aug 2016 15:44:40 -0300
Subject: [U-Boot] [PATCH] mtd: nand: mxs: fix cache alignment for cache
 lines >32
In-Reply-To: <CAPnjgZ2CvuMrDvdDt+8pt1wwOGu7zEqruzyf6mYa2e0=bVVX4A@mail.gmail.com>
References: <20160802065518.24140-1-stefan@agner.ch>
 <CAOMZO5C8eSkDsT9eiU8eW4Lthsk+g+1evWh7EVWJw4x1gwuERw@mail.gmail.com>
 <25dc4a8aeadcdc3698ce4f59b619dc01@agner.ch>
 <CAOMZO5CSQC6+5DSv-NDm34wGQC-nZQy82y5po4wpkvbJRZkP1w@mail.gmail.com>
 <20160803180908.GA9942@bill-the-cat>
 <CAOMZO5CQ1sco-tag=NUqgxKo8Y1LtF=Dt-hmpEQfeNYQUmAdKg@mail.gmail.com>
 <CAPnjgZ2CvuMrDvdDt+8pt1wwOGu7zEqruzyf6mYa2e0=bVVX4A@mail.gmail.com>
Message-ID: <CAOMZO5BD-XCO9OE4oeVs7EvHnXMoZE7dJ2t-z25P7vLD5BCQJQ@mail.gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Simon,

On Wed, Aug 3, 2016 at 3:35 PM, Simon Glass <sjg@chromium.org> wrote:

> Actually I think these are bugs and should be fixed. In this case,
> from what I can tell netboot_common() should cache-align the size in
> the call to:
>
> /* flush cache */
> flush_cache(load_addr, size);

Do you mean like this?

--- a/cmd/net.c
+++ b/cmd/net.c
@@ -244,6 +244,8 @@ static int netboot_common(enum proto_t proto, cmd_tbl_t *cmd
        }

        /* flush cache */
+       load_addr &= ~(CONFIG_SYS_CACHELINE_SIZE - 1);
+       size = ALIGN(size, CONFIG_SYS_CACHELINE_SIZE);
        flush_cache(load_addr, size);

        bootstage_mark(BOOTSTAGE_ID_NET_LOADED);

This makes the net warnings go away.

There is still this one that I am seeing:

Kernel image @ 0x80800000 [ 0x000000 - 0x6fea70 ]
## Flattened Device Tree blob at 83000000
   Booting using the fdt blob at 0x83000000
   Using Device Tree in place at 83000000, end 83009c5d

Starting kernel ...

CACHE: Misaligned operation at range [00900000, 00900529]
[    0.000000] Booting Linux on physical CPU 0x0

Any ideas where it may come from?

Thanks