[U-Boot] [PATCH V2 2/3] net: fec: do not access reserved register for i.MX6UL

[Stefano Babic <sbabic@denx.de>]

Hi Peng,

On 01/09/2015 02:32, Peng Fan wrote:

>> Patch is already applied - however, I have found that this break build
>> for vf610 boards (they have not a get_cpu_rev()). You can check with the
>> current u-boot-imx.
>>
> 
> Sorry, I checked all i.MXes, but missed vf610.

No problem - we fix it !

> 
>> Can you take a look, please ?
> 
> I worked a simple patch: https://patchwork.ozlabs.org/patch/512669/
> Add an empty function to avoid build errors. Current we do not have
> drivers to check vf610 cpu types, later if we need saying
> "is_cpu_type(VF610)", we can add more stuff to the function get_cpu_rev
> for vf610.
> 

I hoped to have a right get_cpu_rev() for VF610, too. Then I checked in
the VF610 manual, but I found revision only for specific controllers -
EHCI, etc. I have not found anything equivalent, but maybe it is better
hidden.

Of course, we can do with an empty function if we have not something
better. Maybe do you can check in Freescale if there is a way to get the
cpu revision instead of a dummy ?

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