[U-Boot] [PATCH V2 6/6] arm: mx5: Add support for DENX M53EVK

Marek Vasut
2013-04-25T14:54:40Z

Dear Stefano Babic,

> On 25/04/2013 15:36, Marek Vasut wrote:
> > Dear Stefano Babic,
>
> Hi Marek,
>
> >> At least add documentation in the README. It is only your choice if you
> >> will add two different targets to boards.cfg
> >
> > I'd prefer not to do that. We're currently booting from NAND and SD, so
> > we use u-boot.imx resp. u-boot-with-nand-spl.imx there. But using the
> > BOOT_FROM "nand" in the imximage.cfg is indeed wrong. We should focus on
> > fixing that.
> >
> > We can now pre-process the imximage.cfg with CPP, so why don't we just
> > replace BOOT_FROM "foobar" with BOOT_OFFSET CONFIG_IMX_BOOT_OFFSET
>
> +1
>
> > as mentioned by
> > someone in the thread already and setup the offset either in board config
> > file or boards.cfg and even have some default value?
>
> Agree

even better, CONFIG_IMX_BOOT_OFFSET can be defined somewhere to either of the
two values -- one for OneNAND and NOR and the other one for the rest. That'd
make it even easier for people to grok down.

I'll wait for general agreement and then possibly look into implementing it.

Best regards,
Marek Vasut