From mboxrd@z Thu Jan 1 00:00:00 1970 From: Frank Kunz Date: Fri, 03 Nov 2017 18:01:20 +0100 Subject: [U-Boot] [PATCH 1/2] sunxi: binman: Add U-Boot binary size check In-Reply-To: <20171103084244.6avbjya53lj74et3@flea> References: <20171019140419.30996-1-maxime.ripard@free-electrons.com> <1714071.5BEO5OyD5H@donald> <20171103084244.6avbjya53lj74et3@flea> Message-ID: <4091367.y9zEdsn68M@donald> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Am Freitag, 3. November 2017, 09:42:44 CET schrieb Maxime Ripard: > Hi Frank, > > On Thu, Nov 02, 2017 at 04:53:54PM +0100, Frank Kunz wrote: > > Am Donnerstag, 19. Oktober 2017, 16:04:18 CET schrieb Maxime Ripard: > > > The U-boot binary may trip over its actual allocated size in the > > > storage. > > > In such a case, the environment will not be readable anymore (because > > > corrupted when the new image was flashed), and any attempt at using > > > saveenv > > > to reconstruct the environment will result in a corrupted U-boot binary. > > > > > > Signed-off-by: Maxime Ripard > > > --- > > > > > > arch/arm/dts/sunxi-u-boot.dtsi | 11 +++++++++++ > > > 1 file changed, 11 insertions(+) > > > > I found that this patch causes the sunxi-fel tool to fail with: > > > > sunxi-fel uboot u-boot-sunxi-with-spl.bin > > U-Boot image data size mismatch: expected 516032, got 389212 > > > > Is there a patch for the sunxi tools missing? > > No, because no one reported it so far. > > Can you test with: > http://code.bulix.org/mb0ic7-221765?raw > That works. Tested-by: Frank Kunz