From mboxrd@z Thu Jan 1 00:00:00 1970 From: Anton Vorontsov Date: Tue, 8 Jan 2008 21:41:09 +0300 Subject: [U-Boot-Users] [PATCH] mpc83xx: UEC: add support for Broadcom BCM5481 Gigabit PHY In-Reply-To: <20080108105018.4d91dd7f.kim.phillips@freescale.com> References: <20071225170126.GB21275@localhost.localdomain> <20080108105018.4d91dd7f.kim.phillips@freescale.com> Message-ID: <20080108184109.GA12103@localhost.localdomain> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Tue, Jan 08, 2008 at 10:50:18AM -0600, Kim Phillips wrote: > On Tue, 25 Dec 2007 20:01:26 +0300 > Anton Vorontsov wrote: > > > This patch adds basic support for Broadcom BCM5481 PHY, > > with the quirk needed for at least MPC8360E-RDK. > > ? the MPC8360E-RDK doesn't exist in the mainline tree. Patch for the board support was sent at the same time. Though, I'll resend it with few improvements. > > Quirk comes from MPC8360E-RDK BSP source, I think author is > > Peter Barada , but I'm not sure. > > solicit his signoff then? Not an absolute necessity. Signed-off-by means: - - - Documentation/SubmittingPatches By making a contribution to this project, I certify that: (a) The contribution was created in whole or in part by me and I have the right to submit it under the open source license indicated in the file; or (b) The contribution is based upon previous work that, to the best of my knowledge, is covered under an appropriate open source license and I have the right under that license to submit that work with modifications, whether created in whole or in part by me, under the same open source license (unless I am permitted to submit under a different license), as indicated in the file; or (c) The contribution was provided directly to me by some other person who certified (a), (b) or (c) and I have not modified it. (d) I understand and agree that this project and the contribution are public and that a record of the contribution (including all personal information I submit with it, including my sign-off) is maintained indefinitely and may be redistributed consistent with this project or the open source license(s) involved. then you just add a line saying Signed-off-by: Random J Developer - - - Here is (b). To the best of my knowledge, code is covered under an appropriate open source license (that is, GPL) and I have the right under that license to submit that work with modifications, whether created in whole or in part by me, under the same open source license (GPL). Though, I'm adding Peter Barada to the Cc and copying the patch. Peter, could you please add your Signed-off-by line on the bcm phy quirk? - - - mpc83xx: UEC: add support for Broadcom BCM5481 Gigabit PHY This patch adds basic support for Broadcom BCM5481 PHY, with the quirk needed for at least MPC8360E-RDK. Quirk comes from MPC8360E-RDK BSP source, I think author is Peter Barada , but I'm not sure. There are no openly available specifications for that PHY. diff --git a/drivers/qe/uec_phy.c b/drivers/qe/uec_phy.c index ca6faa6..6882d03 100644 --- a/drivers/qe/uec_phy.c +++ b/drivers/qe/uec_phy.c @@ -237,6 +237,44 @@ static int gbit_config_aneg (struct uec_mii_info22 *mii_info) return 0; } +static int gbit_read_status(struct uec_mii_info *mii_info) +{ + u16 status; + int err; + + err = genmii_update_link(mii_info); + if (err) + return err; + + if (mii_info->autoneg) { + mii_info->pause = 0; + status = phy_read(mii_info, MII_1000BASETSTATUS); + if (status & (LPA_1000FULL | LPA_1000HALF)) { + mii_info->speed = SPEED_1000; + if (status & LPA_1000FULL) + mii_info->duplex = DUPLEX_FULL; + else + mii_info->duplex = DUPLEX_HALF; + } else { + status = phy_read(mii_info, PHY_ANLPAR); + + if (status & (PHY_ANLPAR_10FD | PHY_ANLPAR_TXFD)) + mii_info->duplex = DUPLEX_FULL; + else + mii_info->duplex = DUPLEX_HALF; + if (status & (PHY_ANLPAR_TXFD | PHY_ANLPAR_TX)) + mii_info->speed = SPEED_100; + else + mii_info->speed = SPEED_10; + } + } + /* + * On non-aneg, we assume what we put in BMCR is the speed, + * though magic-aneg shouldn't prevent this case from occurring. + */ + return 0; +} + static int marvell_config_aneg (struct uec_mii_info *mii_info) { /* The Marvell PHY has an errata which requires @@ -319,6 +357,35 @@ static int genmii_read_status (struct uec_mii_info *mii_info) return 0; } +static int bcm_init(struct uec_mii_info *mii_info) +{ + gbit_config_aneg(mii_info); + +#ifdef CONFIG_MPC8360ERDK + { + u16 val; + int cnt = 50; + + /* Wait for aneg to complete. */ + do + val = phy_read(mii_info, PHY_BMSR); + while (--cnt && !(val & PHY_BMSR_AUTN_COMP)); + + /* Set RDX clk delay. */ + phy_write(mii_info, 0x18, 0x7 | (7 << 12)); + + val = phy_read(mii_info, 0x18); + /* Set RDX-RXC skew. */ + val |= (1<<8); + val |= (7 | (7 << 12)); + /* Write bits 14:0. */ + val |= (1<<15); + phy_write(mii_info, 0x18, val); + } +#endif + return 0; +} + static int marvell_read_status (struct uec_mii_info *mii_info) { u16 status; @@ -491,6 +558,15 @@ static struct phy_info phy_info_marvell = { .config_intr = &marvell_config_intr, }; +static struct phy_info phy_info_bcm5481 = { + .phy_id = 0x0143bca0, + .phy_id_mask = 0xffffff0, + .name = "Broadcom 5481", + .features = MII_GBIT_FEATURES, + .read_status = gbit_read_status, + .init = bcm_init, +}; + static struct phy_info phy_info_genmii = { .phy_id = 0x00000000, .phy_id_mask = 0x00000000, @@ -504,6 +580,7 @@ static struct phy_info *phy_info[] = { &phy_info_dm9161, &phy_info_dm9161a, &phy_info_marvell, + &phy_info_bcm5481, &phy_info_genmii, NULL }; diff --git a/drivers/qe/uec_phy.h b/drivers/qe/uec_phy.h index e59a940..6f769fb 100644 --- a/drivers/qe/uec_phy.h +++ b/drivers/qe/uec_phy.h @@ -29,6 +29,11 @@ #define MII_1000BASETCONTROL_FULLDUPLEXCAP 0x0200 #define MII_1000BASETCONTROL_HALFDUPLEXCAP 0x0100 +/* 1000BT status */ +#define MII_1000BASETSTATUS 0x0a +#define LPA_1000FULL 0x0400 +#define LPA_1000HALF 0x0200 + /* Cicada Extended Control Register 1 */ #define MII_CIS8201_EXT_CON1 0x17 #define MII_CIS8201_EXTCON1_INIT 0x0000 - - - Thanks in advance. > > There are no openly available specifications for that PHY. > > > > Signed-off-by: Anton Vorontsov > > Also, shouldn't networking patches go through Ben Warren? Ok, I'll expand Cc list on respin. Thanks, -- Anton Vorontsov email: cbou at mail.ru backup email: ya-cbou at yandex.ru irc://irc.freenode.net/bd2