[U-Boot] ??: [PATCH v2] include/linux/mii.h: update for supporting GE

Anton Vorontsov avorontsov at mvista.com
Fri Jan 21 10:52:21 CET 2011


On Fri, Jan 21, 2011 at 09:23:12AM +0100, Wolfgang Denk wrote:
[...]
> > In uec_phy:
> > drivers/qe/uec_phy.h:34:#define LPA_1000FULL            0x0400
> > drivers/qe/uec_phy.h:35:#define LPA_1000HALF            0x0200
> >
> > In Linux:
> > include/linux/mii.h:133:#define LPA_1000FULL            0x0800  /* Link partner 1000BASE-T full duplex */
> > include/linux/mii.h:134:#define LPA_1000HALF            0x0400  /* Link partner 1000BASE-T half duplex */
> >
> > Do you know if this is a wrong defintion on uec_phy ?
> > I'm afraid the fix will lead some mis-behavior for boards has uec_phy.
> 
> These definitions were added by commit 91cdaa3a:
> 
> commit 91cdaa3a9d7562b869d96774e9c9ddf142c0848d
> Author: Anton Vorontsov <avorontsov at ru.mvista.com>
> Date:   Mon Mar 24 20:46:24 2008 +0300
> 
>     uec: add support for gbit mii status readings
>     
>     Signed-off-by: Anton Vorontsov <avorontsov at ru.mvista.com>
> 
> 
> Anton, can you please comment?

I think that was just a shift-by-one mistake, Linux code is
correct.

It seems that the only way to trigger this is to force half-duplex
on a gigabit link (and u-boot will think that it's a full duplex
link).

I believe it's safe to fix the wrong definition.

Thanks,

-- 
Anton Vorontsov
Email: cbouatmailru at gmail.com


More information about the U-Boot mailing list