RE: [PATCH v2 2/3] usb: musb: dsps: add phy control logic to glue
Hi, > On Wed, Jul 11, 2012 at 3:59 PM, Damodar Santhapuri > wrote: > > From: Ajay Kumar Gupta > > > > AM335x uses NOP transceiver driver and need to enable builtin PHY > > by writing into usb_ctrl register available in system control > > module register space. This is being added at musb glue driver > > layer untill a separate system control module driver is available. > > > > Signed-off-by: Ajay Kumar Gupta > > Signed-off-by: Damodar Santhapuri > > --- > > Changes from v0: > > - Used platform_get_resource() instead of > platform_get_resource_byname() > > based on Kishon's comment. > > > > arch/arm/mach-omap2/board-ti8168evm.c |1 - > > arch/arm/mach-omap2/omap_phy_internal.c | 35 > > arch/arm/plat-omap/include/plat/usb.h |5 +- > > drivers/usb/musb/musb_dsps.c| 87 > +-- > > 4 files changed, 73 insertions(+), 55 deletions(-) > > > > diff --git a/arch/arm/mach-omap2/board-ti8168evm.c b/arch/arm/mach- > omap2/board-ti8168evm.c > > index d4c8392..0c7c098 100644 > > --- a/arch/arm/mach-omap2/board-ti8168evm.c > > +++ b/arch/arm/mach-omap2/board-ti8168evm.c > > @@ -26,7 +26,6 @@ > > #include > > > > static struct omap_musb_board_data musb_board_data = { > > - .set_phy_power = ti81xx_musb_phy_power, > > .interface_type = MUSB_INTERFACE_ULPI, > > .mode = MUSB_OTG, > > .power = 500, > > diff --git a/arch/arm/mach-omap2/omap_phy_internal.c b/arch/arm/mach- > omap2/omap_phy_internal.c > > index d52651a..d80bb16 100644 > > --- a/arch/arm/mach-omap2/omap_phy_internal.c > > +++ b/arch/arm/mach-omap2/omap_phy_internal.c > > @@ -254,38 +254,3 @@ void am35x_set_mode(u8 musb_mode) > > > > omap_ctrl_writel(devconf2, AM35XX_CONTROL_DEVCONF2); > > } > > - > > -void ti81xx_musb_phy_power(u8 on) > > -{ > > - void __iomem *scm_base = NULL; > > - u32 usbphycfg; > > - > > - scm_base = ioremap(TI81XX_SCM_BASE, SZ_2K); > > - if (!scm_base) { > > - pr_err("system control module ioremap failed\n"); > > - return; > > - } > > - > > - usbphycfg = __raw_readl(scm_base + USBCTRL0); > > - > > - if (on) { > > - if (cpu_is_ti816x()) { > > - usbphycfg |= TI816X_USBPHY0_NORMAL_MODE; > > - usbphycfg &= ~TI816X_USBPHY_REFCLK_OSC; > > - } else if (cpu_is_ti814x()) { > > - usbphycfg &= ~(USBPHY_CM_PWRDN | > USBPHY_OTG_PWRDN > > - | USBPHY_DPINPUT | USBPHY_DMINPUT); > > - usbphycfg |= (USBPHY_OTGVDET_EN | > USBPHY_OTGSESSEND_EN > > - | USBPHY_DPOPBUFCTL | > USBPHY_DMOPBUFCTL); > > - } > > - } else { > > - if (cpu_is_ti816x()) > > - usbphycfg &= ~TI816X_USBPHY0_NORMAL_MODE; > > - else if (cpu_is_ti814x()) > > - usbphycfg |= USBPHY_CM_PWRDN | > USBPHY_OTG_PWRDN; > > - > > - } > > - __raw_writel(usbphycfg, scm_base + USBCTRL0); > > - > > - iounmap(scm_base); > > -} > > diff --git a/arch/arm/plat-omap/include/plat/usb.h b/arch/arm/plat- > omap/include/plat/usb.h > > index 548a4c8..c2aa4ae 100644 > > --- a/arch/arm/plat-omap/include/plat/usb.h > > +++ b/arch/arm/plat-omap/include/plat/usb.h > > @@ -95,7 +95,6 @@ extern void am35x_musb_reset(void); > > extern void am35x_musb_phy_power(u8 on); > > extern void am35x_musb_clear_irq(void); > > extern void am35x_set_mode(u8 musb_mode); > > -extern void ti81xx_musb_phy_power(u8 on); > > > > /* AM35x */ > > /* USB 2.0 PHY Control */ > > @@ -120,8 +119,8 @@ extern void ti81xx_musb_phy_power(u8 on); > > #define CONF2_DATPOL (1 << 1) > > > > /* TI81XX specific definitions */ > > -#define USBCTRL0 0x620 > > -#define USBSTAT0 0x624 > > +#define MUSB_USBSS_REV_816X0x9 > > +#define MUSB_USBSS_REV_814X0xb > > > > /* TI816X PHY controls bits */ > > #define TI816X_USBPHY0_NORMAL_MODE (1 << 0) > > diff --git a/drivers/usb/musb/musb_dsps.c > b/drivers/usb/musb/musb_dsps.c > > index 494772f..72eda64 100644 > > --- a/drivers/usb/musb/musb_dsps.c > > +++ b/drivers/usb/musb/musb_dsps.c > > @@ -115,9 +115,46 @@ struct dsps_glue { > > struct platform_device *musb; /* child musb pdev */ > > const struct dsps_musb_wrapper *wrp; /* wrapper register > offsets */ > > struct timer_list timer;/* otg_workaround timer */ > > + u32 __iomem *usb_ctrl; > > + u8 usbss_rev; > > }; > > > > /** > > + * musb_dsps_phy_control - phy on/off > > + * @glue: struct dsps_glue * > > + * @on: flag for phy to be switched on or off > > + * > > + * This is to enable the PHY using usb_ctrl register in system > control > > + * module space. > > + * > > + * XXX: This function will be removed once we have a seperate driver > for > > + * control module > > + */ > > +static voi
Re: [PATCH v2 2/3] usb: musb: dsps: add phy control logic to glue
Hi, On Wed, Jul 11, 2012 at 3:59 PM, Damodar Santhapuri wrote: > From: Ajay Kumar Gupta > > AM335x uses NOP transceiver driver and need to enable builtin PHY > by writing into usb_ctrl register available in system control > module register space. This is being added at musb glue driver > layer untill a separate system control module driver is available. > > Signed-off-by: Ajay Kumar Gupta > Signed-off-by: Damodar Santhapuri > --- > Changes from v0: > - Used platform_get_resource() instead of > platform_get_resource_byname() > based on Kishon's comment. > > arch/arm/mach-omap2/board-ti8168evm.c |1 - > arch/arm/mach-omap2/omap_phy_internal.c | 35 > arch/arm/plat-omap/include/plat/usb.h |5 +- > drivers/usb/musb/musb_dsps.c| 87 > +-- > 4 files changed, 73 insertions(+), 55 deletions(-) > > diff --git a/arch/arm/mach-omap2/board-ti8168evm.c > b/arch/arm/mach-omap2/board-ti8168evm.c > index d4c8392..0c7c098 100644 > --- a/arch/arm/mach-omap2/board-ti8168evm.c > +++ b/arch/arm/mach-omap2/board-ti8168evm.c > @@ -26,7 +26,6 @@ > #include > > static struct omap_musb_board_data musb_board_data = { > - .set_phy_power = ti81xx_musb_phy_power, > .interface_type = MUSB_INTERFACE_ULPI, > .mode = MUSB_OTG, > .power = 500, > diff --git a/arch/arm/mach-omap2/omap_phy_internal.c > b/arch/arm/mach-omap2/omap_phy_internal.c > index d52651a..d80bb16 100644 > --- a/arch/arm/mach-omap2/omap_phy_internal.c > +++ b/arch/arm/mach-omap2/omap_phy_internal.c > @@ -254,38 +254,3 @@ void am35x_set_mode(u8 musb_mode) > > omap_ctrl_writel(devconf2, AM35XX_CONTROL_DEVCONF2); > } > - > -void ti81xx_musb_phy_power(u8 on) > -{ > - void __iomem *scm_base = NULL; > - u32 usbphycfg; > - > - scm_base = ioremap(TI81XX_SCM_BASE, SZ_2K); > - if (!scm_base) { > - pr_err("system control module ioremap failed\n"); > - return; > - } > - > - usbphycfg = __raw_readl(scm_base + USBCTRL0); > - > - if (on) { > - if (cpu_is_ti816x()) { > - usbphycfg |= TI816X_USBPHY0_NORMAL_MODE; > - usbphycfg &= ~TI816X_USBPHY_REFCLK_OSC; > - } else if (cpu_is_ti814x()) { > - usbphycfg &= ~(USBPHY_CM_PWRDN | USBPHY_OTG_PWRDN > - | USBPHY_DPINPUT | USBPHY_DMINPUT); > - usbphycfg |= (USBPHY_OTGVDET_EN | USBPHY_OTGSESSEND_EN > - | USBPHY_DPOPBUFCTL | USBPHY_DMOPBUFCTL); > - } > - } else { > - if (cpu_is_ti816x()) > - usbphycfg &= ~TI816X_USBPHY0_NORMAL_MODE; > - else if (cpu_is_ti814x()) > - usbphycfg |= USBPHY_CM_PWRDN | USBPHY_OTG_PWRDN; > - > - } > - __raw_writel(usbphycfg, scm_base + USBCTRL0); > - > - iounmap(scm_base); > -} > diff --git a/arch/arm/plat-omap/include/plat/usb.h > b/arch/arm/plat-omap/include/plat/usb.h > index 548a4c8..c2aa4ae 100644 > --- a/arch/arm/plat-omap/include/plat/usb.h > +++ b/arch/arm/plat-omap/include/plat/usb.h > @@ -95,7 +95,6 @@ extern void am35x_musb_reset(void); > extern void am35x_musb_phy_power(u8 on); > extern void am35x_musb_clear_irq(void); > extern void am35x_set_mode(u8 musb_mode); > -extern void ti81xx_musb_phy_power(u8 on); > > /* AM35x */ > /* USB 2.0 PHY Control */ > @@ -120,8 +119,8 @@ extern void ti81xx_musb_phy_power(u8 on); > #define CONF2_DATPOL (1 << 1) > > /* TI81XX specific definitions */ > -#define USBCTRL0 0x620 > -#define USBSTAT0 0x624 > +#define MUSB_USBSS_REV_816X0x9 > +#define MUSB_USBSS_REV_814X0xb > > /* TI816X PHY controls bits */ > #define TI816X_USBPHY0_NORMAL_MODE (1 << 0) > diff --git a/drivers/usb/musb/musb_dsps.c b/drivers/usb/musb/musb_dsps.c > index 494772f..72eda64 100644 > --- a/drivers/usb/musb/musb_dsps.c > +++ b/drivers/usb/musb/musb_dsps.c > @@ -115,9 +115,46 @@ struct dsps_glue { > struct platform_device *musb; /* child musb pdev */ > const struct dsps_musb_wrapper *wrp; /* wrapper register offsets */ > struct timer_list timer;/* otg_workaround timer */ > + u32 __iomem *usb_ctrl; > + u8 usbss_rev; > }; > > /** > + * musb_dsps_phy_control - phy on/off > + * @glue: struct dsps_glue * > + * @on: flag for phy to be switched on or off > + * > + * This is to enable the PHY using usb_ctrl register in system control > + * module space. > + * > + * XXX: This function will be removed once we have a seperate driver for > + * control module > + */ > +static void musb_dsps_phy_control(struct dsps_glue *glue, u8 on) I think this function should be added in your transceiver driver. I don't see glue as an appropriate place for this. > +{ > + u32 usbphycfg; > + > + usbphycfg = __raw_readl(glue->usb_ctrl); > +