[U-Boot] i.MX6 Code Consolidation suggestion/question

Fabio Estevam festevam at gmail.com
Mon Feb 25 13:52:39 UTC 2019


Hi Adam,

On Mon, Feb 25, 2019 at 10:09 AM Adam Ford <aford173 at gmail.com> wrote:
>
> I am looking into the SPL code to boot the imx6_logic board and
> comparing its board file to other i.MX6 boards.
>
> Nearly every board has a virtually identical board_init_f function.
> Some of the entries are done is slightly different orders, but what
> happens appears to be the same.  I don't know if there is an advantage
> to one order or another.
>
> We have a common mach-imx/spl.c file and the very end there exisrts an
> MX6-only ifdef with the dram_init_banksize function in side the ifdef.
>
> Any objections to my creating a weak function called board_init_f
> inside there, then wiping out all the individual board entries that
> are identical?  For those who have a slightly different variation,
> they can be left alone.
>
> It seems logical to me, but I don't have access to everyone's
> hardware, so before I do that, I thought I'd ask first.

Yes, this seems like a good idea.

Thanks


More information about the U-Boot mailing list