[U-Boot] [PATCH 06/10] net: gmac_rk3288: Add RK3288 GMAC driver
Bin Meng
bmeng.cn at gmail.com
Thu Oct 1 13:30:09 CEST 2015
Hi Sjoerd,
On Thu, Oct 1, 2015 at 5:48 PM, Sjoerd Simons
<sjoerd.simons at collabora.co.uk> wrote:
> Add a new driver for the GMAC ethernet interface present in Rockchip
> RK3288 SOCs. This driver subclasses the generic design-ware driver to
> add the glue needed specifically for Rockchip.
>
> Signed-off-by: Sjoerd Simons <sjoerd.simons at collabora.co.uk>
> ---
>
> drivers/net/Kconfig | 7 +++
> drivers/net/Makefile | 1 +
> drivers/net/gmac_rk3288.c | 140 ++++++++++++++++++++++++++++++++++++++++++++++
> 3 files changed, 148 insertions(+)
> create mode 100644 drivers/net/gmac_rk3288.c
>
> diff --git a/drivers/net/Kconfig b/drivers/net/Kconfig
> index bbec6a6..312c660 100644
> --- a/drivers/net/Kconfig
> +++ b/drivers/net/Kconfig
> @@ -92,4 +92,11 @@ config PCH_GBE
> This MAC is present in Intel Platform Controller Hub EG20T. It
> supports 10/100/1000 Mbps operation.
>
> +config GMAC_RK3288
> + bool "Rockchip RK3288 Synopsys Designware Ethernet MAC"
> + depends on DM_ETH && ETH_DESIGNWARE
> + help
> + This driver provides Rockchip RK3288 network support based on the
> + Synopsys Designware driver.
> +
> endif # NETDEVICES
> diff --git a/drivers/net/Makefile b/drivers/net/Makefile
> index 150470c..f0a3992 100644
> --- a/drivers/net/Makefile
> +++ b/drivers/net/Makefile
> @@ -32,6 +32,7 @@ obj-$(CONFIG_FTGMAC100) += ftgmac100.o
> obj-$(CONFIG_FTMAC110) += ftmac110.o
> obj-$(CONFIG_FTMAC100) += ftmac100.o
> obj-$(CONFIG_GRETH) += greth.o
> +obj-$(CONFIG_GMAC_RK3288) += gmac_rk3288.o
> obj-$(CONFIG_DRIVER_TI_KEYSTONE_NET) += keystone_net.o
> obj-$(CONFIG_KS8851_MLL) += ks8851_mll.o
> obj-$(CONFIG_LAN91C96) += lan91c96.o
> diff --git a/drivers/net/gmac_rk3288.c b/drivers/net/gmac_rk3288.c
> new file mode 100644
> index 0000000..6e92c8e
> --- /dev/null
> +++ b/drivers/net/gmac_rk3288.c
> @@ -0,0 +1,140 @@
> +/*
> + * (C) Copyright 2015 Sjoerd Simons <sjoerd.simons at collabora.co.uk>
> + *
> + * SPDX-License-Identifier: GPL-2.0+
> + */
> +
> +/*
> + * Rockchip GMAC ethernet IP driver for U-Boot
> + */
nits: please use one-line comment format for this.
> +#include <common.h>
> +#include <dm.h>
> +#include <dm/pinctrl.h>
> +#include <asm/gpio.h>
> +#include <clk.h>
> +#include <phy.h>
> +#include <syscon.h>
> +#include <asm/io.h>
> +#include <asm/arch/periph.h>
> +#include <asm/arch/clock.h>
> +#include <asm/arch/grf_rk3288.h>
> +#include "designware.h"
> +
> +DECLARE_GLOBAL_DATA_PTR;
> +
> +struct gmac_rk3288_platdata {
> + struct dw_eth_pdata dw_eth_pdata;
> + int tx_delay;
> + int rx_delay;
> +};
> +
> +static int gmac_rk3288_ofdata_to_platdata(struct udevice *dev)
> +{
> + struct gmac_rk3288_platdata *pdata = dev_get_platdata(dev);
> +
> + pdata->tx_delay = fdtdec_get_int (gd->fdt_blob, dev->of_offset,
nits: please remove the space after fdtdec_get_int
> + "tx_delay", 0x30);
nits: and to make this indention match the opening ( above
> + pdata->rx_delay = fdtdec_get_int (gd->fdt_blob, dev->of_offset,
> + "rx_delay", 0x10);
nits: ditto
> +
> + return designware_eth_ofdata_to_platdata(dev);
> +}
> +
> +static int gmac_rk3288_started(struct udevice *dev)
> +{
> + struct rk3288_grf *grf;
> + struct dw_eth_dev *priv = dev_get_priv(dev);
> + int clk;
> +
> + switch (priv->phydev->speed) {
> + case 10:
> + clk = GMAC_CLK_SEL_2_5M;
> + break;
> + case 100:
> + clk = GMAC_CLK_SEL_25M;
> + break;
> + case 1000:
> + clk = GMAC_CLK_SEL_125M;
> + break;
> + default:
> + printf("Unknown phy speed: %d\n", priv->phydev->speed);
> + return -EINVAL;
> + }
> +
> + grf = syscon_get_first_range(ROCKCHIP_SYSCON_GRF);
> +
> + rk_clrsetreg(&grf->soc_con1,
> + GMAC_CLK_SEL_MASK << GMAC_CLK_SEL_SHIFT,
> + clk << GMAC_CLK_SEL_SHIFT);
> +
> + return 0;
> +}
> +
> +static int gmac_rk3288_probe(struct udevice *dev)
> +{
> + int ret;
> + struct gmac_rk3288_platdata *pdata = dev_get_platdata(dev);
> + struct dw_eth_dev *priv = dev_get_priv(dev);
> + enum periph_id periph_id;
> + struct udevice *pinctrl, *clk;
> + struct rk3288_grf *grf;
> +
> + ret = uclass_get_device(UCLASS_PINCTRL, 0, &pinctrl);
> + if (ret)
> + return ret;
> +
> + ret = pinctrl_get_periph_id(pinctrl, dev);
> + if (ret < 0)
> + return ret;
> +
> + periph_id = ret;
> + ret = pinctrl_request(pinctrl, periph_id, 0);
> + if (ret)
> + return ret;
> +
> + ret = uclass_get_device(UCLASS_CLK, CLK_GENERAL, &clk);
> + if (ret)
> + return ret;
> +
> + ret = clk_set_periph_rate(clk, periph_id, 0);
> + if (ret)
> + return ret;
> +
> + /* Set to RGMII mode */
> + grf = syscon_get_first_range(ROCKCHIP_SYSCON_GRF);
> + rk_clrsetreg(&grf->soc_con1,
> + RMII_MODE_MASK << RMII_MODE_SHIFT |
> + GMAC_PHY_INTF_SEL_MASK << GMAC_PHY_INTF_SEL_SHIFT,
> + GMAC_PHY_INTF_SEL_RGMII << GMAC_PHY_INTF_SEL_SHIFT);
> +
> + rk_clrsetreg(&grf->soc_con3,
> + RXCLK_DLY_ENA_GMAC_MASK << RXCLK_DLY_ENA_GMAC_SHIFT |
> + TXCLK_DLY_ENA_GMAC_MASK << TXCLK_DLY_ENA_GMAC_SHIFT |
> + CLK_RX_DL_CFG_GMAC_MASK << CLK_RX_DL_CFG_GMAC_SHIFT |
> + CLK_TX_DL_CFG_GMAC_MASK << CLK_TX_DL_CFG_GMAC_SHIFT,
> + RXCLK_DLY_ENA_GMAC_ENABLE << RXCLK_DLY_ENA_GMAC_SHIFT |
> + TXCLK_DLY_ENA_GMAC_ENABLE << TXCLK_DLY_ENA_GMAC_SHIFT |
> + pdata->rx_delay << CLK_RX_DL_CFG_GMAC_SHIFT |
> + pdata->tx_delay << CLK_TX_DL_CFG_GMAC_SHIFT);
> +
> + priv->started = gmac_rk3288_started;
> +
> + return designware_eth_probe(dev);
> +}
> +
> +static const struct udevice_id rk3288_gmac_ids[] = {
> + { .compatible = "rockchip,rk3288-gmac" },
> + { }
> +};
> +
> +U_BOOT_DRIVER(eth_gmac_rk3288) = {
> + .name = "gmac_rk3288",
> + .id = UCLASS_ETH,
> + .of_match = rk3288_gmac_ids,
> + .ofdata_to_platdata = gmac_rk3288_ofdata_to_platdata,
> + .probe = gmac_rk3288_probe,
> + .ops = &designware_eth_ops,
> + .priv_auto_alloc_size = sizeof(struct dw_eth_dev),
> + .platdata_auto_alloc_size = sizeof(struct gmac_rk3288_platdata),
> +.flags = DM_FLAG_ALLOC_PRIV_DMA,
nits: please indent.
> +};
> --
Regards,
Bin
More information about the U-Boot
mailing list