[U-Boot] [PATCH v1 2/4] udoo: Add ethernet support (FEC + Micrel KSZ9031).

Giuseppe Pagano giuseppe.pagano at seco.com
Fri Nov 15 12:48:35 CET 2013


Hi Stefano,

On Wed, 2013-11-13 at 10:04 +0100, Stefano Babic wrote:
> Hi Giuseppe,
> 
> On 11/11/2013 18:11, Giuseppe Pagano wrote:
> > Add Ethernet and networking support on uDoo board (FEC + phy Micrel KSZ9031).
> > 
> > Signed-off-by: Giuseppe Pagano <giuseppe.pagano at seco.com>
> > CC: Stefano Babic <sbabic at denx.de>
> > CC: Fabio Estevam <fabio.estevam at freescale.com>
> > ---
> 
> ok - remember to increment the version number of the patchset for next
> time (this is really V2 instead of V1).

Ok, I'm going to prepare next patch. Sorry for the error, for this time
I think it would be better to use V2 in next submit to reduce confusion.

> > +int mx6_rgmii_rework(struct phy_device *phydev)
> > +{
> > +	/*
> > +	 * Bug: Apparently uDoo does not works with Gigabit switches...
> > +	 * Limiting speed to 10/100Mbps, and setting master mode, seems to
> > +	 * be the only way to have a successfull PHY auto negotiation.
> > +	 * How to fix: Understand why Linux kernel do not have this issue.
> > +	 */
> 
> Fine. This is ok, but could you also add to the commit message that
> Ethernet is currently limited to 10/100Mbps ?

Ok, I will do.

> 
> Best regards,
> Stefano Babic

Best Regards
Giuseppe Pagano



More information about the U-Boot mailing list