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.pag...@seco.com>
> > CC: Stefano Babic <sba...@denx.de>
> > CC: Fabio Estevam <fabio.este...@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
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
