Heiko Schocher wrote:
> Hello Stefano,
> 

Hi Heiko,

> your following commit breaks building the imx27lite board
> (and all boards, which uses the ./drivers/net/fec_mxc.c
> I think), because the following function is missing:
> 
> [...@pollux u-boot]$ grep -lr imx_get_fecclk .
> ./drivers/net/fec_mxc.c
> ./drivers/net/fec_mxc.o
> ./drivers/net/libnet.a
> ./LOG/imx27lite.ERR
> [...@pollux u-boot]$
> 
> and this function is only used in this driver ...
> 
> ---------------------------------------------------------------
> 
> author        Stefano Babic <[email protected]>
> commit        4294b2485bf0e8d68c893190a96bb0e7856b12c4
> 
> fec_mxc: add support for MX51 processor
> 
> The patch add support for the Freescale mx51 processor
> 
> to the FEC ethernet driver.
> 
> 
> Signed-off-by: Stefano Babic <[email protected]>
> Signed-off-by: Ben Warren <[email protected]>
> ---------------------------------------------------------------
> 
> ... any ideas?

Yes, at the same time two patches for the fec_imx was sent by me and by
John. John sends to me and to the ML a comment informing that he made
patches to factor a lot of SOC specific, cleaning up code and avoiding
nasty #ifdef.

I see his patch in archive as
http://www.mail-archive.com/[email protected]/msg27481.html.

I have already used the proposed function, but John's patch is not
already in the mainline - probably it is still in the imx queue. Jason
is working to bring a lot of MX patches in the mainline this week.

Regards,
Stefano

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: [email protected]
=====================================================================
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to