On 12/22/2010 02:23 PM, Jason Liu wrote:
This patch add the MX53 boot image support
This patch has been tested on Freescale MX53EVK board.
Signed-off-by: Jason Liu r64343@freescale.com
board/freescale/mx53evk/config.mk | 25 ++++++++ board/freescale/mx53evk/imximage.cfg | 108 ++++++++++++++++++++++++++++++++++ tools/imximage.c | 101 +++++++++++++++++++++++++++++-- tools/imximage.h | 75 ++++++++++++++++++++---
You mixed two patches with different meaning. You add support for mx53evk board (and patches for config.mk and imximage.cfg must belong to this patch) and you modify imximage to support the MX53 processor, not related to the mx53evk board. Please split this patch into two separate patches.
diff --git a/tools/imximage.c b/tools/imximage.c index 39f89c2..884feff 100644 --- a/tools/imximage.c +++ b/tools/imximage.c @@ -82,7 +82,7 @@ static int imximage_check_image_types(uint8_t type) static int imximage_verify_header(unsigned char *ptr, int image_size, struct mkimage_params *params) {
+#ifdef CONFIG_MX51
NAK. This is wrong: mkimage is a tool running on host and must be possible to include it in a distro. It must be able (as it now does) to produce the correct image at runtime. For this reason there are no #ifdef in the actual code. If a different behavior is required, this must be added extending the syntax of the imximage.cfg file, for example adding a processor type (or a version number, or whatever is needed).
+#if defined(CONFIG_MX51) static void imximage_print_header(const void *ptr) { struct imx_header *imx_hdr = (struct imx_header *) ptr; flash_header_t *hdr = &imx_hdr->fhdr; uint32_t size; uint32_t length;
- dcd_t *dcd = &imx_hdr->dcd_table;
- dcd_t *dcd = &imx_hdr->dcd_table;
I do not see any changes on these lines. Please add only changes where they are required.
Best regards, Stefano Babic