From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Babic Date: Tue, 21 Feb 2012 18:57:03 +0100 Subject: [U-Boot] [PATCH] imximage: header v2: Remove overwriting of flash_offset In-Reply-To: <1329814920-12295-1-git-send-email-dirk.behme@de.bosch.com> References: <1329814920-12295-1-git-send-email-dirk.behme@de.bosch.com> Message-ID: <4F43DAEF.8050907@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 On 21/02/2012 10:02, Dirk Behme wrote: > The flash header v2 supports different flash offsets for different > boot devices. E.g. parallel NOR or OneNAND use a different offset > than FLASH_OFFSET_STANDARD (== 0x400). > > The flash offset is correctly read from the configuration in > parse_cfg_cmd(). But is then overwritten wrongly in set_imx_hdr_v2(). > > Fix this by removing this overwriting. Use the flash offset > correclty read from the configuration, instead. > > Signed-off-by: Dirk Behme > CC: Jason Liu > CC: Stefano Babic > --- Hi Dirk, > tools/imximage.c | 3 --- > 1 files changed, 0 insertions(+), 3 deletions(-) > > diff --git a/tools/imximage.c b/tools/imximage.c > index 1e0f5d4..e9947f1 100644 > --- a/tools/imximage.c > +++ b/tools/imximage.c > @@ -253,9 +253,6 @@ static void set_imx_hdr_v2(struct imx_header *imxhdr, uint32_t dcd_len, > imx_header_v2_t *hdr_v2 = &imxhdr->header.hdr_v2; > flash_header_v2_t *fhdr_v2 = &hdr_v2->fhdr; > > - /* Set default offset */ > - imxhdr->flash_offset = FLASH_OFFSET_STANDARD; > - > /* Set magic number */ > fhdr_v2->header.tag = IVT_HEADER_TAG; /* 0xD1 */ > fhdr_v2->header.length = cpu_to_be16(sizeof(flash_header_v2_t)); I think we have then a problem when there is not a BOOT_FROM statement in the configuration file, that let imxhdr->flash_offset unset. We need to set it with the default value at the beginning of the processing or to raise an error if we want to make this statement mandatory in the configuration file. Best regards, Stefano Babic -- ===================================================================== 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 =====================================================================