Re: [PATCH 1/6 v2] omap: musb: Adding hwmod data for ti81xx

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Sep 8, 2011 at 1:30 PM, Felipe Balbi <balbi@xxxxxx> wrote:
> Hi,
>
> On Thu, Sep 08, 2011 at 11:56:25AM +0530, Munegowda, Keshava wrote:
>> On Wed, Sep 7, 2011 at 10:32 PM, Ajay Kumar Gupta <ajay.gupta@xxxxxx> wrote:
>> > From: Ravi Babu <ravibabu@xxxxxx>
>> >
>> > The usb subsystem (usbss) in ti81xx has two musb interfaces. There are three
>> > irqs and three address spaces for usbss, musb0 and musb1 respectively. Created
>> > one hwmod with three irq and memory resources.
>> >
>> > Cc: Benoit Cousson <b-cousson@xxxxxx>
>> > Cc: Keshava Munegowda <keshava_mgowda@xxxxxx>
>> > Signed-off-by: Ajay Kumar Gupta <ajay.gupta@xxxxxx>
>> > Signed-off-by: Ravi Babu <ravibabu@xxxxxx>
>> > ---
>> >  arch/arm/mach-omap2/omap_hwmod_81xx_data.c |   74 ++++++++++++++++++++++++++++
>> >  1 files changed, 74 insertions(+), 0 deletions(-)
>> >
>> > diff --git a/arch/arm/mach-omap2/omap_hwmod_81xx_data.c b/arch/arm/mach-omap2/omap_hwmod_81xx_data.c
>> > index e73a4c6..e960d70 100644
>> > --- a/arch/arm/mach-omap2/omap_hwmod_81xx_data.c
>> > +++ b/arch/arm/mach-omap2/omap_hwmod_81xx_data.c
>> > @@ -194,6 +194,7 @@ static struct omap_hwmod ti81xx_timer4_hwmod;
>> >  static struct omap_hwmod ti81xx_timer5_hwmod;
>> >  static struct omap_hwmod ti81xx_timer6_hwmod;
>> >  static struct omap_hwmod ti81xx_timer7_hwmod;
>> > +static struct omap_hwmod ti81xx_usbss_hwmod;
>> >
>> >  /* L4 SLOW -> TIMER1 interface */
>> >  static struct omap_hwmod_addr_space ti81xx_timer1_addr_space[] = {
>> > @@ -650,6 +651,78 @@ static struct omap_hwmod ti81xx_timer7_hwmod = {
>> >        .omap_chip      = OMAP_CHIP_INIT(CHIP_IS_TI816X | CHIP_IS_TI814X)
>> >  };
>> >
>> > +/* L3 SLOW -> USBSS interface */
>> > +static struct omap_hwmod_addr_space ti81xx_usbss_addr_space[] = {
>> > +       {
>> > +               .name           = "usbss",
>> > +               .pa_start       = 0x47400000,
>> > +               .pa_end         = 0x47400000 + SZ_4K - 1,
>> > +               .flags          = ADDR_TYPE_RT
>> > +       },
>> > +       {
>> > +               .name           = "musb0",
>> > +               .pa_start       = 0x47401000,
>> > +               .pa_end         = 0x47401000 + SZ_2K - 1,
>> > +               .flags          = ADDR_TYPE_RT
>> > +       },
>> > +       {
>> > +               .name           = "musb1",
>> > +               .pa_start       = 0x47401800,
>> > +               .pa_end         = 0x47401800 + SZ_2K - 1,
>> > +               .flags          = ADDR_TYPE_RT
>> > +       },
>> > +       {
>> > +       },
>> > +};
>> > +
>> > +static struct omap_hwmod_class_sysconfig ti81xx_usbhsotg_sysc = {
>> > +       .rev_offs       = 0x0,
>> > +       .sysc_offs      = 0x10,
>> > +       .sysc_flags     = (SYSC_HAS_SIDLEMODE | SYSC_HAS_SOFTRESET),
>> > +       .idlemodes      = (SIDLE_FORCE | SIDLE_NO | SIDLE_SMART),
>> > +       .sysc_fields    = &omap_hwmod_sysc_type2,
>> > +};
>> > +
>> > +static struct omap_hwmod_class ti81xx_usbotg_class = {
>> > +       .name = "usbotg",
>> > +       .sysc = &ti81xx_usbhsotg_sysc,
>> > +};
>> > +
>> > +static struct omap_hwmod_irq_info ti81xx_usbss_mpu_irqs[] = {
>> > +       { .name = "usbss-irq", .irq = 17, },
>> > +       { .name = "musb0-irq", .irq = 18, },
>> > +       { .name = "musb1-irq", .irq = 19, },
>> > +       { .irq = -1, },
>> > +};
>> > +
>> > +static struct omap_hwmod_ocp_if ti81xx_l3_slow__usbss = {
>> > +       .master         = &ti81xx_l3_slow_hwmod,
>> > +       .slave          = &ti81xx_usbss_hwmod,
>> > +       .clk            = "usbotg_ick",
>> > +       .addr           = ti81xx_usbss_addr_space,
>> > +       .user           = OCP_USER_MPU,
>> > +       .flags          = OCPIF_SWSUP_IDLE,
>> > +};
>> > +
>> > +static struct omap_hwmod_ocp_if *ti81xx_usbss_slaves[] = {
>> > +       &ti81xx_l3_slow__usbss,
>> > +};
>> > +
>> > +static struct omap_hwmod ti81xx_usbss_hwmod = {
>> > +       .name           = "usb_otg_hs",
>> > +       .mpu_irqs       = ti81xx_usbss_mpu_irqs,
>> > +       .main_clk       = "usbotg_ick",
>> > +       .clkdm_name     = "default_usb_clkdm",
>> > +       .prcm           = {
>> > +               .omap4 = {
>> > +                       .clkctrl_offs = TI816X_CM_DEFAULT_USB_CLKCTRL_OFFSET,
>> > +               },
>> > +       },
>> > +       .slaves         = ti81xx_usbss_slaves,
>> > +       .slaves_cnt     = ARRAY_SIZE(ti81xx_usbss_slaves),
>> > +       .class          = &ti81xx_usbotg_class,
>> > +       .omap_chip      = OMAP_CHIP_INIT(CHIP_IS_TI816X | CHIP_IS_TI814X)
>> > +};
>> >
>> >  static __initdata struct omap_hwmod *ti81xx_hwmods[] = {
>> >        &ti81xx_l3_fast_hwmod,
>> > @@ -668,6 +741,7 @@ static __initdata struct omap_hwmod *ti81xx_hwmods[] = {
>> >        &ti81xx_timer5_hwmod,
>> >        &ti81xx_timer6_hwmod,
>> >        &ti81xx_timer7_hwmod,
>> > +       &ti81xx_usbss_hwmod,
>> >        NULL,
>> >  };
>> >
>> > --
>> > 1.6.2.4
>>
>> looks good to me.
>
> how about you reply with your Acked-by then ?
>
> --
> balbi

yes balbi,
     here it is

Acked-By: Keshava Munegowda <keshava_mgowda@xxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux