From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefan Roese Date: Mon, 30 Sep 2013 08:37:56 +0200 Subject: [U-Boot] [PATCH 1/2] SPL: Add CONFIG_SPL_BOOTCOUNT_SUPPORT In-Reply-To: <1380309969-5618-2-git-send-email-trini@ti.com> References: <1380309969-5618-1-git-send-email-trini@ti.com> <1380309969-5618-2-git-send-email-trini@ti.com> Message-ID: <52491C44.80906@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Tom, On 27.09.2013 21:26, Tom Rini wrote: > Add a new symbol, CONFIG_SPL_BOOTCOUNT_SUPPORT, to make use of the > existing BOOTCOUNT_SUPPORT within SPL. It is strongly discouraged to > use bootcount in both SPL and full U-Boot, as they use the same counter. I just noticed that I missed sending the SPL bootcount implementation I did a few weeks ago to the list. I'll send it shortly. The main differences I see right now are: - My implementation requires CONFIG_BOOTCOUNT_LIMIT and CONFIG_SPL_BOOTCOUNT_SUPPORT to be configured - The check is not added to the board-specific code, but the medium code (here spl_nor.c). This might be consolidated even more by moving it to a non-boot-medium specific location (spl.c)? Please take a look at it and let me know if which way to go. Thanks, Stefan