[U-Boot] [PATCH 3/4] imx: mx7: SPL support for i.MX7

Stefano Babic sbabic at denx.de
Mon Oct 2 15:10:04 UTC 2017


Hi Uri,

On 02/10/2017 16:59, Uri Mashiach wrote:
> Hi Stefano,
> 
> On 10/02/2017 05:22 PM, Stefano Babic wrote:
>> Hi Uri,
>>
>> On 24/09/2017 15:32, Fabio Estevam wrote:
>>> Hi Uri,
>>>
>>> On Sun, Sep 24, 2017 at 3:00 AM, Uri Mashiach
>>> <uri.mashiach at compulab.co.il> wrote:
>>>> Add configuration file and spl_boot_device function for the i.MX7 SPL.
>>>>
>>>> Signed-off-by: Uri Mashiach <uri.mashiach at compulab.co.il>
>>>
>>> Thanks for working on this!
>>>
>>> I will test your series and will try to convert imx7d-sabresd to SPL.
>>>
>>
>> It looks like that at least a define is missing - by applying the patch
>> and testing, I get the error:
>>
>> +board/compulab/cl-som-imx7/cl-som-imx7.c: In function
>> 'cl_som_imx7_setup_fec':
>> +board/compulab/cl-som-imx7/cl-som-imx7.c:231:15: error: 'ENET_125MHz'
>> undeclared (first use in this function)
>>
>>
>> In fact, ENET_125MHz is not definded anywhere in U-Boot. Could you
>> please check and fix this ?
>>
>> Best regards,
>> Stefano Babic
>>
>>
> 
> The patch is compiling in u-boot-imx/master.
> Should it be based on mainline u-boot?

No, it is ok - see above, I fix the Camelcase. There is still a small
conflict in arch/arm/mach-imx/spl.c (due to other patches I am merging),
I fix this as well. If all is fine, I psuh it to the server. This
simplifies Fabio's work and someone else can help with tests.
Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================


More information about the U-Boot mailing list