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

Stefano Babic sbabic at denx.de
Mon Oct 2 15:07:01 UTC 2017


Hi Otavio, Fabio,

On 02/10/2017 16:25, Otavio Salvador wrote:
> On Mon, Oct 2, 2017 at 11:22 AM, Stefano Babic <sbabic at denx.de> 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 ?
> 
> It is defined but now it is all upper case. Eric did rename it.


Yes, thanks both ! It was the "Camelcase" patch, I have forgotten it.

Best regards,
Stefano

-- 
=====================================================================
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