[U-Boot] [PATCH] imx: i.mx6q: add the initial support for LogicPD i.MX6Q SOM

[Adam Ford]

On Wed, Mar 22, 2017 at 7:27 PM, Tom Rini <trini at konsulko.com> wrote:
> On Wed, Mar 22, 2017 at 05:07:02PM -0500, Adam Ford wrote:
>
>> Logic PD has an i.MX6Q system on module (SOM) with a development kit.
>> This have been verified to boot the i.MX6Q version over either SD
>> on the development kit or NAND built into the SOM.
>>

[snip]

>
>> +     "fdt_high=0xffffffff\0"   \
>> +     "initrd_high=0xffffffff\0" \
>
> I'm really really not a fan of disabling these relocations, please use
> bootm_size to tell U-Boot where to make sure everything resides without
> overlap.  Also, distro boot?

If I remove "initrd_high" then the booting with a RAM Disk fails even
when I define bootm_size.  I have looked at several i.MX6 boards, and
it seems like the ones with RAM Disk options have this parameter.

I am sure what distro boot is.  Can you elaborate?
>
> [snip]
>> +# define MTDPARTS_DEFAULT    "mtdparts=gpmi-nand:4m(uboot)," \
>> +                                     "1m(env),8m(kernel),1m(dtb),-(fs)"
>
> Are you really sure all of these sizes are what you can live with for a
> long long time?  That kernel size feels small.

I defined the kernel to 8m based on the vast majority of the other
i.MX6 boards, but I can see your point.  I think I will just aim high
and go for 16.  This has plenty of NAND.

>
> Thanks!
>
Thank you for the feedback.

adam
> --
> Tom