From mboxrd@z Thu Jan  1 00:00:00 1970
From: Simon Goldschmidt <simon.k.r.goldschmidt@gmail.com>
Date: Wed, 21 Nov 2018 16:48:59 +0100
Subject: [U-Boot] [swupdate] Re: SWUpdate - U-Boot environment library
 dependency
In-Reply-To: <20181121154139.A7197240403@gemini.denx.de>
References: <20180904102959.GA1766@iiotirae>
 <90014ca9-64d8-8e38-a6c0-4c06e991c92c@gmail.com>
 <20181121083121.629FC240403@gemini.denx.de>
 <6b07725e-51ba-7309-1506-e7211487b947@denx.de>
 <CAAh8qszr_4bxvdaLgT3ebhhFV4-g_xb8j37QEB6ot6-2ZmEkkQ@mail.gmail.com>
 <9efc8990-118c-d5b9-802d-8190db2326d3@denx.de>
 <20181121114520.0DAF9240403@gemini.denx.de>
 <7089ef62-ed0f-87f4-e979-8c18a6ae4b62@denx.de>
 <20181121142749.E7986240403@gemini.denx.de>
 <CAAh8qsyopV-HftdCWMbMu+fdcpqWWQtBnZDDyqFyv6iC1eoCDA@mail.gmail.com>
 <20181121154139.A7197240403@gemini.denx.de>
Message-ID: <e7751e81-610b-9cdd-e14d-04fc5b7a297f@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

On 21.11.2018 16:41, Wolfgang Denk wrote:
> Dear Simon,
>
> In message <CAAh8qsyopV-HftdCWMbMu+fdcpqWWQtBnZDDyqFyv6iC1eoCDA@mail.gmail.com> you wrote:
>>> Only if you use signed images.  With plain U-Boot, there is not even
>>> a checksum for it...
>> When SPL loads U-Boot from a legacy image, isn't there a CRC involved over
>> the full image including the environment?
> Yes, but when - for example - the compiled in default environment
> gets corrupted while loading from the storage device or while
> writing it to RAM, there will be zero check when reading it.

Hmm, on my board, SPL loads U-Boot from NOR to SDRAM, then checks the 
uimage CRC. I'd say this is enough checks. That might not hold for all 
boards though.

But I still think keeping the default environment in a different place 
would be good.

Could we put U-Boot into a fit image and put the environment in a 
subimage? But it might increase SPL code size when a fit image has to be 
parsed...

Regards,
Simon