From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Babic Date: Fri, 08 Oct 2010 18:03:59 +0200 Subject: [U-Boot] [PATCH] Crash in env_relocate_spec() of env_mmc.c In-Reply-To: References: <1286525010-13277-1-git-send-email-sbabic@denx.de> Message-ID: <4CAF40EF.8090009@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 Steve Sakoman wrote: > On Fri, Oct 8, 2010 at 1:03 AM, Stefano Babic wrote: >> mmc_initialize is not called at the startup if the >> relocation takes place and the environment is stored >> into a MMC card. >> Hallo Steve, > I think it might make more sense to put the MMC ifdef after the > onenand code so that it doesn't split the nand/onenand grouping. > Otherwise it matches what I did. Yes, it makes more sens - and I do not see any problem to move it > > This is a void function and shouldn't have a return value. I fixed > this in my version. Agree. > >> gd->env_valid = 1; > > I removed this in my version, probably an error to do that though :-) Or probably not...env_valid should signalize if the checksum is correct, and now the checksum is verified by the env_import function, as I have understood. > I previously submitted a patch to fix this and Wolfgang sent an email > saying that it had been applied. Ah, ok, I have missed it ;-) > I'll revise my patch to add the missing gd->env_valid = 1; and re-test. Ok, thanks Stefano -- ===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de =====================================================================