Message ID | 1487666085-26465-1-git-send-email-hannes.schmelzer@br-automation.com |
---|---|
State | Superseded |
Delegated to: | Joe Hershberger |
Headers | show |
Hi Hannes 2017-02-21 9:34 GMT+01:00 Hannes Schmelzer <hannes.schmelzer@br-automation.com>: > This patch adds support for having a "fixed-link" to some other MAC > (like some embedded switch-device). > I really looking forward that u-boot gains support for fixed-links. > For this purpose we introduce a new phy-driver, called "Fixed PHY". > > Fixed PHY works only with CONFIG_DM_ETH enabled, since the fixed-link is > described with a subnode below ethernet interface. > > Most ethernet drivers (unfortunately not all are following same scheme > for searching/attaching phys) are calling "phy_connect(...)" for getting > a phy-device. > At this point we link in, we search here for a subnode called "fixed- > link", once found we start phy_device_create(...) with the special phy- > id PHY_FIXED_ID (0xa5a55a5a). > > During init the "Fixed PHY" driver has registered with this id and now > gets probed, during probe we get all the details about fixed-link out of > dts, later on the phy reports this values. > > Signed-off-by: Hannes Schmelzer <hannes.schmelzer@br-automation.com> > > --- > > drivers/net/phy/Makefile | 1 + > drivers/net/phy/fixed.c | 86 ++++++++++++++++++++++++++++++++++++++++++++++++ > drivers/net/phy/phy.c | 23 +++++++++++-- > include/phy.h | 3 ++ > 4 files changed, 110 insertions(+), 3 deletions(-) > create mode 100644 drivers/net/phy/fixed.c > > diff --git a/drivers/net/phy/Makefile b/drivers/net/phy/Makefile > index d372971..88c00a5 100644 > --- a/drivers/net/phy/Makefile > +++ b/drivers/net/phy/Makefile > @@ -28,3 +28,4 @@ obj-$(CONFIG_PHY_TI) += ti.o > obj-$(CONFIG_PHY_XILINX) += xilinx_phy.o > obj-$(CONFIG_PHY_VITESSE) += vitesse.o > obj-$(CONFIG_PHY_MSCC) += mscc.o > +obj-$(CONFIG_PHY_FIXED) += fixed.o > diff --git a/drivers/net/phy/fixed.c b/drivers/net/phy/fixed.c > new file mode 100644 > index 0000000..fc86a3f > --- /dev/null > +++ b/drivers/net/phy/fixed.c > @@ -0,0 +1,86 @@ > +/* > + * Fixed-Link phy > + * > + * Copyright 2017 Bernecker & Rainer Industrieelektronik GmbH > + * > + * SPDX-License-Identifier: GPL-2.0+ > + */ > + > +#include <config.h> > +#include <common.h> > +#include <phy.h> > +#include <dm.h> > +#include <fdt_support.h> > + > +DECLARE_GLOBAL_DATA_PTR; > + > +int fixedphy_probe(struct phy_device *phydev) > +{ > + struct fixed_link *priv; > + int ofnode = phydev->addr; > + u32 val; > + > + debug("%s: start.\n", __func__); Do we really need this debug(..) here? > + /* check for mandatory properties within fixed-link node */ > + val = fdt_getprop_u32_default_node(gd->fdt_blob, > + ofnode, 0, "speed", ~0UL); > + if (val == ~0UL || > + (val != SPEED_10 && val != SPEED_1000 && val != SPEED_1000)) { Maybe you wanted to write something like: (val != SPEED_10 && val != SPEED_100 && val != SPEED_1000)) { > + printf("ERROR: no/invalid speed given in fixed-link node!"); > + return -1; > + } > + > + priv = malloc(sizeof(*priv)); > + if (!priv) > + return -ENOMEM; > + memset(priv, 0, sizeof(*priv)); > + > + phydev->priv = priv; > + phydev->addr = 0; > + > + priv->link_speed = val; > + priv->duplex = fdtdec_get_bool(gd->fdt_blob, ofnode, "full-duplex"); > + priv->pause = fdtdec_get_bool(gd->fdt_blob, ofnode, "pause"); > + priv->asym_pause = fdtdec_get_bool(gd->fdt_blob, ofnode, "asym-pause"); > + > + /* set this for preventing stack doing some unary reset */ > + phydev->flags |= PHY_FLAG_BROKEN_RESET; > + > + return 0; > +} > + > +int fixedphy_startup(struct phy_device *phydev) > +{ > + struct fixed_link *priv = phydev->priv; > + > + debug("%s: start.\n", __func__); > + Do we really need this debug(..) here? > + phydev->asym_pause = priv->asym_pause; > + phydev->pause = priv->pause; > + phydev->duplex = priv->duplex; > + phydev->speed = priv->link_speed; > + phydev->link = 1; > + > + return 0; > +} > + > +int fixedphy_shutdown(struct phy_device *phydev) > +{ > + return 0; > +} > + > +static struct phy_driver fixedphy_driver = { > + .uid = PHY_FIXED_ID, > + .mask = 0xffffffff, > + .name = "Fixed PHY", > + .features = PHY_GBIT_FEATURES | SUPPORTED_MII, > + .probe = fixedphy_probe, > + .startup = fixedphy_startup, > + .shutdown = fixedphy_shutdown, > +}; > + > +int phy_fixed_init(void) > +{ > + phy_register(&fixedphy_driver); > + return 0; > +} > diff --git a/drivers/net/phy/phy.c b/drivers/net/phy/phy.c > index 8db6574..a569d0b 100644 > --- a/drivers/net/phy/phy.c > +++ b/drivers/net/phy/phy.c > @@ -515,7 +515,9 @@ int phy_init(void) > #ifdef CONFIG_PHY_MSCC > phy_mscc_init(); > #endif > - > +#ifdef CONFIG_PHY_FIXED > + phy_fixed_init(); > +#endif > return 0; > } > > @@ -854,9 +856,24 @@ struct phy_device *phy_connect(struct mii_dev *bus, int addr, > struct eth_device *dev, phy_interface_t interface) > #endif > { > - struct phy_device *phydev; > + struct phy_device *phydev = NULL; > +#if defined(CONFIG_DM_ETH) && defined(CONFIG_PHY_FIXED) > + int sn; > + const char *name; > + sn = fdt_first_subnode(gd->fdt_blob, dev->of_offset); > + while (sn > 0) { > + name = fdt_get_name(gd->fdt_blob, sn, NULL); > + if (name != NULL && strcmp(name, "fixed-link") == 0) { > + phydev = phy_device_create(bus, > + sn, PHY_FIXED_ID, interface); > + break; > + } > + sn = fdt_next_subnode(gd->fdt_blob, sn); > + } > +#endif > + if (phydev == NULL) > + phydev = phy_find_by_mask(bus, 1 << addr, interface); > > - phydev = phy_find_by_mask(bus, 1 << addr, interface); > if (phydev) > phy_connect_dev(phydev, dev); > else > diff --git a/include/phy.h b/include/phy.h > index 5477496..f5fdc04 100644 > --- a/include/phy.h > +++ b/include/phy.h > @@ -15,6 +15,8 @@ > #include <linux/ethtool.h> > #include <linux/mdio.h> > > +#define PHY_FIXED_ID 0xa5a55a5a > + > #define PHY_MAX_ADDR 32 > > #define PHY_FLAG_BROKEN_RESET (1 << 0) /* soft reset not supported */ > @@ -267,6 +269,7 @@ int phy_ti_init(void); > int phy_vitesse_init(void); > int phy_xilinx_init(void); > int phy_mscc_init(void); > +int phy_fixed_init(void); > > int board_phy_config(struct phy_device *phydev); > int get_phy_id(struct mii_dev *bus, int addr, int devad, u32 *phy_id); > -- > 1.9.1 greets -- Christian Gmeiner, MSc https://www.youtube.com/user/AloryOFFICIAL https://soundcloud.com/christian-gmeiner
On 03/21/2017 09:32 AM, Christian Gmeiner wrote: > Hi Hannes Hi Christian, many thanks for response (the 1st one). I will do some rework on this and send a v2 till end of this week. best regards, Hannes > > > 2017-02-21 9:34 GMT+01:00 Hannes Schmelzer <hannes.schmelzer@br-automation.com>: >> This patch adds support for having a "fixed-link" to some other MAC >> (like some embedded switch-device). >> > I really looking forward that u-boot gains support for fixed-links. > >> For this purpose we introduce a new phy-driver, called "Fixed PHY". >> >> Fixed PHY works only with CONFIG_DM_ETH enabled, since the fixed-link is >> described with a subnode below ethernet interface. >> >> Most ethernet drivers (unfortunately not all are following same scheme >> for searching/attaching phys) are calling "phy_connect(...)" for getting >> a phy-device. >> At this point we link in, we search here for a subnode called "fixed- >> link", once found we start phy_device_create(...) with the special phy- >> id PHY_FIXED_ID (0xa5a55a5a). >> >> During init the "Fixed PHY" driver has registered with this id and now >> gets probed, during probe we get all the details about fixed-link out of >> dts, later on the phy reports this values. >> >> Signed-off-by: Hannes Schmelzer <hannes.schmelzer@br-automation.com> >> >> --- >> >> drivers/net/phy/Makefile | 1 + >> drivers/net/phy/fixed.c | 86 ++++++++++++++++++++++++++++++++++++++++++++++++ >> drivers/net/phy/phy.c | 23 +++++++++++-- >> include/phy.h | 3 ++ >> 4 files changed, 110 insertions(+), 3 deletions(-) >> create mode 100644 drivers/net/phy/fixed.c >> >> diff --git a/drivers/net/phy/Makefile b/drivers/net/phy/Makefile >> index d372971..88c00a5 100644 >> --- a/drivers/net/phy/Makefile >> +++ b/drivers/net/phy/Makefile >> @@ -28,3 +28,4 @@ obj-$(CONFIG_PHY_TI) += ti.o >> obj-$(CONFIG_PHY_XILINX) += xilinx_phy.o >> obj-$(CONFIG_PHY_VITESSE) += vitesse.o >> obj-$(CONFIG_PHY_MSCC) += mscc.o >> +obj-$(CONFIG_PHY_FIXED) += fixed.o >> diff --git a/drivers/net/phy/fixed.c b/drivers/net/phy/fixed.c >> new file mode 100644 >> index 0000000..fc86a3f >> --- /dev/null >> +++ b/drivers/net/phy/fixed.c >> @@ -0,0 +1,86 @@ >> +/* >> + * Fixed-Link phy >> + * >> + * Copyright 2017 Bernecker & Rainer Industrieelektronik GmbH >> + * >> + * SPDX-License-Identifier: GPL-2.0+ >> + */ >> + >> +#include <config.h> >> +#include <common.h> >> +#include <phy.h> >> +#include <dm.h> >> +#include <fdt_support.h> >> + >> +DECLARE_GLOBAL_DATA_PTR; >> + >> +int fixedphy_probe(struct phy_device *phydev) >> +{ >> + struct fixed_link *priv; >> + int ofnode = phydev->addr; >> + u32 val; >> + >> + debug("%s: start.\n", __func__); > Do we really need this debug(..) here? > >> + /* check for mandatory properties within fixed-link node */ >> + val = fdt_getprop_u32_default_node(gd->fdt_blob, >> + ofnode, 0, "speed", ~0UL); >> + if (val == ~0UL || >> + (val != SPEED_10 && val != SPEED_1000 && val != SPEED_1000)) { > Maybe you wanted to write something like: > (val != SPEED_10 && val != SPEED_100 && val != SPEED_1000)) { > >> + printf("ERROR: no/invalid speed given in fixed-link node!"); >> + return -1; >> + } >> + >> + priv = malloc(sizeof(*priv)); >> + if (!priv) >> + return -ENOMEM; >> + memset(priv, 0, sizeof(*priv)); >> + >> + phydev->priv = priv; >> + phydev->addr = 0; >> + >> + priv->link_speed = val; >> + priv->duplex = fdtdec_get_bool(gd->fdt_blob, ofnode, "full-duplex"); >> + priv->pause = fdtdec_get_bool(gd->fdt_blob, ofnode, "pause"); >> + priv->asym_pause = fdtdec_get_bool(gd->fdt_blob, ofnode, "asym-pause"); >> + >> + /* set this for preventing stack doing some unary reset */ >> + phydev->flags |= PHY_FLAG_BROKEN_RESET; >> + >> + return 0; >> +} >> + >> +int fixedphy_startup(struct phy_device *phydev) >> +{ >> + struct fixed_link *priv = phydev->priv; >> + >> + debug("%s: start.\n", __func__); >> + > Do we really need this debug(..) here? > >> + phydev->asym_pause = priv->asym_pause; >> + phydev->pause = priv->pause; >> + phydev->duplex = priv->duplex; >> + phydev->speed = priv->link_speed; >> + phydev->link = 1; >> + >> + return 0; >> +} >> + >> +int fixedphy_shutdown(struct phy_device *phydev) >> +{ >> + return 0; >> +} >> + >> +static struct phy_driver fixedphy_driver = { >> + .uid = PHY_FIXED_ID, >> + .mask = 0xffffffff, >> + .name = "Fixed PHY", >> + .features = PHY_GBIT_FEATURES | SUPPORTED_MII, >> + .probe = fixedphy_probe, >> + .startup = fixedphy_startup, >> + .shutdown = fixedphy_shutdown, >> +}; >> + >> +int phy_fixed_init(void) >> +{ >> + phy_register(&fixedphy_driver); >> + return 0; >> +} >> diff --git a/drivers/net/phy/phy.c b/drivers/net/phy/phy.c >> index 8db6574..a569d0b 100644 >> --- a/drivers/net/phy/phy.c >> +++ b/drivers/net/phy/phy.c >> @@ -515,7 +515,9 @@ int phy_init(void) >> #ifdef CONFIG_PHY_MSCC >> phy_mscc_init(); >> #endif >> - >> +#ifdef CONFIG_PHY_FIXED >> + phy_fixed_init(); >> +#endif >> return 0; >> } >> >> @@ -854,9 +856,24 @@ struct phy_device *phy_connect(struct mii_dev *bus, int addr, >> struct eth_device *dev, phy_interface_t interface) >> #endif >> { >> - struct phy_device *phydev; >> + struct phy_device *phydev = NULL; >> +#if defined(CONFIG_DM_ETH) && defined(CONFIG_PHY_FIXED) >> + int sn; >> + const char *name; >> + sn = fdt_first_subnode(gd->fdt_blob, dev->of_offset); >> + while (sn > 0) { >> + name = fdt_get_name(gd->fdt_blob, sn, NULL); >> + if (name != NULL && strcmp(name, "fixed-link") == 0) { >> + phydev = phy_device_create(bus, >> + sn, PHY_FIXED_ID, interface); >> + break; >> + } >> + sn = fdt_next_subnode(gd->fdt_blob, sn); >> + } >> +#endif >> + if (phydev == NULL) >> + phydev = phy_find_by_mask(bus, 1 << addr, interface); >> >> - phydev = phy_find_by_mask(bus, 1 << addr, interface); >> if (phydev) >> phy_connect_dev(phydev, dev); >> else >> diff --git a/include/phy.h b/include/phy.h >> index 5477496..f5fdc04 100644 >> --- a/include/phy.h >> +++ b/include/phy.h >> @@ -15,6 +15,8 @@ >> #include <linux/ethtool.h> >> #include <linux/mdio.h> >> >> +#define PHY_FIXED_ID 0xa5a55a5a >> + >> #define PHY_MAX_ADDR 32 >> >> #define PHY_FLAG_BROKEN_RESET (1 << 0) /* soft reset not supported */ >> @@ -267,6 +269,7 @@ int phy_ti_init(void); >> int phy_vitesse_init(void); >> int phy_xilinx_init(void); >> int phy_mscc_init(void); >> +int phy_fixed_init(void); >> >> int board_phy_config(struct phy_device *phydev); >> int get_phy_id(struct mii_dev *bus, int addr, int devad, u32 *phy_id); >> -- >> 1.9.1 > greets > -- > Christian Gmeiner, MSc > > https://www.youtube.com/user/AloryOFFICIAL > https://soundcloud.com/christian-gmeiner > _______________________________________________ > U-Boot mailing list > U-Boot@lists.denx.de > https://lists.denx.de/listinfo/u-boot
diff --git a/drivers/net/phy/Makefile b/drivers/net/phy/Makefile index d372971..88c00a5 100644 --- a/drivers/net/phy/Makefile +++ b/drivers/net/phy/Makefile @@ -28,3 +28,4 @@ obj-$(CONFIG_PHY_TI) += ti.o obj-$(CONFIG_PHY_XILINX) += xilinx_phy.o obj-$(CONFIG_PHY_VITESSE) += vitesse.o obj-$(CONFIG_PHY_MSCC) += mscc.o +obj-$(CONFIG_PHY_FIXED) += fixed.o diff --git a/drivers/net/phy/fixed.c b/drivers/net/phy/fixed.c new file mode 100644 index 0000000..fc86a3f --- /dev/null +++ b/drivers/net/phy/fixed.c @@ -0,0 +1,86 @@ +/* + * Fixed-Link phy + * + * Copyright 2017 Bernecker & Rainer Industrieelektronik GmbH + * + * SPDX-License-Identifier: GPL-2.0+ + */ + +#include <config.h> +#include <common.h> +#include <phy.h> +#include <dm.h> +#include <fdt_support.h> + +DECLARE_GLOBAL_DATA_PTR; + +int fixedphy_probe(struct phy_device *phydev) +{ + struct fixed_link *priv; + int ofnode = phydev->addr; + u32 val; + + debug("%s: start.\n", __func__); + /* check for mandatory properties within fixed-link node */ + val = fdt_getprop_u32_default_node(gd->fdt_blob, + ofnode, 0, "speed", ~0UL); + if (val == ~0UL || + (val != SPEED_10 && val != SPEED_1000 && val != SPEED_1000)) { + printf("ERROR: no/invalid speed given in fixed-link node!"); + return -1; + } + + priv = malloc(sizeof(*priv)); + if (!priv) + return -ENOMEM; + memset(priv, 0, sizeof(*priv)); + + phydev->priv = priv; + phydev->addr = 0; + + priv->link_speed = val; + priv->duplex = fdtdec_get_bool(gd->fdt_blob, ofnode, "full-duplex"); + priv->pause = fdtdec_get_bool(gd->fdt_blob, ofnode, "pause"); + priv->asym_pause = fdtdec_get_bool(gd->fdt_blob, ofnode, "asym-pause"); + + /* set this for preventing stack doing some unary reset */ + phydev->flags |= PHY_FLAG_BROKEN_RESET; + + return 0; +} + +int fixedphy_startup(struct phy_device *phydev) +{ + struct fixed_link *priv = phydev->priv; + + debug("%s: start.\n", __func__); + + phydev->asym_pause = priv->asym_pause; + phydev->pause = priv->pause; + phydev->duplex = priv->duplex; + phydev->speed = priv->link_speed; + phydev->link = 1; + + return 0; +} + +int fixedphy_shutdown(struct phy_device *phydev) +{ + return 0; +} + +static struct phy_driver fixedphy_driver = { + .uid = PHY_FIXED_ID, + .mask = 0xffffffff, + .name = "Fixed PHY", + .features = PHY_GBIT_FEATURES | SUPPORTED_MII, + .probe = fixedphy_probe, + .startup = fixedphy_startup, + .shutdown = fixedphy_shutdown, +}; + +int phy_fixed_init(void) +{ + phy_register(&fixedphy_driver); + return 0; +} diff --git a/drivers/net/phy/phy.c b/drivers/net/phy/phy.c index 8db6574..a569d0b 100644 --- a/drivers/net/phy/phy.c +++ b/drivers/net/phy/phy.c @@ -515,7 +515,9 @@ int phy_init(void) #ifdef CONFIG_PHY_MSCC phy_mscc_init(); #endif - +#ifdef CONFIG_PHY_FIXED + phy_fixed_init(); +#endif return 0; } @@ -854,9 +856,24 @@ struct phy_device *phy_connect(struct mii_dev *bus, int addr, struct eth_device *dev, phy_interface_t interface) #endif { - struct phy_device *phydev; + struct phy_device *phydev = NULL; +#if defined(CONFIG_DM_ETH) && defined(CONFIG_PHY_FIXED) + int sn; + const char *name; + sn = fdt_first_subnode(gd->fdt_blob, dev->of_offset); + while (sn > 0) { + name = fdt_get_name(gd->fdt_blob, sn, NULL); + if (name != NULL && strcmp(name, "fixed-link") == 0) { + phydev = phy_device_create(bus, + sn, PHY_FIXED_ID, interface); + break; + } + sn = fdt_next_subnode(gd->fdt_blob, sn); + } +#endif + if (phydev == NULL) + phydev = phy_find_by_mask(bus, 1 << addr, interface); - phydev = phy_find_by_mask(bus, 1 << addr, interface); if (phydev) phy_connect_dev(phydev, dev); else diff --git a/include/phy.h b/include/phy.h index 5477496..f5fdc04 100644 --- a/include/phy.h +++ b/include/phy.h @@ -15,6 +15,8 @@ #include <linux/ethtool.h> #include <linux/mdio.h> +#define PHY_FIXED_ID 0xa5a55a5a + #define PHY_MAX_ADDR 32 #define PHY_FLAG_BROKEN_RESET (1 << 0) /* soft reset not supported */ @@ -267,6 +269,7 @@ int phy_ti_init(void); int phy_vitesse_init(void); int phy_xilinx_init(void); int phy_mscc_init(void); +int phy_fixed_init(void); int board_phy_config(struct phy_device *phydev); int get_phy_id(struct mii_dev *bus, int addr, int devad, u32 *phy_id);
This patch adds support for having a "fixed-link" to some other MAC (like some embedded switch-device). For this purpose we introduce a new phy-driver, called "Fixed PHY". Fixed PHY works only with CONFIG_DM_ETH enabled, since the fixed-link is described with a subnode below ethernet interface. Most ethernet drivers (unfortunately not all are following same scheme for searching/attaching phys) are calling "phy_connect(...)" for getting a phy-device. At this point we link in, we search here for a subnode called "fixed- link", once found we start phy_device_create(...) with the special phy- id PHY_FIXED_ID (0xa5a55a5a). During init the "Fixed PHY" driver has registered with this id and now gets probed, during probe we get all the details about fixed-link out of dts, later on the phy reports this values. Signed-off-by: Hannes Schmelzer <hannes.schmelzer@br-automation.com> --- drivers/net/phy/Makefile | 1 + drivers/net/phy/fixed.c | 86 ++++++++++++++++++++++++++++++++++++++++++++++++ drivers/net/phy/phy.c | 23 +++++++++++-- include/phy.h | 3 ++ 4 files changed, 110 insertions(+), 3 deletions(-) create mode 100644 drivers/net/phy/fixed.c