[U-Boot] [PATCH V3 1/8] tools: imximage: add plugin support

[Eric Nelson]

Hi Peng,

On 10/09/2016 08:12 AM, Peng Fan wrote:
>> On 10/09/2016 04:20 AM, Peng Fan wrote:
>>> On Sat, Oct 08, 2016 at 05:26:18PM +0200, Eric Nelson wrote:
>>>> On 10/08/2016 08:58 AM, Peng Fan wrote:

<snip>

>>>>
>>>> I also don't understand why the choice to make the union
>>>> with either a DCD table or a plugin.
>>>
>>> Confirmed with ROM team. DCD and plugin are exclusive,
>>>
>>> You can not have both at the same time.
>>>
>>
>> That's too bad, since porting from DCD-style to SPL can be
>> useful (as Fabio has seen trying to keep some boards up-to-date).
> 
> I saw the patches.
> If using plugin, there is no need to use DCD.
> 
> all the things in DCD can be done in plugin code.
> 
>>
>> If we get SPL-as-plugin working, then this would form a
>> dividing line where you need to get rid of the DCD altogether.
>>
>> What about the CSF test? Your patches say that CSF and plugins
>> are also not supported.
> 
> CSF is supported  and code in nxp community. But that piece code is not included
> in the nxp released uboot.
> 

Cool. Otherwise

> https://community.nxp.com/docs/DOC-332725
> 

This page is telling me that "Access is restricted".

>>

<snip>

>>>>
>>>> Where is the 16k limit coming from? I don't think this is necessary,
>>>> and if it is, we won't be able to load SPL using a plugin.
>>>
>>> Confirmed with ROM team, there is no limitation. But SPL code needs to be small
>>> to be in OCRAM.
>>>
>>
>> Whew! This would have killed the idea of SPL-as-plugin.
> 
> SPL is designed to run in OCRAM for i.MX6, so it's still ok to let SPL as plugin image.
> 

Cool.

I'll take a stab at instrumenting the startup code and let you know
what I find.

Regards,


Eric