Re: [PATCH 1/1] mx53 loco: enable smsc phy driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Nov 15, 2012 at 12:56:02PM +0100, Jean-Christophe PLAGNIOL-VILLARD wrote:
> Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@xxxxxxxxxxxx>
> ---
>  arch/arm/configs/freescale_mx53_loco_defconfig |    1 +
>  1 file changed, 1 insertion(+)

We're getting closer \o/

With this the LOCO correctly waits for a link, unfortunately during
board init time. This is due to:

commit 3a17af33c01fd066ab3939c8218037c87ba26eed
Author: Sascha Hauer <s.hauer@xxxxxxxxxxxxxx>
Date:   Fri Oct 26 16:16:54 2012 +0200

    net fec: connect phy at probe time
    
    Several i.MX boards (all i.MX6 boards) need to do some adjustments
    to phy registers. If barebox itself does not use network, networking
    won't work in the kernel if the kernel does not have the fixups. Connect
    the phy at probe time so that these tweaks are done during probe so that
    the kernel works without phy register tweaks. Also this has the effect
    that the phy device is present and introspectable without doing fake
    network transfers beforehand.

We could revert it which fixes things for the LOCO, but then we also
lose the phy fixups which this commit intentionally pushed to probe
time.

It would be nice to separate phy registration from actually waiting for
a link. I find it rather inconvenient anyway that the phy device pops
up during the first network transfer.

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox


[Index of Archives]     [Linux Embedded]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux