From: Vivek Gautam <gautamvivek1987@gmail.com>
To: Jingoo Han <jg1.han@samsung.com>
Cc: Vivek Gautam <gautam.vivek@samsung.com>,
linux-usb@vger.kernel.org, linux-samsung-soc@vger.kernel.org,
devicetree-discuss@lists.ozlabs.org, gregkh@linuxfoundation.org,
stern@rowland.harvard.edu, balbi@ti.com, rob.herring@calxeda.com,
kgene.kim@samsung.com, thomas.abraham@linaro.org, kishon@ti.com,
p.paneri@samsung.com, yulgon.kim@samsung.com
Subject: Re: [PATCH 1/4 v2] usb: phy: samsung: Add host phy support to samsung-phy driver
Date: Mon, 22 Oct 2012 10:56:29 +0530 [thread overview]
Message-ID: <CAFp+6iGD5tWa6NYk_DDWpPeuxttb_zSXBswf6OKF2W+S4fwZGA@mail.gmail.com> (raw)
In-Reply-To: <00c701cdb011$b63bd700$22b38500$%han@samsung.com>
HI,
On Mon, Oct 22, 2012 at 10:27 AM, Jingoo Han <jg1.han@samsung.com> wrote:
> On Wednesday, October 10, 2012 7:43 PM Vivek Gautam wrote
>>
>> This patch adds host phy support to samsung-usbphy.c and
>> further adds support for samsung's exynos5250 usb-phy.
>>
>> Signed-off-by: Vivek Gautam <gautam.vivek@samsung.com>
>> ---
>> arch/arm/mach-exynos/setup-usb-phy.c | 2 +-
>> arch/arm/mach-s3c64xx/setup-usb-phy.c | 2 +-
>> arch/arm/plat-samsung/include/plat/usb-phy.h | 2 +-
>> drivers/usb/phy/Kconfig | 1 -
>> drivers/usb/phy/samsung-usbphy.c | 368 ++++++++++++++++++++++++--
>> include/linux/platform_data/samsung-usbphy.h | 8 +-
>> 6 files changed, 352 insertions(+), 31 deletions(-)
>>
>> diff --git a/arch/arm/mach-exynos/setup-usb-phy.c b/arch/arm/mach-exynos/setup-usb-phy.c
>> index 1c62d20..be6cd4f 100644
>> --- a/arch/arm/mach-exynos/setup-usb-phy.c
>> +++ b/arch/arm/mach-exynos/setup-usb-phy.c
>> @@ -222,7 +222,7 @@ int s5p_usb_phy_exit(struct platform_device *pdev, int type)
>> return -EINVAL;
>> }
>>
>> -void s5p_usb_phy_pmu_isolation(int on)
>> +void s5p_usb_phy_pmu_isolation(int on, int type)
>> {
>> if (on) {
>> writel(readl(S5P_USBDEVICE_PHY_CONTROL)
>> diff --git a/arch/arm/mach-s3c64xx/setup-usb-phy.c b/arch/arm/mach-s3c64xx/setup-usb-phy.c
>> index 3aee778..b7d1d95 100644
>> --- a/arch/arm/mach-s3c64xx/setup-usb-phy.c
>> +++ b/arch/arm/mach-s3c64xx/setup-usb-phy.c
>> @@ -13,7 +13,7 @@
>> #include <mach/map.h>
>> #include <mach/regs-sys.h>
>>
>> -void s5p_usb_phy_pmu_isolation(int on)
>> +void s5p_usb_phy_pmu_isolation(int on, int type)
>> {
>> if (on) {
>> writel(readl(S3C64XX_OTHERS) & ~S3C64XX_OTHERS_USBMASK,
>> diff --git a/arch/arm/plat-samsung/include/plat/usb-phy.h b/arch/arm/plat-samsung/include/plat/usb-phy.h
>> index 165ffe7..7a4a959 100644
>> --- a/arch/arm/plat-samsung/include/plat/usb-phy.h
>> +++ b/arch/arm/plat-samsung/include/plat/usb-phy.h
>> @@ -18,6 +18,6 @@ enum s5p_usb_phy_type {
>>
>> extern int s5p_usb_phy_init(struct platform_device *pdev, int type);
>> extern int s5p_usb_phy_exit(struct platform_device *pdev, int type);
>> -extern void s5p_usb_phy_pmu_isolation(int on);
>> +extern void s5p_usb_phy_pmu_isolation(int on, int type);
>>
>> #endif /* __PLAT_SAMSUNG_USB_PHY_H */
>> diff --git a/drivers/usb/phy/Kconfig b/drivers/usb/phy/Kconfig
>> index 313685f..1ce5b32 100644
>> --- a/drivers/usb/phy/Kconfig
>> +++ b/drivers/usb/phy/Kconfig
>> @@ -35,7 +35,6 @@ config MV_U3D_PHY
>>
>> config SAMSUNG_USBPHY
>> bool "Samsung USB PHY controller Driver"
>> - depends on USB_S3C_HSOTG
>> select USB_OTG_UTILS
>> help
>> Enable this to support Samsung USB phy controller for samsung
>> diff --git a/drivers/usb/phy/samsung-usbphy.c b/drivers/usb/phy/samsung-usbphy.c
>> index ee2dee0..bd6a5e8 100644
>> --- a/drivers/usb/phy/samsung-usbphy.c
>> +++ b/drivers/usb/phy/samsung-usbphy.c
>> @@ -61,9 +61,120 @@
>> #define MHZ (1000*1000)
>> #endif
>
>
> How about using exact name for the definition, as possible?
> PHY SFRs are defined in Exynos5250 and Exynos5410:
> HOSTPHYCTRL0 0x0000
> HOSTPHYTUNE0 0x0004
> HSICPHYCTRL1 0x0010
> HSICPHYTUNE1 0x0014
> HSICPHYCTRL2 0x0020
> HSICPHYTUNE2 0x0024
> HOSTEHCICTRL 0x0030
> HOSTOHCICTRL 0x0034
> USBOTG_SYS 0x0038
> USBOTG_TUNE 0x0040
>
Yes, using above suggested names will be good.
> It would be better to use 'EXYNOS5_PHY_' as the common prefix.
>
shall modify these SFRs' names accordingly and resubmit. Thanks for
pointing out.
>>
>> +/* EXYNOS5 */
>> +#define EXYNOS5_PHYHOST (0x00)
>
> EXYNOS5_PHY_HOST_CTRL0???
>
shall change this as suggested.
>
>> +
>> +#define EXYNOS5_PHYHOST_PHYSWRSTALL (0x1 << 31)
>> +
>> +#define EXYNOS5_PHYHOST_REFCLKSEL_MASK (0x3)
>> +#define EXYNOS5_PHYHOST_REFCLKSEL(_x) ((_x) << 19)
>> +#define EXYNOS5_PHYHOST_REFCLKSEL_XTAL \
>> + EXYNOS5_PHYHOST_REFCLKSEL(0x0)
>> +#define EXYNOS5_PHYHOST_REFCLKSEL_EXTL \
>> + EXYNOS5_PHYHOST_REFCLKSEL(0x1)
>> +#define EXYNOS5_PHYHOST_REFCLKSEL_CLKCORE \
>> + EXYNOS5_PHYHOST_REFCLKSEL(0x2)
>> +
>> +#define EXYNOS5_PHYHOST_FSEL_MASK (0x7 << 16)
>> +#define EXYNOS5_PHYHOST_FSEL(_x) ((_x) << 16)
>> +#define EXYNOS5_PHYHOST_FSEL_CLKSEL_50M (0x7)
>> +#define EXYNOS5_PHYHOST_FSEL_CLKSEL_24M (0x5)
>> +#define EXYNOS5_PHYHOST_FSEL_CLKSEL_20M (0x4)
>> +#define EXYNOS5_PHYHOST_FSEL_CLKSEL_19200K (0x3)
>> +#define EXYNOS5_PHYHOST_FSEL_CLKSEL_12M (0x2)
>> +#define EXYNOS5_PHYHOST_FSEL_CLKSEL_10M (0x1)
>> +#define EXYNOS5_PHYHOST_FSEL_CLKSEL_9600K (0x0)
>> +
>> +#define EXYNOS5_PHYHOST_TESTBURNIN (0x1 << 11)
>> +#define EXYNOS5_PHYHOST_RETENABLE (0x1 << 10)
>> +#define EXYNOS5_PHYHOST_COMMONON_N (0x1 << 9)
>> +#define EXYNOS5_PHYHOST_SIDDQ (0x1 << 6)
>> +#define EXYNOS5_PHYHOST_FORCESLEEP (0x1 << 5)
>> +#define EXYNOS5_PHYHOST_FORCESUSPEND (0x1 << 4)
>> +#define EXYNOS5_PHYHOST_WORDINTERFACE (0x1 << 3)
>> +#define EXYNOS5_PHYHOST_UTMISWRST (0x1 << 2)
>> +#define EXYNOS5_PHYHOST_LINKSWRST (0x1 << 1)
>> +#define EXYNOS5_PHYHOST_PHYSWRST (0x1 << 0)
>> +
>> +#define EXYNOS5_PHYHOST_TUNE0 (0x04)
>
> EXYNOS5_PHY_HOST_TUNE0?
>
ditto.
>> +
>> +#define EXYNOS5_PHYHSIC1 (0x10)
>
> EXYNOS5_PHY_HSIC_CTRL1?
>
ditto.
>> +
>> +#define EXYNOS5_PHYHSIC_TUNE1 (0x14)
>
> EXYNOS5_PHY_HSIC_TUNE1?
>
ditto.
>> +
>> +#define EXYNOS5_PHYHSIC2 (0x20)
>
> EXYNOS5_PHY_HSIC_CTRL2 ?
>
ditto.
>> +
>> +#define EXYNOS5_PHYHSIC_TUNE2 (0x24)
>
> EXYNOS5_PHY_HSIC_TUNE2 ?
>
ditto.
>> +
>> +#define EXYNOS5_PHYHSIC_REFCLKSEL_MASK (0x3)
>> +#define EXYNOS5_PHYHSIC_REFCLKSEL (0x2 << 23)
>> +
>> +#define EXYNOS5_PHYHSIC_REFCLKDIV_MASK (0x7f)
>> +#define EXYNOS5_PHYHSIC_REFCLKDIV(_x) ((_x) << 16)
>> +#define EXYNOS5_PHYHSIC_REFCLKDIV_12 \
>> + EXYNOS5_PHYHSIC_REFCLKDIV(0x24)
>> +#define EXYNOS5_PHYHSIC_REFCLKDIV_15 \
>> + EXYNOS5_PHYHSIC_REFCLKDIV(0x1C)
>> +#define EXYNOS5_PHYHSIC_REFCLKDIV_16 \
>> + EXYNOS5_PHYHSIC_REFCLKDIV(0x1A)
>> +#define EXYNOS5_PHYHSIC_REFCLKDIV_19_2 \
>> + EXYNOS5_PHYHSIC_REFCLKDIV(0x15)
>> +#define EXYNOS5_PHYHSIC_REFCLKDIV_20 \
>> + EXYNOS5_PHYHSIC_REFCLKDIV(0x14)
>> +
>> +#define EXYNOS5_PHYHSIC_SIDDQ (0x1 << 6)
>> +#define EXYNOS5_PHYHSIC_FORCESLEEP (0x1 << 5)
>> +#define EXYNOS5_PHYHSIC_FORCESUSPEND (0x1 << 4)
>> +#define EXYNOS5_PHYHSIC_WORDINTERFACE (0x1 << 3)
>> +#define EXYNOS5_PHYHSIC_UTMISWRST (0x1 << 2)
>> +#define EXYNOS5_PHYHSIC_PHYSWRST (0x1 << 0)
>> +
>> +#define EXYNOS5_EHCICTRL (0x30)
>
> EXYNOS5_PHY_HOST_EHCICTRL ?
>
ditto.
>> +
>> +#define EXYNOS5_EHCICTRL_ENAINCRXALIGN (0x1 << 29)
>> +#define EXYNOS5_EHCICTRL_ENAINCR4 (0x1 << 28)
>> +#define EXYNOS5_EHCICTRL_ENAINCR8 (0x1 << 27)
>> +#define EXYNOS5_EHCICTRL_ENAINCR16 (0x1 << 26)
>> +
>> +#define EXYNOS5_OHCICTRL (0x34)
>
> EXYNOS5_PHY_HOST_OHCICTRL ?
>
ditto.
>> +
>> +#define EXYNOS5_OHCICTRL_SUSPLGCY (0x1 << 3)
>> +#define EXYNOS5_OHCICTRL_APPSTARTCLK (0x1 << 2)
>> +#define EXYNOS5_OHCICTRL_CNTSEL (0x1 << 1)
>> +#define EXYNOS5_OHCICTRL_CLKCKTRST (0x1 << 0)
>> +
>> +#define EXYNOS5_PHYOTG (0x38)
>
> EXYNOS5_PHY_OTG_TUNE ?
>
ditto.
>> +
>> +#define EXYNOS5_PHYOTG_PHYLINK_SWRESET (0x1 << 14)
>> +#define EXYNOS5_PHYOTG_LINKSWRST_UOTG (0x1 << 13)
>> +#define EXYNOS5_PHYOTG_PHY0_SWRST (0x1 << 12)
>> +
>> +#define EXYNOS5_PHYOTG_REFCLKSEL_MASK (0x3 << 9)
>> +#define EXYNOS5_PHYOTG_REFCLKSEL(_x) ((_x) << 9)
>> +#define EXYNOS5_PHYOTG_REFCLKSEL_XTAL \
>> + EXYNOS5_PHYOTG_REFCLKSEL(0x0)
>> +#define EXYNOS5_PHYOTG_REFCLKSEL_EXTL \
>> + EXYNOS5_PHYOTG_REFCLKSEL(0x1)
>> +#define EXYNOS5_PHYOTG_REFCLKSEL_CLKCORE \
>> + EXYNOS5_PHYOTG_REFCLKSEL(0x2)
>> +
>> +#define EXYNOS5_PHYOTG_IDPULLUP_UOTG (0x1 << 8)
>> +#define EXYNOS5_PHYOTG_COMMON_ON (0x1 << 7)
>> +
>> +#define EXYNOS5_PHYOTG_FSEL_MASK (0x7 << 4)
>> +#define EXYNOS5_PHYOTG_FSEL(_x) ((_x) << 4)
>> +
>> +#define EXYNOS5_PHYOTG_FORCESLEEP (0x1 << 3)
>> +#define EXYNOS5_PHYOTG_OTGDISABLE (0x1 << 2)
>> +#define EXYNOS5_PHYOTG_SIDDQ_UOTG (0x1 << 1)
>> +#define EXYNOS5_PHYOTG_FORCESUSPEND (0x1 << 0)
>> +
>> +#define EXYNOS5_PHYOTG_TUNE (0x40)
>
> EXYNOS5_PHY_OTG_TUNE ?
>
ditto.
>> +
>> enum samsung_cpu_type {
>> TYPE_S3C64XX,
>> TYPE_EXYNOS4210,
>> + TYPE_EXYNOS5250,
>> };
>>
>> /*
>> @@ -84,6 +195,7 @@ struct samsung_usbphy {
>> void __iomem *regs;
>> int ref_clk_freq;
>> int cpu_type;
>> + atomic_t host_usage;
>> };
>>
>> #define phy_to_sphy(x) container_of((x), struct samsung_usbphy, phy)
>> @@ -96,30 +208,175 @@ static int samsung_usbphy_get_refclk_freq(struct samsung_usbphy *sphy)
>> struct clk *ref_clk;
>> int refclk_freq = 0;
>>
>> - ref_clk = clk_get(sphy->dev, "xusbxti");
>> + if (sphy->cpu_type == TYPE_EXYNOS5250)
>> + ref_clk = clk_get(sphy->dev, "ext_xtal");
>> + else
>> + ref_clk = clk_get(sphy->dev, "xusbxti");
>> if (IS_ERR(ref_clk)) {
>> dev_err(sphy->dev, "Failed to get reference clock\n");
>> return PTR_ERR(ref_clk);
>> }
>>
>> - switch (clk_get_rate(ref_clk)) {
>> - case 12 * MHZ:
>> - refclk_freq |= S3C_PHYCLK_CLKSEL_12M;
>> - break;
>> - case 24 * MHZ:
>> - refclk_freq |= S3C_PHYCLK_CLKSEL_24M;
>> - break;
>> - default:
>> - case 48 * MHZ:
>> - /* default reference clock */
>> - refclk_freq |= S3C_PHYCLK_CLKSEL_48M;
>> - break;
>> + if (sphy->cpu_type == TYPE_EXYNOS5250) {
>> + /* set clock frequency for PLL */
>> + switch (clk_get_rate(ref_clk)) {
>> + case 96 * 100000:
>> + refclk_freq |= EXYNOS5_PHYHOST_FSEL_CLKSEL_9600K;
>> + break;
>> + case 10 * MHZ:
>> + refclk_freq |= EXYNOS5_PHYHOST_FSEL_CLKSEL_10M;
>> + break;
>> + case 12 * MHZ:
>> + refclk_freq |= EXYNOS5_PHYHOST_FSEL_CLKSEL_12M;
>> + break;
>> + case 192 * 100000:
>> + refclk_freq |= EXYNOS5_PHYHOST_FSEL_CLKSEL_19200K;
>> + break;
>> + case 20 * MHZ:
>> + refclk_freq |= EXYNOS5_PHYHOST_FSEL_CLKSEL_20M;
>> + break;
>> + case 50 * MHZ:
>> + refclk_freq |= EXYNOS5_PHYHOST_FSEL_CLKSEL_50M;
>> + break;
>> + case 24 * MHZ:
>> + default:
>> + /* default reference clock */
>> + refclk_freq |= EXYNOS5_PHYHOST_FSEL_CLKSEL_24M;
>> + break;
>> + }
>> + } else {
>> + switch (clk_get_rate(ref_clk)) {
>> + case 12 * MHZ:
>> + refclk_freq |= S3C_PHYCLK_CLKSEL_12M;
>> + break;
>> + case 24 * MHZ:
>> + refclk_freq |= S3C_PHYCLK_CLKSEL_24M;
>> + break;
>> + default:
>> + case 48 * MHZ:
>> + /* default reference clock */
>> + refclk_freq |= S3C_PHYCLK_CLKSEL_48M;
>> + break;
>> + }
>> }
>> clk_put(ref_clk);
>>
>> return refclk_freq;
>> }
>>
>> +static int exynos5_phyhost_is_on(void *regs)
>> +{
>> + return (readl(regs + EXYNOS5_PHYHOST) & EXYNOS5_PHYHOST_SIDDQ) ? 0 : 1;
>> +}
>> +
>> +static void samsung_exynos5_usbphy_enable(struct samsung_usbphy *sphy)
>> +{
>> + void __iomem *regs = sphy->regs;
>> + u32 phyclk = sphy->ref_clk_freq;
>> + u32 phyhost;
>> + u32 phyotg;
>> + u32 phyhsic;
>> + u32 ehcictrl;
>> + u32 ohcictrl;
>> +
>> + atomic_inc(&sphy->host_usage);
>> +
>> + if (exynos5_phyhost_is_on(regs)) {
>> + dev_info(sphy->dev, "Already power on PHY\n");
>> + return;
>> + }
>> +
>> + /* Selecting Host/OTG mode; After reset USB2.0PHY_CFG: HOST */
>> + if (sphy->plat && sphy->plat->phy_cfg_sel) {
>> + if (sphy->plat->type)
>> + sphy->plat->phy_cfg_sel(sphy->dev, sphy->plat->type);
>> + else
>> + sphy->plat->phy_cfg_sel(sphy->dev, TYPE_OTG);
>> + }
>> +
>> + /* Host configuration */
>> + phyhost = readl(regs + EXYNOS5_PHYHOST);
>> +
>> + /* phy reference clock configuration */
>> + phyhost &= ~(EXYNOS5_PHYHOST_FSEL_MASK);
>> + phyhost |= EXYNOS5_PHYHOST_FSEL(phyclk);
>> +
>> + /* host phy reset */
>> + phyhost &= ~(EXYNOS5_PHYHOST_PHYSWRST |
>> + EXYNOS5_PHYHOST_PHYSWRSTALL |
>> + EXYNOS5_PHYHOST_SIDDQ |
>> + /* Enable normal mode of operation */
>> + EXYNOS5_PHYHOST_FORCESUSPEND |
>> + EXYNOS5_PHYHOST_FORCESLEEP);
>> +
>> + /* Link reset */
>> + phyhost |= (EXYNOS5_PHYHOST_LINKSWRST |
>> + EXYNOS5_PHYHOST_UTMISWRST |
>> + /* COMMON Block configuration during suspend */
>> + EXYNOS5_PHYHOST_COMMONON_N);
>> + writel(phyhost, regs + EXYNOS5_PHYHOST);
>> + udelay(10);
>> + phyhost &= ~(EXYNOS5_PHYHOST_LINKSWRST |
>> + EXYNOS5_PHYHOST_UTMISWRST);
>> + writel(phyhost, regs + EXYNOS5_PHYHOST);
>> +
>> + /* OTG configuration */
>> + phyotg = readl(regs + EXYNOS5_PHYOTG);
>> +
>> + /* phy reference clock configuration */
>> + phyotg &= ~(EXYNOS5_PHYOTG_FSEL_MASK);
>> + phyotg |= EXYNOS5_PHYOTG_FSEL(phyclk);
>> +
>> + /* Enable normal mode of operation */
>> + phyotg &= ~(EXYNOS5_PHYOTG_FORCESUSPEND |
>> + EXYNOS5_PHYOTG_SIDDQ_UOTG |
>> + EXYNOS5_PHYOTG_FORCESLEEP |
>> + EXYNOS5_PHYOTG_REFCLKSEL_MASK |
>> + /* COMMON Block configuration during suspend */
>> + EXYNOS5_PHYOTG_COMMON_ON);
>> +
>> + /* OTG phy & link reset */
>> + phyotg |= (EXYNOS5_PHYOTG_PHY0_SWRST |
>> + EXYNOS5_PHYOTG_LINKSWRST_UOTG |
>> + EXYNOS5_PHYOTG_PHYLINK_SWRESET |
>> + EXYNOS5_PHYOTG_OTGDISABLE |
>> + /* Set refclk */
>> + EXYNOS5_PHYOTG_REFCLKSEL_CLKCORE);
>> +
>> + writel(phyotg, regs + EXYNOS5_PHYOTG);
>> + udelay(10);
>> + phyotg &= ~(EXYNOS5_PHYOTG_PHY0_SWRST |
>> + EXYNOS5_PHYOTG_LINKSWRST_UOTG |
>> + EXYNOS5_PHYOTG_PHYLINK_SWRESET);
>> + writel(phyotg, regs + EXYNOS5_PHYOTG);
>> +
>> + /* HSIC phy configuration */
>> + phyhsic = (EXYNOS5_PHYHSIC_REFCLKDIV_12 |
>> + EXYNOS5_PHYHSIC_REFCLKSEL |
>> + EXYNOS5_PHYHSIC_PHYSWRST);
>> + writel(phyhsic, regs + EXYNOS5_PHYHSIC1);
>> + writel(phyhsic, regs + EXYNOS5_PHYHSIC2);
>> + udelay(10);
>> + phyhsic &= ~(EXYNOS5_PHYHSIC_PHYSWRST);
>> + writel(phyhsic, regs + EXYNOS5_PHYHSIC1);
>> + writel(phyhsic, regs + EXYNOS5_PHYHSIC2);
>> +
>> + udelay(80);
>> +
>> + /* enable EHCI DMA burst */
>> + ehcictrl = readl(regs + EXYNOS5_EHCICTRL);
>> + ehcictrl |= (EXYNOS5_EHCICTRL_ENAINCRXALIGN |
>> + EXYNOS5_EHCICTRL_ENAINCR4 |
>> + EXYNOS5_EHCICTRL_ENAINCR8 |
>> + EXYNOS5_EHCICTRL_ENAINCR16);
>> + writel(ehcictrl, regs + EXYNOS5_EHCICTRL);
>> +
>> + /* set ohci_suspend_on_n */
>> + ohcictrl = readl(regs + EXYNOS5_OHCICTRL);
>> + ohcictrl |= EXYNOS5_OHCICTRL_SUSPLGCY;
>> + writel(ohcictrl, regs + EXYNOS5_OHCICTRL);
>> +}
>> +
>> static void samsung_usbphy_enable(struct samsung_usbphy *sphy)
>> {
>> void __iomem *regs = sphy->regs;
>> @@ -155,6 +412,41 @@ static void samsung_usbphy_enable(struct samsung_usbphy *sphy)
>> writel(rstcon, regs + S3C_RSTCON);
>> }
>>
>> +static void samsung_exynos5_usbphy_disable(struct samsung_usbphy *sphy)
>> +{
>> + void __iomem *regs = sphy->regs;
>> + u32 phyhost;
>> + u32 phyotg;
>> + u32 phyhsic;
>> +
>> + if (atomic_dec_return(&sphy->host_usage) > 0) {
>> + dev_info(sphy->dev, "still being used\n");
>> + return;
>> + }
>> +
>> + phyhsic = (EXYNOS5_PHYHSIC_REFCLKDIV_12 |
>> + EXYNOS5_PHYHSIC_REFCLKSEL |
>> + EXYNOS5_PHYHSIC_SIDDQ |
>> + EXYNOS5_PHYHSIC_FORCESLEEP |
>> + EXYNOS5_PHYHSIC_FORCESUSPEND);
>> + writel(phyhsic, regs + EXYNOS5_PHYHSIC1);
>> + writel(phyhsic, regs + EXYNOS5_PHYHSIC2);
>> +
>> + phyhost = readl(regs + EXYNOS5_PHYHOST);
>> + phyhost |= (EXYNOS5_PHYHOST_SIDDQ |
>> + EXYNOS5_PHYHOST_FORCESUSPEND |
>> + EXYNOS5_PHYHOST_FORCESLEEP |
>> + EXYNOS5_PHYHOST_PHYSWRST |
>> + EXYNOS5_PHYHOST_PHYSWRSTALL);
>> + writel(phyhost, regs + EXYNOS5_PHYHOST);
>> +
>> + phyotg = readl(regs + EXYNOS5_PHYOTG);
>> + phyotg |= (EXYNOS5_PHYOTG_FORCESUSPEND |
>> + EXYNOS5_PHYOTG_SIDDQ_UOTG |
>> + EXYNOS5_PHYOTG_FORCESLEEP);
>> + writel(phyotg, regs + EXYNOS5_PHYOTG);
>> +}
>> +
>> static void samsung_usbphy_disable(struct samsung_usbphy *sphy)
>> {
>> void __iomem *regs = sphy->regs;
>> @@ -194,11 +486,18 @@ static int samsung_usbphy_init(struct usb_phy *phy)
>> }
>>
>> /* Disable phy isolation */
>> - if (sphy->plat && sphy->plat->pmu_isolation)
>> - sphy->plat->pmu_isolation(false);
>> + if (sphy->plat && sphy->plat->pmu_isolation) {
>> + if (sphy->plat->type)
>> + sphy->plat->pmu_isolation(false, sphy->plat->type);
>> + else
>> + sphy->plat->pmu_isolation(false, TYPE_OTG);
>> + }
>>
>> /* Initialize usb phy registers */
>> - samsung_usbphy_enable(sphy);
>> + if (sphy->cpu_type == TYPE_EXYNOS5250)
>> + samsung_exynos5_usbphy_enable(sphy);
>> + else
>> + samsung_usbphy_enable(sphy);
>>
>> /* Disable the phy clock */
>> clk_disable_unprepare(sphy->clk);
>> @@ -220,11 +519,18 @@ static void samsung_usbphy_shutdown(struct usb_phy *phy)
>> }
>>
>> /* De-initialize usb phy registers */
>> - samsung_usbphy_disableditto.(sphy);
>> + if (sphy->cpu_type == TYPE_EXYNOS5250)
>> + samsung_exynos5_usbphy_disable(sphy);
>> + else
>> + samsung_usbphy_disable(sphy);
>>
>> /* Enable phy isolation */
>> - if (sphy->plat && sphy->plat->pmu_isolation)
>> - sphy->plat->pmu_isolation(true);
>> + if (sphy->plat && sphy->plat->pmu_isolation) {
>> + if (sphy->plat->type)
>> + sphy->plat->pmu_isolation(true, sphy->plat->type);
>> + else
>> + sphy->plat->pmu_isolation(true, TYPE_OTG);
>> + }
>>
>> clk_disable_unprepare(sphy->clk);
>> }
>> @@ -277,16 +583,9 @@ static int __devinit samsung_usbphy_probe(struct platform_device *pdev)
>> if (!sphy)
>> return -ENOMEM;
>>
>> - clk = devm_clk_get(dev, "otg");
>> - if (IS_ERR(clk)) {
>> - dev_err(dev, "Failed to get otg clock\n");
>> - return PTR_ERR(clk);
>> - }
>> -
>> sphy->dev = &pdev->dev;
>> sphy->plat = pdata;
>> sphy->regs = phy_base;
>> - sphy->clk = clk;
>> sphy->phy.dev = sphy->dev;
>> sphy->phy.label = "samsung-usbphy";
>> sphy->phy.init = samsung_usbphy_init;
>> @@ -294,6 +593,17 @@ static int __devinit samsung_usbphy_probe(struct platform_device *pdev)
>> sphy->cpu_type = samsung_usbphy_get_driver_data(pdev);
>> sphy->ref_clk_freq = samsung_usbphy_get_refclk_freq(sphy);
>>
>> + if (sphy->cpu_type == TYPE_EXYNOS5250)
>> + clk = devm_clk_get(dev, "usbhost");
>> + else
>> + clk = devm_clk_get(dev, "otg");
>> + if (IS_ERR(clk)) {
>> + dev_err(dev, "Failed to get otg clock\n");
>> + return PTR_ERR(clk);
>> + }
>> +
>> + sphy->clk = clk;
>> +
>> ret = usb_add_phy(&sphy->phy, USB_PHY_TYPE_USB2);
>> return ret;
>> }
>> @@ -315,6 +625,9 @@ static const struct of_device_id samsung_usbphy_dt_match[] = {
>> }, {
>> .compatible = "samsung,exynos4210-usbphy",
>> .data = (void *)TYPE_EXYNOS4210,
>> + }, {
>> + .compatible = "samsung,exynos5250-usbphy",
>> + .data = (void *)TYPE_EXYNOS5250,
>> },
>> {},
>> };
>> @@ -330,6 +643,9 @@ static struct platform_device_id samsung_usbphy_driver_ids[] = {
>> }, {
>> .name = "exynos4210-usbphy",
>> .driver_data = TYPE_EXYNOS4210,
>> + }, {
>> + .name = "exynos5250-usbphy",
>> + .driver_data = TYPE_EXYNOS5250,
>> },
>> {},
>> };
>> diff --git a/include/linux/platform_data/samsung-usbphy.h b/include/linux/platform_data/samsung-usbphy.h
>> index 1bd24cb..364a38a 100644
>> --- a/include/linux/platform_data/samsung-usbphy.h
>> +++ b/include/linux/platform_data/samsung-usbphy.h
>> @@ -18,8 +18,14 @@
>> * samsung_usbphy_data - Platform data for USB PHY driver.
>> * @pmu_isolation: Function to control usb phy isolation in PMU.
>> */
>> +enum phy_type {
>> + TYPE_OTG,
>> + TYPE_HOST,
>> +};
>> +
>> struct samsung_usbphy_data {
>> - void (*pmu_isolation)(int on);
>> + void (*pmu_isolation)(int on, int phy_type);
>> + int type;
>> };
>>
>> extern void samsung_usbphy_set_pdata(struct samsung_usbphy_data *pd);
>> --
>> 1.7.6.5
>
> --
> 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
next prev parent reply other threads:[~2012-10-22 5:26 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-10 10:42 [PATCH 0/4 v2] Adding usb2.0 host-phy support for exynos5250 Vivek Gautam
2012-10-10 10:42 ` [PATCH 1/4 v2] usb: phy: samsung: Add host phy support to samsung-phy driver Vivek Gautam
[not found] ` <1349865780-7701-2-git-send-email-gautam.vivek-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2012-10-22 4:57 ` Jingoo Han
2012-10-22 5:26 ` Vivek Gautam [this message]
2012-10-10 10:42 ` [PATCH 2/4 v2] ARM: Exynos5250: Enabling samsung-usbphy driver Vivek Gautam
2012-10-22 5:03 ` Jingoo Han
2012-10-22 5:18 ` Vivek Gautam
[not found] ` <1349865780-7701-1-git-send-email-gautam.vivek-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2012-10-10 10:42 ` [PATCH 3/4 v2] USB: ehci-s5p: Add phy driver support Vivek Gautam
[not found] ` <1349865780-7701-4-git-send-email-gautam.vivek-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2012-10-15 5:52 ` Jingoo Han
2012-10-10 10:43 ` [PATCH 4/4 v2] USB: ohci-exynos: " Vivek Gautam
2012-10-15 5:52 ` Jingoo Han
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAFp+6iGD5tWa6NYk_DDWpPeuxttb_zSXBswf6OKF2W+S4fwZGA@mail.gmail.com \
--to=gautamvivek1987@gmail.com \
--cc=balbi@ti.com \
--cc=devicetree-discuss@lists.ozlabs.org \
--cc=gautam.vivek@samsung.com \
--cc=gregkh@linuxfoundation.org \
--cc=jg1.han@samsung.com \
--cc=kgene.kim@samsung.com \
--cc=kishon@ti.com \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=p.paneri@samsung.com \
--cc=rob.herring@calxeda.com \
--cc=stern@rowland.harvard.edu \
--cc=thomas.abraham@linaro.org \
--cc=yulgon.kim@samsung.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).