Hi, On Friday 25 October 2013 07:45 PM, Kamil Debski wrote: > Add support for Exynos 5250. This is work-in-progress commit. Not > for merging. > > Signed-off-by: Kamil Debski <k.debski@xxxxxxxxxxx> > Signed-off-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx> > --- > drivers/phy/Kconfig | 7 + > drivers/phy/Makefile | 1 + > drivers/phy/phy-exynos-usb.c | 10 + > drivers/phy/phy-exynos-usb.h | 1 + > drivers/phy/phy-exynos5250-usb.c | 411 ++++++++++++++++++++++++++++++++++++++ > 5 files changed, 430 insertions(+) > create mode 100644 drivers/phy/phy-exynos5250-usb.c > > diff --git a/drivers/phy/Kconfig b/drivers/phy/Kconfig > index 2f7ac0a..0f598d0 100644 > --- a/drivers/phy/Kconfig > +++ b/drivers/phy/Kconfig > @@ -36,4 +36,11 @@ config PHY_EXYNOS4212_USB > help > Enable USB PHY support for Exynos 4212 > > +config PHY_EXYNOS5250_USB > + bool "Support for Exynos 5250" > + depends on PHY_EXYNOS_USB This should be a separate driver. Not necessary to use PHY_EXYNOS_USB. > + depends on SOC_EXYNOS5250 > + help > + Enable USB PHY support for Exynos 5250 > + > endmenu > diff --git a/drivers/phy/Makefile b/drivers/phy/Makefile > index ca3dc82..0dff0dd 100644 > --- a/drivers/phy/Makefile > +++ b/drivers/phy/Makefile > @@ -6,3 +6,4 @@ obj-$(CONFIG_GENERIC_PHY) += phy-core.o > obj-$(CONFIG_PHY_EXYNOS_USB) += phy-exynos-usb.o > obj-$(CONFIG_PHY_EXYNOS4210_USB) += phy-exynos4210-usb.o > obj-$(CONFIG_PHY_EXYNOS4212_USB) += phy-exynos4212-usb.o > +obj-$(CONFIG_PHY_EXYNOS5250_USB) += phy-exynos5250-usb.o > diff --git a/drivers/phy/phy-exynos-usb.c b/drivers/phy/phy-exynos-usb.c > index d4a26df..172b774 100644 > --- a/drivers/phy/phy-exynos-usb.c > +++ b/drivers/phy/phy-exynos-usb.c > @@ -212,6 +212,10 @@ extern const struct uphy_config exynos4210_uphy_config; > extern const struct uphy_config exynos4212_uphy_config; > #endif > > +#ifdef CONFIG_PHY_EXYNOS5250_USB > +extern const struct uphy_config exynos5250_uphy_config; > +#endif > + > static const struct of_device_id exynos_uphy_of_match[] = { > #ifdef CONFIG_PHY_EXYNOS4210_USB > { > @@ -225,6 +229,12 @@ static const struct of_device_id exynos_uphy_of_match[] = { > .data = &exynos4212_uphy_config, > }, > #endif > +#ifdef CONFIG_PHY_EXYNOS5250_USB > + { > + .compatible = "samsung,exynos5250-usbphy", > + .data = &exynos5250_uphy_config, > + }, > +#endif > { }, > }; > > diff --git a/drivers/phy/phy-exynos-usb.h b/drivers/phy/phy-exynos-usb.h > index f45cb3c..a9febfa 100644 > --- a/drivers/phy/phy-exynos-usb.h > +++ b/drivers/phy/phy-exynos-usb.h > @@ -42,6 +42,7 @@ enum samsung_cpu_type { > TYPE_S3C64XX, > TYPE_EXYNOS4210, > TYPE_EXYNOS4212, > + TYPE_EXYNOS5250, No cpu types here. > }; > > enum uphy_state { > diff --git a/drivers/phy/phy-exynos5250-usb.c b/drivers/phy/phy-exynos5250-usb.c > new file mode 100644 > index 0000000..156093b > --- /dev/null > +++ b/drivers/phy/phy-exynos5250-usb.c > @@ -0,0 +1,411 @@ > +/* > + * Samsung S5P/EXYNOS SoC series USB PHY driver > + * > + * Copyright (C) 2013 Samsung Electronics Co., Ltd. > + * Author: Kamil Debski <k.debski@xxxxxxxxxxx> > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License version 2 as > + * published by the Free Software Foundation. > + */ > + > +#include <linux/clk.h> > +#include <linux/delay.h> > +#include <linux/io.h> > +#include <linux/kernel.h> > +#include <linux/module.h> > +#include <linux/of.h> > +#include <linux/of_address.h> > +#include <linux/phy/phy.h> > +#include <linux/platform_device.h> > +#include <linux/spinlock.h> > +#include "phy-exynos-usb.h" > + > +/* Exynos USB PHY registers */ > +#define EXYNOS_5250_REFCLKSEL_CRYSTAL 0x0 > +#define EXYNOS_5250_REFCLKSEL_XO 0x1 > +#define EXYNOS_5250_REFCLKSEL_CLKCORE 0x2 > + > +#define EXYNOS_5250_FSEL_9MHZ6 0x0 > +#define EXYNOS_5250_FSEL_10MHZ 0x1 > +#define EXYNOS_5250_FSEL_12MHZ 0x2 > +#define EXYNOS_5250_FSEL_19MHZ2 0x3 > +#define EXYNOS_5250_FSEL_20MHZ 0x4 > +#define EXYNOS_5250_FSEL_24MHZ 0x5 > +#define EXYNOS_5250_FSEL_50MHZ 0x7 > + > +/* Normal host */ > +#define EXYNOS_5250_HOSTPHYCTRL0 0x0 > + > +#define EXYNOS_5250_HOSTPHYCTRL0_PHYSWRSTALL (0x1 << 31) > +#define EXYNOS_5250_HOSTPHYCTRL0_REFCLKSEL_SHIFT 19 > +#define EXYNOS_5250_HOSTPHYCTRL0_REFCLKSEL_MASK \ > + (0x3 << EXYNOS_5250_HOSTPHYCTRL0_REFCLKSEL_SHIFT) > +#define EXYNOS_5250_HOSTPHYCTRL0_FSEL_SHIFT 16 > +#define EXYNOS_5250_HOSTPHYCTRL0_FSEL_MASK \ > + (0x7 << EXYNOS_5250_HOSTPHYCTRL0_FSEL_SHIFT) > +#define EXYNOS_5250_HOSTPHYCTRL0_TESTBURNIN (0x1 << 11) > +#define EXYNOS_5250_HOSTPHYCTRL0_RETENABLE (0x1 << 10) > +#define EXYNOS_5250_HOSTPHYCTRL0_COMMON_ON_N (0x1 << 9) > +#define EXYNOS_5250_HOSTPHYCTRL0_VATESTENB_MASK (0x3 << 7) > +#define EXYNOS_5250_HOSTPHYCTRL0_VATESTENB_DUAL (0x0 << 7) > +#define EXYNOS_5250_HOSTPHYCTRL0_VATESTENB_ID0 (0x1 << 7) > +#define EXYNOS_5250_HOSTPHYCTRL0_VATESTENB_ANALOGTEST (0x2 << 7) > +#define EXYNOS_5250_HOSTPHYCTRL0_SIDDQ (0x1 << 6) > +#define EXYNOS_5250_HOSTPHYCTRL0_FORCESLEEP (0x1 << 5) > +#define EXYNOS_5250_HOSTPHYCTRL0_FORCESUSPEND (0x1 << 4) > +#define EXYNOS_5250_HOSTPHYCTRL0_WORDINTERFACE (0x1 << 3) > +#define EXYNOS_5250_HOSTPHYCTRL0_UTMISWRST (0x1 << 2) > +#define EXYNOS_5250_HOSTPHYCTRL0_LINKSWRST (0x1 << 1) > +#define EXYNOS_5250_HOSTPHYCTRL0_PHYSWRST (0x1 << 0) > + > +/* HSIC0 & HSCI1 */ > +#define EXYNOS_5250_HOSTPHYCTRL1 0x10 > +#define EXYNOS_5250_HOSTPHYCTRL2 0x20 > + > +#define EXYNOS_5250_HOSTPHYCTRLX_REFCLKSEL_MASK (0x3 << 23) > +#define EXYNOS_5250_HOSTPHYCTRLX_REFCLKDIV_MASK (0x7f << 16) > +#define EXYNOS_5250_HOSTPHYCTRLX_SIDDQ (0x1 << 6) > +#define EXYNOS_5250_HOSTPHYCTRLX_FORCESLEEP (0x1 << 5) > +#define EXYNOS_5250_HOSTPHYCTRLX_FORCESUSPEND (0x1 << 4) > +#define EXYNOS_5250_HOSTPHYCTRLX_WORDINTERFACE (0x1 << 3) > +#define EXYNOS_5250_HOSTPHYCTRLX_UTMISWRST (0x1 << 2) > +#define EXYNOS_5250_HOSTPHYCTRLX_PHYSWRST (0x1 << 0) > + > +/* EHCI control */ > +#define EXYNOS_5250_HOSTEHCICTRL 0x30 > +#define EXYNOS_5250_HOSTEHCICTRL_ENAINCRXALIGN (0x1 << 29) > +#define EXYNOS_5250_HOSTEHCICTRL_ENAINCR4 (0x1 << 28) > +#define EXYNOS_5250_HOSTEHCICTRL_ENAINCR8 (0x1 << 27) > +#define EXYNOS_5250_HOSTEHCICTRL_ENAINCR16 (0x1 << 26) > +#define EXYNOS_5250_HOSTEHCICTRL_AUTOPPDONOVRCUREN (0x1 << 25) > +#define EXYNOS_5250_HOSTEHCICTRL_FLADJVAL0_SHIFT 19 > +#define EXYNOS_5250_HOSTEHCICTRL_FLADJVAL0_MASK \ > + (0x3f << EXYNOS_5250_HOSTEHCICTRL_FLADJVAL0_SHIFT) > +#define EXYNOS_5250_HOSTEHCICTRL_FLADJVAL1_SHIFT 13 > +#define EXYNOS_5250_HOSTEHCICTRL_FLADJVAL1_MASK \ > + (0x3f << EXYNOS_5250_HOSTEHCICTRL_FLADJVAL1_SHIFT) > +#define EXYNOS_5250_HOSTEHCICTRL_FLADJVAL2_SHIFT 7 > +#define EXYNOS_5250_HOSTEHCICTRL_FLADJVAL0_MASK \ > + (0x3f << EXYNOS_5250_HOSTEHCICTRL_FLADJVAL0_SHIFT) > +#define EXYNOS_5250_HOSTEHCICTRL_FLADJVALHOST_SHIFT 1 > +#define EXYNOS_5250_HOSTEHCICTRL_FLADJVALHOST_MASK \ > + (0x1 << EXYNOS_5250_HOSTEHCICTRL_FLADJVALHOST_SHIFT) > +#define EXYNOS_5250_HOSTEHCICTRL_SIMULATIONMODE (0x1 << 0) > + > +/* OHCI control */ > +#define EXYNOS_5250_HOSTOHCICTRL 0x34 > +#define EXYNOS_5250_HOSTOHCICTRL_FRAMELENVAL_SHIFT 1 > +#define EXYNOS_5250_HOSTOHCICTRL_FRAMELENVAL_MASK \ > + (0x3ff << EXYNOS_5250_HOSTOHCICTRL_FRAMELENVAL_SHIFT) > +#define EXYNOS_5250_HOSTOHCICTRL_FRAMELENVALEN (0x1 << 0) > + > +/* USBOTG */ > +#define EXYNOS_5250_USBOTGSYS 0x38 > +#define EXYNOS_5250_USBOTGSYS_PHYLINK_SW_RESET (0x1 << 14) > +#define EXYNOS_5250_USBOTGSYS_LINK_SW_RST_UOTG (0x1 << 13) > +#define EXYNOS_5250_USBOTGSYS_PHY_SW_RST (0x1 << 12) > +#define EXYNOS_5250_USBOTGSYS_REFCLKSEL_SHIFT 9 > +#define EXYNOS_5250_USBOTGSYS_REFCLKSEL_MASK \ > + (0x3 << EXYNOS_5250_USBOTGSYS_REFCLKSEL_SHIFT) > +#define EXYNOS_5250_USBOTGSYS_ID_PULLUP (0x1 << 8) > +#define EXYNOS_5250_USBOTGSYS_COMMON_ON (0x1 << 7) > +#define EXYNOS_5250_USBOTGSYS_FSEL_SHIFT 4 > +#define EXYNOS_5250_USBOTGSYS_FSEL_MASK \ > + (0x3 << EXYNOS_5250_USBOTGSYS_FSEL_SHIFT) > +#define EXYNOS_5250_USBOTGSYS_FORCE_SLEEP (0x1 << 3) > +#define EXYNOS_5250_USBOTGSYS_OTGDISABLE (0x1 << 2) > +#define EXYNOS_5250_USBOTGSYS_SIDDQ_UOTG (0x1 << 1) > +#define EXYNOS_5250_USBOTGSYS_FORCE_SUSPEND (0x1 << 0) > + > +/* Isolation, configured in the power management unit */ > +#define EXYNOS_5250_USB_ISOL_OTG_OFFSET 0x0 > +#define EXYNOS_5250_USB_ISOL_OTG (1 << 0) > +#define EXYNOS_5250_USB_ISOL_HOST_OFFSET 0x4 > +#define EXYNOS_5250_USB_ISOL_HOST (1 << 0) > + > +enum exynos4x12_phy_id { > + EXYNOS5250_DEVICE, > + EXYNOS5250_HOST, > + EXYNOS5250_HSIC0, > + EXYNOS5250_HSIC1, > + EXYNOS5250_NUM_PHYS, > +}; > + > +/* exynos5250_rate_to_clk() converts the supplied clock rate to the value that > + * can be written to the phy register. */ Follow coding guidelines ;-) > +static u32 exynos5250_rate_to_clk(unsigned long rate) > +{ > + unsigned int clksel; > + > + /* EXYNOS_5250_FSEL_MASK */ > + > + switch (rate) { > + case 9600 * KHZ: > + clksel = EXYNOS_5250_FSEL_9MHZ6; > + break; > + case 10 * MHZ: > + clksel = EXYNOS_5250_FSEL_10MHZ; > + break; > + case 12 * MHZ: > + clksel = EXYNOS_5250_FSEL_12MHZ; > + break; > + case 19200 * KHZ: > + clksel = EXYNOS_5250_FSEL_19MHZ2; > + break; > + case 20 * MHZ: > + clksel = EXYNOS_5250_FSEL_20MHZ; > + break; > + case 24 * MHZ: > + clksel = EXYNOS_5250_FSEL_24MHZ; > + break; > + case 50 * MHZ: > + clksel = EXYNOS_5250_FSEL_50MHZ; > + break; > + default: > + clksel = CLKSEL_ERROR; > + } > + > + return clksel; > +} > + > +static void exynos5250_isol(struct uphy_instance *inst, bool on) > +{ > + struct uphy_driver *drv = inst->drv; > + u32 offset; > + u32 mask; > + u32 tmp; > + > + if (!drv->reg_isol) > + return; > + > + switch (inst->cfg->id) { > + case EXYNOS5250_DEVICE: > + offset = EXYNOS_5250_USB_ISOL_OTG_OFFSET; > + mask = EXYNOS_5250_USB_ISOL_OTG; > + break; > + case EXYNOS5250_HOST: > + offset = EXYNOS_5250_USB_ISOL_HOST_OFFSET; > + mask = EXYNOS_5250_USB_ISOL_HOST; > + break; > + default: > + return; > + }; > + > + tmp = readl(drv->reg_isol + offset); > + if (on) > + tmp &= ~mask; > + else > + tmp |= mask; > + writel(tmp, drv->reg_isol + offset); > +} > + > +static void exynos5250_phy_pwr(struct uphy_instance *inst, bool on) > +{ > + struct uphy_driver *drv = inst->drv; > + u32 ctrl0; > + u32 otg; > + u32 ehci; > + u32 ohci; > + > + switch (inst->cfg->id) { > + case EXYNOS5250_DEVICE: > + writel(0, drv->reg_mode); > + > + /* OTG configuration */ > + otg = readl(drv->reg_phy + EXYNOS_5250_USBOTGSYS); > + /* The clock */ > + otg &= ~EXYNOS_5250_USBOTGSYS_FSEL_MASK; > + otg |= inst->clk << EXYNOS_5250_USBOTGSYS_FSEL_SHIFT; > + /* Reset */ > + otg &= ~(EXYNOS_5250_USBOTGSYS_FORCE_SUSPEND | > + EXYNOS_5250_USBOTGSYS_FORCE_SLEEP | > + EXYNOS_5250_USBOTGSYS_SIDDQ_UOTG); > + /* TODO: Clear 4 bits as the old driver does. */ > + otg |= EXYNOS_5250_USBOTGSYS_PHY_SW_RST | > + EXYNOS_5250_USBOTGSYS_PHYLINK_SW_RESET | > + EXYNOS_5250_USBOTGSYS_LINK_SW_RST_UOTG | > + EXYNOS_5250_USBOTGSYS_OTGDISABLE; > + /* Ref clock */ > + otg &= EXYNOS_5250_USBOTGSYS_REFCLKSEL_MASK; > + otg |= EXYNOS_5250_REFCLKSEL_CLKCORE << > + EXYNOS_5250_USBOTGSYS_REFCLKSEL_SHIFT; > + writel(otg, drv->reg_phy + EXYNOS_5250_USBOTGSYS); > + udelay(10); > + otg &= ~(EXYNOS_5250_USBOTGSYS_PHY_SW_RST | > + EXYNOS_5250_USBOTGSYS_LINK_SW_RST_UOTG | > + EXYNOS_5250_USBOTGSYS_PHYLINK_SW_RESET); > + > + > + break; > + case EXYNOS5250_HOST: > + /* Host registers configuration */ > + ctrl0 = readl(drv->reg_phy + EXYNOS_5250_HOSTPHYCTRL0); > + /* The clock */ > + ctrl0 &= ~EXYNOS_5250_HOSTPHYCTRL0_FSEL_MASK; > + ctrl0 |= inst->clk << EXYNOS_5250_HOSTPHYCTRL0_FSEL_SHIFT; > + > + /* Reset */ > + ctrl0 &= ~( EXYNOS_5250_HOSTPHYCTRL0_PHYSWRST | > + EXYNOS_5250_HOSTPHYCTRL0_PHYSWRSTALL | > + EXYNOS_5250_HOSTPHYCTRL0_SIDDQ | > + EXYNOS_5250_HOSTPHYCTRL0_FORCESUSPEND | > + EXYNOS_5250_HOSTPHYCTRL0_FORCESLEEP); > + ctrl0 |= EXYNOS_5250_HOSTPHYCTRL0_LINKSWRST | > + EXYNOS_5250_HOSTPHYCTRL0_UTMISWRST | > + EXYNOS_5250_HOSTPHYCTRL0_COMMON_ON_N; > + writel(ctrl0, drv->reg_phy + EXYNOS_5250_HOSTPHYCTRL0); > + udelay(10); > + ctrl0 &= ~( EXYNOS_5250_HOSTPHYCTRL0_LINKSWRST | > + EXYNOS_5250_HOSTPHYCTRL0_UTMISWRST); > + writel(ctrl0, drv->reg_phy + EXYNOS_5250_HOSTPHYCTRL0); > + > + /* OTG configuration */ > + otg = readl(drv->reg_phy + EXYNOS_5250_USBOTGSYS); > + /* The clock */ > + otg &= ~EXYNOS_5250_USBOTGSYS_FSEL_MASK; > + otg |= inst->clk << EXYNOS_5250_USBOTGSYS_FSEL_SHIFT; > + /* Reset */ > + otg &= ~(EXYNOS_5250_USBOTGSYS_FORCE_SUSPEND | > + EXYNOS_5250_USBOTGSYS_FORCE_SLEEP | > + EXYNOS_5250_USBOTGSYS_SIDDQ_UOTG); > + /* TODO: Clear 4 bits as the old driver does. */ > + otg |= EXYNOS_5250_USBOTGSYS_PHY_SW_RST | > + EXYNOS_5250_USBOTGSYS_PHYLINK_SW_RESET | > + EXYNOS_5250_USBOTGSYS_LINK_SW_RST_UOTG | > + EXYNOS_5250_USBOTGSYS_OTGDISABLE; > + /* Ref clock */ > + otg &= EXYNOS_5250_USBOTGSYS_REFCLKSEL_MASK; > + otg |= EXYNOS_5250_REFCLKSEL_CLKCORE << > + EXYNOS_5250_USBOTGSYS_REFCLKSEL_SHIFT; > + writel(otg, drv->reg_phy + EXYNOS_5250_USBOTGSYS); > + udelay(10); > + otg &= ~(EXYNOS_5250_USBOTGSYS_PHY_SW_RST | > + EXYNOS_5250_USBOTGSYS_LINK_SW_RST_UOTG | > + EXYNOS_5250_USBOTGSYS_PHYLINK_SW_RESET); > + > + /* Enable EHCI DMA burst */ > + ehci = readl(drv->reg_phy + EXYNOS_5250_HOSTEHCICTRL); > + ehci |= EXYNOS_5250_HOSTEHCICTRL_ENAINCRXALIGN | > + EXYNOS_5250_HOSTEHCICTRL_ENAINCR4 | > + EXYNOS_5250_HOSTEHCICTRL_ENAINCR8 | > + EXYNOS_5250_HOSTEHCICTRL_ENAINCR16; > + writel(ehci, drv->reg_phy + EXYNOS_5250_HOSTEHCICTRL); > + > + /* OHCI settings */ > + ohci = readl(drv->reg_phy + EXYNOS_5250_HOSTOHCICTRL); > + /* Let's do some undocumented magic (based on the old driver) */ > + ohci |= 0x1 << 3; > + writel(ohci, drv->reg_phy + EXYNOS_5250_HOSTOHCICTRL); > + > + break; > + } > +} > + > +static int exynos5250_power_on(struct uphy_instance *inst) > +{ > + struct uphy_driver *drv = inst->drv; > + > + if (inst->state == STATE_ON) { > + dev_err(drv->dev, "usb phy \"%s\" already on", > + inst->cfg->label); > + return -ENODEV; > + } > + > + inst->state = STATE_ON; > + inst->ref_cnt++; > + if (inst->ref_cnt > 1) > + return 0; > + > + exynos5250_phy_pwr(inst, 1); > + exynos5250_isol(inst, 0); > + > + /* Power on the device, as it is necessary for HSIC to work */ > + if (inst->cfg->id == EXYNOS5250_HSIC0) { > + struct uphy_instance *device = > + &drv->uphy_instances[EXYNOS5250_DEVICE]; > + device->ref_cnt++; > + if (device->ref_cnt > 1) > + return 0; > + exynos5250_phy_pwr(device, 1); > + exynos5250_isol(device, 0); > + } > + > + return 0; > +} > + > +static int exynos5250_power_off(struct uphy_instance *inst) > +{ > + struct uphy_driver *drv = inst->drv; > + > + if (inst->state == STATE_OFF) { > + dev_err(drv->dev, "usb phy \"%s\" already off", > + inst->cfg->label); > + return -EINVAL; > + } > + > + inst->state = STATE_OFF; > + inst->ref_cnt--; > + > + if (inst->ref_cnt > 0) > + return 0; > + > + exynos5250_isol(inst, 1); > + exynos5250_phy_pwr(inst, 0); > + > + if (inst->cfg->id == EXYNOS5250_HSIC0) { > + struct uphy_instance *device = > + &drv->uphy_instances[EXYNOS5250_DEVICE]; > + device->ref_cnt--; > + if (device->ref_cnt > 0) > + return 0; > + exynos5250_isol(device, 1); > + exynos5250_phy_pwr(device, 0); > + } > + > + return 0; > +} > + > + > +static const struct common_phy exynos5250_phys[] = { > + { > + .label = "device", > + .type = PHY_DEVICE, > + .id = EXYNOS5250_DEVICE, > + .rate_to_clk = exynos5250_rate_to_clk, > + .power_on = exynos5250_power_on, > + .power_off = exynos5250_power_off, > + }, > + { > + .label = "host", > + .type = PHY_HOST, > + .id = EXYNOS5250_HOST, > + .rate_to_clk = exynos5250_rate_to_clk, > + .power_on = exynos5250_power_on, > + .power_off = exynos5250_power_off, > + }, > + { > + .label = "hsic0", > + .type = PHY_HOST, > + .id = EXYNOS5250_HSIC0, > + .rate_to_clk = exynos5250_rate_to_clk, > + .power_on = exynos5250_power_on, > + .power_off = exynos5250_power_off, > + }, > + { > + .label = "hsic1", > + .type = PHY_HOST, > + .id = EXYNOS5250_HSIC1, > + .rate_to_clk = exynos5250_rate_to_clk, > + .power_on = exynos5250_power_on, > + .power_off = exynos5250_power_off, > + }, > + {}, > +}; All this can be removed if you don't use exynos common PHY. Thanks Kishon -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html