Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751997AbaFFGjK (ORCPT ); Fri, 6 Jun 2014 02:39:10 -0400 Received: from mail-qg0-f42.google.com ([209.85.192.42]:38440 "EHLO mail-qg0-f42.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750710AbaFFGjI convert rfc822-to-8bit (ORCPT ); Fri, 6 Jun 2014 02:39:08 -0400 MIME-Version: 1.0 In-Reply-To: <1401983326-19205-5-git-send-email-antoine.tenart@free-electrons.com> References: <1401983326-19205-1-git-send-email-antoine.tenart@free-electrons.com> <1401983326-19205-5-git-send-email-antoine.tenart@free-electrons.com> Date: Fri, 6 Jun 2014 12:09:06 +0530 X-Google-Sender-Auth: L_AbQpFX2E0ymuWYB5T_Q6pf2Ic Message-ID: Subject: Re: [PATCH 4/9] usb: phy: add the Berlin USB PHY driver From: Vivek Gautam To: =?UTF-8?Q?Antoine_T=C3=A9nart?= Cc: kishon , sebastian.hesselbarth@gmail.com, Felipe Balbi , alexandre.belloni@free-electrons.com, thomas.petazzoni@free-electrons.com, zmxu@marvell.com, jszhang@marvell.com, "linux-arm-kernel@lists.infradead.org" , Linux USB Mailing List , "linux-kernel@vger.kernel.org" Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org + Kishon. Hi, On Thu, Jun 5, 2014 at 9:18 PM, Antoine Ténart wrote: > Add the driver driving the Marvell Berlin USB PHY. This allows to > initialize the PHY and to use it from the USB driver later. Just out of curiosity, going forward we would like to have phy drivers based on generic phy framework (drivers/phy). Any particular reason that we are still adding phy drivers in usb-phy layer ? Looking at it, seems like it can very well be written based on phy framework. > > Signed-off-by: Antoine Ténart > --- > drivers/usb/phy/Kconfig | 9 ++ > drivers/usb/phy/Makefile | 1 + > drivers/usb/phy/phy-berlin-usb.c | 223 +++++++++++++++++++++++++++++++++++++++ > 3 files changed, 233 insertions(+) > create mode 100644 drivers/usb/phy/phy-berlin-usb.c > > diff --git a/drivers/usb/phy/Kconfig b/drivers/usb/phy/Kconfig > index 416e0c8cf6ff..8be8d4afc428 100644 > --- a/drivers/usb/phy/Kconfig > +++ b/drivers/usb/phy/Kconfig > @@ -27,6 +27,15 @@ config AB8500_USB > This transceiver supports high and full speed devices plus, > in host mode, low speed. > > +config BERLIN_USBPHY > + tristate "Marvell Berlin USB Transceiver Driver" > + depends on ARCH_BERLIN > + select USB_PHY > + help > + Enable this to support the USB tranceiver on Marvell Berlin > + SoCs. > + > + > config FSL_USB2_OTG > bool "Freescale USB OTG Transceiver Driver" > depends on USB_EHCI_FSL && USB_FSL_USB2 && USB_OTG_FSM && PM_RUNTIME > diff --git a/drivers/usb/phy/Makefile b/drivers/usb/phy/Makefile > index f8fa719a31b9..9253f59cf82a 100644 > --- a/drivers/usb/phy/Makefile > +++ b/drivers/usb/phy/Makefile > @@ -8,6 +8,7 @@ obj-$(CONFIG_USB_OTG_FSM) += phy-fsm-usb.o > # transceiver drivers, keep the list sorted > > obj-$(CONFIG_AB8500_USB) += phy-ab8500-usb.o > +obj-$(CONFIG_BERLIN_USBPHY) += phy-berlin-usb.o > obj-$(CONFIG_FSL_USB2_OTG) += phy-fsl-usb.o > obj-$(CONFIG_ISP1301_OMAP) += phy-isp1301-omap.o > obj-$(CONFIG_MV_U3D_PHY) += phy-mv-u3d-usb.o > diff --git a/drivers/usb/phy/phy-berlin-usb.c b/drivers/usb/phy/phy-berlin-usb.c > new file mode 100644 > index 000000000000..79416668a71b > --- /dev/null > +++ b/drivers/usb/phy/phy-berlin-usb.c > @@ -0,0 +1,223 @@ > +/* > + * Copyright (C) 2014 Marvell Technology Group Ltd. > + * > + * Antoine Ténart > + * Jisheng Zhang > + * > + * This file is licensed under the terms of the GNU General Public > + * License version 2. This program is licensed "as is" without any > + * warranty of any kind, whether express or implied. > + */ > + > +#include > +#include > +#include > +#include > +#include > +#include > +#include > + > +#define USB_PHY_PLL 0x04 > +#define USB_PHY_PLL_CONTROL 0x08 > +#define USB_PHY_TX_CTRL0 0x10 > +#define USB_PHY_TX_CTRL1 0x14 > +#define USB_PHY_TX_CTRL2 0x18 > +#define USB_PHY_RX_CTRL 0x20 > +#define USB_PHY_ANALOG 0x34 > + > +/* USB_PHY_PLL */ > +#define CLK_REF_DIV(x) ((x) << 4) > +#define FEEDBACK_CLK_DIV(x) ((x) << 8) > + > +/* USB_PHY_PLL_CONTROL */ > +#define CLK_STABLE (0x1 << 0) > +#define PLL_CTRL_PIN (0x1 << 1) > +#define PLL_CTRL_REG (0x1 << 2) > +#define PLL_ON (0x1 << 3) > +#define PHASE_OFF_TOL_125 (0x0 << 5) > +#define PHASE_OFF_TOL_250 (0x1 << 5) > +#define KVC0_CALIB (0x0 << 9) > +#define KVC0_REG_CTRL (0x1 << 9) > +#define KVC0_HIGH (0x0 << 10) > +#define KVC0_LOW (0x3 << 10) > +#define CLK_BLK_EN (0x1 << 13) > + > +/* USB_PHY_TX_CTRL0 */ > +#define EXT_HS_RCAL_EN (0x1 << 3) > +#define EXT_FS_RCAL_EN (0x1 << 4) > +#define IMPCAL_VTH_DIV(x) ((x) << 5) > +#define EXT_RS_RCAL_DIV(x) ((x) << 8) > +#define EXT_FS_RCAL_DIV(x) ((x) << 12) > + > +/* USB_PHY_TX_CTRL1 */ > +#define TX_VDD15_14 (0x0 << 4) > +#define TX_VDD15_15 (0x1 << 4) > +#define TX_VDD15_16 (0x2 << 4) > +#define TX_VDD15_17 (0x3 << 4) > +#define TX_VDD12_VDD (0x0 << 6) > +#define TX_VDD12_11 (0x1 << 6) > +#define TX_VDD12_12 (0x2 << 6) > +#define TX_VDD12_13 (0x3 << 6) > +#define LOW_VDD_EN (0x1 << 8) > +#define TX_OUT_AMP(x) ((x) << 9) > + > +/* USB_PHY_TX_CTRL2 */ > +#define TX_CHAN_CTRL_REG(x) ((x) << 0) > +#define DRV_SLEWRATE(x) ((x) << 4) > +#define IMP_CAL_FS_HS_DLY_0 (0x0 << 6) > +#define IMP_CAL_FS_HS_DLY_1 (0x1 << 6) > +#define IMP_CAL_FS_HS_DLY_2 (0x2 << 6) > +#define IMP_CAL_FS_HS_DLY_3 (0x3 << 6) > +#define FS_DRV_EN_MASK(x) ((x) << 8) > +#define HS_DRV_EN_MASK(x) ((x) << 12) > + > +/* USB_PHY_RX_CTRL */ > +#define PHASE_FREEZE_DLY_2_CL (0x0 << 0) > +#define PHASE_FREEZE_DLY_4_CL (0x1 << 0) > +#define ACK_LENGTH_8_CL (0x0 << 2) > +#define ACK_LENGTH_12_CL (0x1 << 2) > +#define ACK_LENGTH_16_CL (0x2 << 2) > +#define ACK_LENGTH_20_CL (0x3 << 2) > +#define SQ_LENGTH_3 (0x0 << 4) > +#define SQ_LENGTH_6 (0x1 << 4) > +#define SQ_LENGTH_9 (0x2 << 4) > +#define SQ_LENGTH_12 (0x3 << 4) > +#define DISCON_THRESHOLD_260 (0x0 << 6) > +#define DISCON_THRESHOLD_270 (0x1 << 6) > +#define DISCON_THRESHOLD_280 (0x2 << 6) > +#define DISCON_THRESHOLD_290 (0x3 << 6) > +#define SQ_THRESHOLD(x) ((x) << 8) > +#define LPF_COEF(x) ((x) << 12) > +#define INTPL_CUR_10 (0x0 << 14) > +#define INTPL_CUR_20 (0x1 << 14) > +#define INTPL_CUR_30 (0x2 << 14) > +#define INTPL_CUR_40 (0x3 << 14) > + > +/* USB_PHY_ANALOG */ > +#define ANA_PWR_UP (0x1 << 1) > +#define ANA_PWR_DOWN (0x1 << 2) > +#define V2I_VCO_RATIO(x) ((x) << 7) > +#define R_ROTATE_90 (0x0 << 10) > +#define R_ROTATE_0 (0x1 << 10) > +#define MODE_TEST_EN (0x1 << 11) > +#define ANA_TEST_DC_CTRL(x) ((x) << 12) > + > +#define to_berlin_phy_priv(p) container_of((p), struct berlin_phy_priv, phy) > + > +struct berlin_phy_priv { > + void __iomem *base; > + struct usb_phy phy; > + struct reset_control *rst_ctrl; > + int pwr_gpio; > +}; > + > +static int berlin_phy_init(struct usb_phy *phy) > +{ > + struct berlin_phy_priv *priv = to_berlin_phy_priv(phy); > + int ret; > + > + reset_control_reset(priv->rst_ctrl); > + > + writel(CLK_REF_DIV(0xc) | FEEDBACK_CLK_DIV(0x54), > + priv->base + USB_PHY_PLL); > + writel(CLK_STABLE | PLL_CTRL_REG | PHASE_OFF_TOL_250 | KVC0_REG_CTRL | > + CLK_BLK_EN, priv->base + USB_PHY_PLL_CONTROL); > + writel(V2I_VCO_RATIO(0x5) | R_ROTATE_0 | ANA_TEST_DC_CTRL(0x5), > + priv->base + USB_PHY_ANALOG); > + writel(PHASE_FREEZE_DLY_4_CL | ACK_LENGTH_16_CL | SQ_LENGTH_12 | > + DISCON_THRESHOLD_260 | SQ_THRESHOLD(0xa) | LPF_COEF(0x2) | > + INTPL_CUR_30, priv->base + USB_PHY_RX_CTRL); > + > + writel(TX_VDD12_13 | TX_OUT_AMP(0x3), priv->base + USB_PHY_TX_CTRL1); > + writel(EXT_HS_RCAL_EN | IMPCAL_VTH_DIV(0x3) | EXT_RS_RCAL_DIV(0x4), > + priv->base + USB_PHY_TX_CTRL0); > + > + writel(EXT_HS_RCAL_EN | IMPCAL_VTH_DIV(0x3) | EXT_RS_RCAL_DIV(0x4) | > + EXT_FS_RCAL_DIV(0x2), priv->base + USB_PHY_TX_CTRL0); > + > + writel(EXT_HS_RCAL_EN | IMPCAL_VTH_DIV(0x3) | EXT_RS_RCAL_DIV(0x4), > + priv->base + USB_PHY_TX_CTRL0); > + writel(TX_CHAN_CTRL_REG(0xf) | DRV_SLEWRATE(0x3) | IMP_CAL_FS_HS_DLY_3 | > + FS_DRV_EN_MASK(0xd), priv->base + USB_PHY_TX_CTRL2); > + > + ret = gpio_direction_output(priv->pwr_gpio, 0); > + if (ret) > + return ret; > + > + gpio_set_value(priv->pwr_gpio, 1); > + > + return 0; > +} > + > +static void berlin_phy_shutdown(struct usb_phy *phy) > +{ > + struct berlin_phy_priv *priv = to_berlin_phy_priv(phy); > + > + gpio_set_value(priv->pwr_gpio, 0); > +} > + > +static int berlin_phy_probe(struct platform_device *pdev) > +{ > + struct device_node *np = pdev->dev.of_node; > + struct berlin_phy_priv *priv; > + struct resource *res; > + int ret, gpio; > + > + priv = devm_kzalloc(&pdev->dev, sizeof(*priv), GFP_KERNEL); > + if (!priv) > + return -ENOMEM; > + > + res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > + priv->base = devm_ioremap_resource(&pdev->dev, res); > + if (IS_ERR(priv->base)) > + return PTR_ERR(priv->base); > + > + priv->rst_ctrl = devm_reset_control_get(&pdev->dev, NULL); > + if (IS_ERR(priv->rst_ctrl)) { > + ret = PTR_ERR(priv->rst_ctrl); > + dev_err(&pdev->dev, "cannot get reset controller: %d\n", ret); > + return ret; > + } > + > + gpio = of_get_named_gpio(np, "power-gpio", 0); > + if (!gpio_is_valid(gpio)) > + return gpio; > + > + ret = gpio_request(gpio, "power-gpio"); > + if (ret) { > + dev_err(&pdev->dev, "cannot request GPIO %d", gpio); > + return ret; > + } > + priv->pwr_gpio = gpio; > + > + priv->phy.io_priv = priv->base; > + priv->phy.dev = &pdev->dev; > + priv->phy.label = "phy-berlin-usb"; > + priv->phy.init = berlin_phy_init; > + priv->phy.shutdown = berlin_phy_shutdown; > + priv->phy.type = USB_PHY_TYPE_USB2; > + > + platform_set_drvdata(pdev, priv); > + > + return usb_add_phy_dev(&priv->phy); > +} > + > +static const struct of_device_id phy_berlin_sata_of_match[] = { > + { .compatible = "marvell,berlin-usbphy" }, > + { }, > +}; > +MODULE_DEVICE_TABLE(of, phy_berlin_sata_of_match); > + > +static struct platform_driver phy_berlin_usb_driver = { > + .probe = berlin_phy_probe, > + .driver = { > + .name = "phy-berlin-usb", > + .owner = THIS_MODULE, > + .of_match_table = phy_berlin_sata_of_match, > + }, > +}; > +module_platform_driver(phy_berlin_usb_driver); > + > +MODULE_AUTHOR("Antoine Ténart "); > +MODULE_DESCRIPTION("Marvell Berlin USB PHY driver"); > +MODULE_LICENSE("GPL"); > -- > 1.9.1 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-usb" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Best Regards Vivek Gautam Samsung R&D Institute, Bangalore India -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/