[PATCH 1/4] net: designware: move probe to generic driver
Sascha Hauer
s.hauer at pengutronix.de
Thu Jan 11 00:34:47 PST 2018
On Wed, Jan 10, 2018 at 09:14:17AM +0100, Steffen Trumtrar wrote:
> The designware ethernet core is used on multiple different SoCs.
> The linux kernel has a generic driver and SoC-specific drivers.
> Do the same here.
>
> Signed-off-by: Steffen Trumtrar <s.trumtrar at pengutronix.de>
> ---
> drivers/net/Kconfig | 12 +++++++-
> drivers/net/Makefile | 1 +
> drivers/net/designware.c | 60 ++++------------------------------------
> drivers/net/designware.h | 33 ++++++++++++++++++++++
> drivers/net/designware_generic.c | 59 +++++++++++++++++++++++++++++++++++++++
> 5 files changed, 110 insertions(+), 55 deletions(-)
> create mode 100644 drivers/net/designware_generic.c
Applied, thanks
Sascha
>
> diff --git a/drivers/net/Kconfig b/drivers/net/Kconfig
> index 9d69b6aeb09c..36b257f43e8a 100644
> --- a/drivers/net/Kconfig
> +++ b/drivers/net/Kconfig
> @@ -64,13 +64,23 @@ config DRIVER_NET_DAVINCI_EMAC
> select PHYLIB
>
> config DRIVER_NET_DESIGNWARE
> - bool "Designware Universal MAC ethernet driver"
> + bool "Designware Universal MAC ethernet platform support"
> depends on HAS_DMA
> select PHYLIB
> help
> This option enables support for the Synopsys
> Designware Core Univesal MAC 10M/100M/1G ethernet IP.
>
> +if DRIVER_NET_DESIGNWARE
> +
> +config DRIVER_NET_DESIGNWARE_GENERIC
> + bool "Designware Universal MAC ethernet generic driver"
> + help
> + This option enables support for the Synopsys
> + Designware Core Univesal MAC 10M/100M/1G ethernet IP on SoCFPGA.
> +
> +endif
> +
> config DRIVER_NET_DM9K
> bool "Davicom dm9k[E|A|B] ethernet driver"
> depends on HAS_DM9000
> diff --git a/drivers/net/Makefile b/drivers/net/Makefile
> index 86c8ac32f93e..eb07434ab42c 100644
> --- a/drivers/net/Makefile
> +++ b/drivers/net/Makefile
> @@ -9,6 +9,7 @@ obj-$(CONFIG_DRIVER_NET_CS8900) += cs8900.o
> obj-$(CONFIG_DRIVER_NET_CPSW) += cpsw.o
> obj-$(CONFIG_DRIVER_NET_DAVINCI_EMAC) += davinci_emac.o
> obj-$(CONFIG_DRIVER_NET_DESIGNWARE) += designware.o
> +obj-$(CONFIG_DRIVER_NET_DESIGNWARE_GENERIC) += designware_generic.o
> obj-$(CONFIG_DRIVER_NET_DM9K) += dm9k.o
> obj-$(CONFIG_DRIVER_NET_E1000) += e1000/regio.o e1000/main.o e1000/eeprom.o
> obj-$(CONFIG_DRIVER_NET_ENC28J60) += enc28j60.o
> diff --git a/drivers/net/designware.c b/drivers/net/designware.c
> index 1d3a68384edb..ec20da09b388 100644
> --- a/drivers/net/designware.c
> +++ b/drivers/net/designware.c
> @@ -32,36 +32,6 @@
> #include <linux/err.h>
> #include "designware.h"
>
> -struct dw_eth_dev {
> - struct eth_device netdev;
> - struct mii_bus miibus;
> -
> - void (*fix_mac_speed)(int speed);
> - u8 macaddr[6];
> - u32 tx_currdescnum;
> - u32 rx_currdescnum;
> -
> - struct dmamacdescr *tx_mac_descrtable;
> - struct dmamacdescr *rx_mac_descrtable;
> -
> - u8 *txbuffs;
> - u8 *rxbuffs;
> -
> - struct eth_mac_regs *mac_regs_p;
> - struct eth_dma_regs *dma_regs_p;
> - int phy_addr;
> - phy_interface_t interface;
> - int enh_desc;
> -};
> -
> -struct dw_eth_drvdata {
> - bool enh_desc;
> -};
> -
> -static struct dw_eth_drvdata dwmac_370a_drvdata = {
> - .enh_desc = 1,
> -};
> -
> /* Speed specific definitions */
> #define SPEED_10M 1
> #define SPEED_100M 2
> @@ -447,7 +417,7 @@ static int dwc_probe_dt(struct device_d *dev, struct dw_eth_dev *priv)
> return 0;
> }
>
> -static int dwc_ether_probe(struct device_d *dev)
> +struct dw_eth_dev *dwc_drv_probe(struct device_d *dev)
> {
> struct resource *iores;
> struct dw_eth_dev *priv;
> @@ -462,7 +432,7 @@ static int dwc_ether_probe(struct device_d *dev)
>
> ret = dev_get_drvdata(dev, (const void **)&drvdata);
> if (ret)
> - return ret;
> + return ERR_PTR(ret);
>
> priv->enh_desc = drvdata->enh_desc;
>
> @@ -473,12 +443,12 @@ static int dwc_ether_probe(struct device_d *dev)
> } else {
> ret = dwc_probe_dt(dev, priv);
> if (ret)
> - return ret;
> + return ERR_PTR(ret);
> }
>
> iores = dev_request_mem_resource(dev, 0);
> if (IS_ERR(iores))
> - return PTR_ERR(iores);
> + return ERR_CAST(iores);
> base = IOMEM(iores->start);
>
> priv->mac_regs_p = base;
> @@ -512,24 +482,6 @@ static int dwc_ether_probe(struct device_d *dev)
>
> mdiobus_register(miibus);
> eth_register(edev);
> - return 0;
> -}
>
> -static __maybe_unused struct of_device_id dwc_ether_compatible[] = {
> - {
> - .compatible = "snps,dwmac-3.70a",
> - .data = &dwmac_370a_drvdata,
> - }, {
> - .compatible = "snps,dwmac-3.72a",
> - .data = &dwmac_370a_drvdata,
> - }, {
> - /* sentinel */
> - }
> -};
> -
> -static struct driver_d dwc_ether_driver = {
> - .name = "designware_eth",
> - .probe = dwc_ether_probe,
> - .of_compatible = DRV_OF_COMPAT(dwc_ether_compatible),
> -};
> -device_platform_driver(dwc_ether_driver);
> + return priv;
> +}
> diff --git a/drivers/net/designware.h b/drivers/net/designware.h
> index c36ba787790b..c9dc27a59474 100644
> --- a/drivers/net/designware.h
> +++ b/drivers/net/designware.h
> @@ -20,6 +20,39 @@
> #ifndef __DESIGNWARE_ETH_H
> #define __DESIGNWARE_ETH_H
>
> +struct eth_device;
> +struct mii_bus;
> +
> +struct dw_eth_dev {
> + struct eth_device netdev;
> + struct mii_bus miibus;
> +
> + void (*fix_mac_speed)(int speed);
> + u8 macaddr[6];
> + u32 tx_currdescnum;
> + u32 rx_currdescnum;
> +
> + struct dmamacdescr *tx_mac_descrtable;
> + struct dmamacdescr *rx_mac_descrtable;
> +
> + u8 *txbuffs;
> + u8 *rxbuffs;
> +
> + struct eth_mac_regs *mac_regs_p;
> + struct eth_dma_regs *dma_regs_p;
> + int phy_addr;
> + phy_interface_t interface;
> + int enh_desc;
> +
> + struct reset_control *rst;
> +};
> +
> +struct dw_eth_drvdata {
> + bool enh_desc;
> +};
> +
> +struct dw_eth_dev *dwc_drv_probe(struct device_d *dev);
> +
> #define CONFIG_TX_DESCR_NUM 16
> #define CONFIG_RX_DESCR_NUM 16
> #define CONFIG_ETH_BUFSIZE 2048
> diff --git a/drivers/net/designware_generic.c b/drivers/net/designware_generic.c
> new file mode 100644
> index 000000000000..0e5b9c067b6b
> --- /dev/null
> +++ b/drivers/net/designware_generic.c
> @@ -0,0 +1,59 @@
> +/*
> + * (C) Copyright 2010
> + * Vipin Kumar, ST Micoelectronics, vipin.kumar at st.com.
> + *
> + * See file CREDITS for list of people who contributed to this
> + * project.
> + *
> + * This program is free software; you can redistribute it and/or
> + * modify it under the terms of the GNU General Public License as
> + * published by the Free Software Foundation; either version 2 of
> + * the License, or (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU General Public License for more details.
> + *
> + */
> +
> +/*
> + * Designware ethernet IP driver for u-boot
> + */
> +
> +#include <common.h>
> +#include "designware.h"
> +
> +static struct dw_eth_drvdata dwmac_370a_drvdata = {
> + .enh_desc = 1,
> +};
> +
> +static int dwc_ether_probe(struct device_d *dev)
> +{
> + struct dw_eth_dev *dwc;
> +
> + dwc = dwc_drv_probe(dev);
> + if (IS_ERR(dwc))
> + return PTR_ERR(dwc);
> +
> + return 0;
> +}
> +
> +static __maybe_unused struct of_device_id dwc_ether_compatible[] = {
> + {
> + .compatible = "snps,dwmac-3.70a",
> + .data = &dwmac_370a_drvdata,
> + }, {
> + .compatible = "snps,dwmac-3.72a",
> + .data = &dwmac_370a_drvdata,
> + }, {
> + /* sentinel */
> + }
> +};
> +
> +static struct driver_d dwc_ether_driver = {
> + .name = "designware_eth",
> + .probe = dwc_ether_probe,
> + .of_compatible = DRV_OF_COMPAT(dwc_ether_compatible),
> +};
> +device_platform_driver(dwc_ether_driver);
> --
> 2.11.0
>
>
> _______________________________________________
> barebox mailing list
> barebox at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox
>
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
More information about the barebox
mailing list