[U-Boot] imx27lite board support no longer compile

Hello Stefano,
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:
[hs@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 [hs@pollux u-boot]$
and this function is only used in this driver ...
---------------------------------------------------------------
author Stefano Babic sbabic@denx.de 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 sbabic@denx.de Signed-off-by: Ben Warren biggerbadderben@gmail.com ---------------------------------------------------------------
... any ideas?
bye Heiko

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:
[hs@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 [hs@pollux u-boot]$
and this function is only used in this driver ...
author Stefano Babic sbabic@denx.de 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 sbabic@denx.de Signed-off-by: Ben Warren biggerbadderben@gmail.com
... 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/u-boot@lists.denx.de/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

Heiko Schocher wrote:
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:
Hello Heiko,
I have applied John's patches to the u-boot-imx tree. Building of imx27lite is ok using that tree.
Regards, Stefano

Hello Stefano,
Stefano Babic wrote:
Heiko Schocher wrote:
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:
Hello Heiko,
I have applied John's patches to the u-boot-imx tree. Building of imx27lite is ok using that tree.
Thanks for the info!
bye Heiko
participants (2)
-
Heiko Schocher
-
Stefano Babic