From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Schocher Date: Tue, 04 Jun 2013 17:15:29 +0200 Subject: [U-Boot] net, cpsw: disable gigabit support through plattform data In-Reply-To: <20130604150045.3454038306D@gemini.denx.de> References: <1370335914-14027-1-git-send-email-hs@denx.de> <20130604150045.3454038306D@gemini.denx.de> Message-ID: <51AE0491.7020507@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hello Wolfgang, Am 04.06.2013 17:00, schrieb Wolfgang Denk: > Dear Heiko, > > In message <1370335914-14027-1-git-send-email-hs@denx.de> you wrote: >> add possibility to disable gigabit support through >> plattform data. Current boards should not be affected >> through this patch. > ... >> + if (phy->speed == 1000) { >> + if (priv->data.gigabit_en) { >> + mac_control |= GIGABITEN; >> + } else { >> + /* Disable gigabit as it's non-functional */ >> + mac_control &= ~GIGABITEN; >> + phy->speed = 100; > > Is this reliable? I mean, how do we know it's a 100 Mbps connection > (and not for example a 10 Mbs one) ? Indeed. Anyway, I have to check, if this patch is necessary, as Tom noted ... bye, Heiko -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany