From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ben Warren Date: Mon, 20 Jul 2009 23:22:34 -0700 Subject: [U-Boot] [PATCH 5/6] net: Kirkwood_egiga: forced interface speed config support In-Reply-To: <20090718184338.GA16951@game.jcrosoft.org> References: <1247758084-12296-1-git-send-email-prafulla@marvell.com> <1247758084-12296-2-git-send-email-prafulla@marvell.com> <1247758084-12296-3-git-send-email-prafulla@marvell.com> <1247758084-12296-4-git-send-email-prafulla@marvell.com> <1247758084-12296-5-git-send-email-prafulla@marvell.com> <20090718184338.GA16951@game.jcrosoft.org> Message-ID: <4A655EAA.4000401@gmail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Jean-Christophe PLAGNIOL-VILLARD wrote: > On 20:58 Thu 16 Jul , Prafulla Wadaskar wrote: > >> By default Auto Negotiation is enabled for interface speed >> but on some platforms like RD6281A it does not work. >> If you want to forced program it to desired speed, >> this patch helps- >> >> Through this patch Auto negotiation can be disabled and >> desired interface speed can be configured >> >> This patch is tested on RD6281A Kirkwood board >> >> Signed-off-by: Prafulla Wadaskar >> --- >> drivers/net/kirkwood_egiga.c | 24 ++++++++++++++++++++++++ >> 1 files changed, 24 insertions(+), 0 deletions(-) >> >> diff --git a/drivers/net/kirkwood_egiga.c b/drivers/net/kirkwood_egiga.c >> index 3c5db19..1dfd567 100644 >> --- a/drivers/net/kirkwood_egiga.c >> +++ b/drivers/net/kirkwood_egiga.c >> @@ -415,7 +415,31 @@ static int kwgbe_init(struct eth_device *dev) >> /* Assign port configuration and command. */ >> KWGBEREG_WR(regs->pxc, PRT_CFG_VAL); >> KWGBEREG_WR(regs->pxcx, PORT_CFG_EXTEND_VALUE); >> + /* >> + * Forced 10/100/1000BASE-T interface speed configuration >> + * By default Auto Negotiation of interface speed is enabled >> + * This can be forced disabled and desired speed can be configured >> + */ >> +#ifdef CONFIG_DIS_AUTO_NEG_SPEED_GMII >> +#if (!defined (CONFIG_PHY_SPEED) || (CONFIG_PHY_SPEED == _1000BASET)) >> > Could you find a better config taht _1000BASET & co > > Global configs for this type of thing are a bad idea (what if you want one interface @ 1000 and one @ 100? I guess this will have to do for now, but let's ensure that it's temporary. > Best Regards, > J. regards, Ben