Re: [PATCH v5 2/8] OMAP2430: hwmod data: Add HSMMC

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

 



Hi Paul,

On Sat, Feb 26, 2011 at 3:07 AM, Paul Walmsley <paul@xxxxxxxxx> wrote:
> Hello Kishore,
>
> There are several problems with this data that generate warning messages
> on boot on 2430SDP.
>
> Did you actually test this on 2430SDP?  The warning messages would have
> been obvious in the boot log.

Thanks for catching this.
Sorry I missed the warning messages in the boot log.
I will resend the patch fixing the comments.

>
> Comments below:
>
> On Fri, 25 Feb 2011, Kishore Kadiyala wrote:
>
>> From: Paul Walmsley <paul@xxxxxxxxx>
>>
>> Update the omap2430 hwmod data with the HSMMC info.
>>
>> Signed-off-by: Paul Walmsley <paul@xxxxxxxxx>
>> Signed-off-by: Kevin Hilman <khilman@xxxxxxxxxxxxxxxxxxx>
>> Signed-off-by: Rajendra Nayak <rnayak@xxxxxx
>> Signed-off-by: Kishore Kadiyala <kishore.kadiyala@xxxxxx>
>> Cc: Benoit Cousson <b-cousson@xxxxxx>
>> ---
>>  arch/arm/mach-omap2/omap_hwmod_2430_data.c |  146 ++++++++++++++++++++++++++++
>>  1 files changed, 146 insertions(+), 0 deletions(-)
>>
>> diff --git a/arch/arm/mach-omap2/omap_hwmod_2430_data.c b/arch/arm/mach-omap2/omap_hwmod_2430_data.c
>> index de0015d..a1c3c5e 100644
>> --- a/arch/arm/mach-omap2/omap_hwmod_2430_data.c
>> +++ b/arch/arm/mach-omap2/omap_hwmod_2430_data.c
>> @@ -54,6 +54,8 @@ static struct omap_hwmod omap2430_dma_system_hwmod;
>>  static struct omap_hwmod omap2430_mcspi1_hwmod;
>>  static struct omap_hwmod omap2430_mcspi2_hwmod;
>>  static struct omap_hwmod omap2430_mcspi3_hwmod;
>> +static struct omap_hwmod omap2430_mmc1_hwmod;
>> +static struct omap_hwmod omap2430_mmc2_hwmod;
>>
>>  /* L3 -> L4_CORE interface */
>>  static struct omap_hwmod_ocp_if omap2430_l3_main__l4_core = {
>> @@ -250,6 +252,42 @@ static struct omap_hwmod_ocp_if *omap2430_usbhsotg_slaves[] = {
>>       &omap2430_l4_core__usbhsotg,
>>  };
>>
>> +/* L4 CORE -> MMC1 interface */
>> +static struct omap_hwmod_addr_space omap2430_mmc1_addr_space[] = {
>> +     {
>> +             .pa_start       = 0x4809c000,
>> +             .pa_end         = 0x4809c1ff,
>> +             .flags          = ADDR_TYPE_RT,
>> +     },
>> +};
>> +
>> +static struct omap_hwmod_ocp_if omap2430_l4_core__mmc1 = {
>> +     .master         = &omap2430_l4_core_hwmod,
>> +     .slave          = &omap2430_mmc1_hwmod,
>> +     .clk            = "mmchs1_ick",
>> +     .addr           = omap2430_mmc1_addr_space,
>> +     .addr_cnt       = ARRAY_SIZE(omap2430_mmc1_addr_space),
>> +     .user           = OCP_USER_MPU | OCP_USER_SDMA,
>> +};
>> +
>> +/* L4 CORE -> MMC2 interface */
>> +static struct omap_hwmod_addr_space omap2430_mmc2_addr_space[] = {
>> +     {
>> +             .pa_start       = 0x480b4000,
>> +             .pa_end         = 0x480b41ff,
>> +             .flags          = ADDR_TYPE_RT,
>> +     },
>> +};
>> +
>> +static struct omap_hwmod_ocp_if omap2430_l4_core__mmc2 = {
>> +     .master         = &omap2430_l4_core_hwmod,
>> +     .slave          = &omap2430_mmc2_hwmod,
>> +     .addr           = omap2430_mmc2_addr_space,
>> +     .clk            = "mmchs2_ick",
>> +     .addr_cnt       = ARRAY_SIZE(omap2430_mmc2_addr_space),
>> +     .user           = OCP_USER_MPU | OCP_USER_SDMA,
>> +};
>> +
>>  /* Slave interfaces on the L4_CORE interconnect */
>>  static struct omap_hwmod_ocp_if *omap2430_l4_core_slaves[] = {
>>       &omap2430_l3_main__l4_core,
>> @@ -258,6 +296,8 @@ static struct omap_hwmod_ocp_if *omap2430_l4_core_slaves[] = {
>>  /* Master interfaces on the L4_CORE interconnect */
>>  static struct omap_hwmod_ocp_if *omap2430_l4_core_masters[] = {
>>       &omap2430_l4_core__l4_wkup,
>> +     &omap2430_l4_core__mmc1,
>> +     &omap2430_l4_core__mmc2,
>>  };
>>
>>  /* L4 CORE */
>> @@ -1508,6 +1548,110 @@ static struct omap_hwmod omap2430_usbhsotg_hwmod = {
>>
>>
>>
>> +/* MMC/SD/SDIO common */
>> +
>> +static struct omap_hwmod_class_sysconfig omap2430_mmc_sysc = {
>> +     .rev_offs       = 0x1fc,
>> +     .sysc_offs      = 0x10,
>> +     .syss_offs      = 0x14,
>> +     .sysc_flags     = (SYSC_HAS_CLOCKACTIVITY | SYSC_HAS_SIDLEMODE |
>> +                        SYSC_HAS_ENAWAKEUP | SYSC_HAS_SOFTRESET |
>> +                        SYSC_HAS_AUTOIDLE | SYSS_HAS_RESET_STATUS),
>> +     .idlemodes      = (SIDLE_FORCE | SIDLE_NO | SIDLE_SMART),
>> +     .sysc_fields    = &omap_hwmod_sysc_type1,
>> +};
>> +
>> +static struct omap_hwmod_class omap2430_mmc_class = {
>> +     .name = "mmc",
>> +     .sysc = &omap2430_mmc_sysc,
>> +};
>> +
>> +/* MMC/SD/SDIO1 */
>> +
>> +static struct omap_hwmod_irq_info omap2430_mmc1_mpu_irqs[] = {
>> +     { .irq = 83 },
>> +};
>> +
>> +static struct omap_hwmod_dma_info omap2430_mmc1_sdma_reqs[] = {
>> +     { .name = "tx", .dma_req = 61 }, /* DMA_MMC1_TX */
>> +     { .name = "rx", .dma_req = 62 }, /* DMA_MMC1_RX */
>> +};
>> +
>> +static struct omap_hwmod_opt_clk omap2430_mmc1_opt_clks[] = {
>> +     { .role = "dbck", .clk = "mmchsdb_fck" },
>
> This is not the correct name of the clock.  This generates warnings on
> boot and prevents the HSMMC block from being soft-reset.

Ok , will fix

>
>> +};
>> +
>> +static struct omap_hwmod_ocp_if *omap2430_mmc1_slaves[] = {
>> +     &omap2430_l4_core__mmc1,
>> +};
>> +
>> +static struct omap_hwmod omap2430_mmc1_hwmod = {
>> +     .name           = "mmc1",
>> +     .mpu_irqs       = omap2430_mmc1_mpu_irqs,
>> +     .mpu_irqs_cnt   = ARRAY_SIZE(omap2430_mmc1_mpu_irqs),
>> +     .sdma_reqs      = omap2430_mmc1_sdma_reqs,
>> +     .sdma_reqs_cnt  = ARRAY_SIZE(omap2430_mmc1_sdma_reqs),
>> +     .opt_clks       = omap2430_mmc1_opt_clks,
>> +     .opt_clks_cnt   = ARRAY_SIZE(omap2430_mmc1_opt_clks),
>> +     .main_clk       = "mmchs1_fck",
>> +     .prcm           = {
>> +             .omap2 = {
>> +                     .module_offs = CORE_MOD,
>> +                     .prcm_reg_id = 2,
>> +                     .module_bit  = OMAP2430_EN_MMCHS1_SHIFT,
>> +                     .idlest_reg_id = 2,
>> +                     .idlest_idle_bit = OMAP2430_ST_MMCHS1_SHIFT,
>> +             },
>> +     },
>> +     .slaves         = omap2430_mmc1_slaves,
>> +     .slaves_cnt     = ARRAY_SIZE(omap2430_mmc1_slaves),
>> +     .class          = &omap2430_mmc_class,
>> +     .omap_chip      = OMAP_CHIP_INIT(CHIP_IS_OMAP2430),
>
> This needs to have HWMOD_CONTROL_OPT_CLKS_IN_RESET set in the flags,
> otherwise the softreset will not be possible.  See the OMAP2430 TRM Rev Z
> [SWPU090Z] Table 23-10 "MMCHS_SYSSTATUS", the RESETDONE bit.

Ok, will update

>
>> +};
>> +
>> +/* MMC/SD/SDIO2 */
>> +
>> +static struct omap_hwmod_irq_info omap2430_mmc2_mpu_irqs[] = {
>> +     { .irq = 86 },
>> +};
>> +
>> +static struct omap_hwmod_dma_info omap2430_mmc2_sdma_reqs[] = {
>> +     { .name = "tx", .dma_req = 47 }, /* DMA_MMC2_TX */
>> +     { .name = "rx", .dma_req = 48 }, /* DMA_MMC2_RX */
>> +};
>> +
>> +static struct omap_hwmod_opt_clk omap2430_mmc2_opt_clks[] = {
>> +     { .role = "dbck", .clk = "mmchsdb_fck" },
>
> This is not the correct name of the clock.  This generates warnings on
> boot and prevents the HSMMC block from being soft-reset.
>
>> +};
>> +
>> +static struct omap_hwmod_ocp_if *omap2430_mmc2_slaves[] = {
>> +     &omap2430_l4_core__mmc2,
>> +};
>> +
>> +static struct omap_hwmod omap2430_mmc2_hwmod = {
>> +     .name           = "mmc2",
>> +     .mpu_irqs       = omap2430_mmc2_mpu_irqs,
>> +     .mpu_irqs_cnt   = ARRAY_SIZE(omap2430_mmc2_mpu_irqs),
>> +     .sdma_reqs      = omap2430_mmc2_sdma_reqs,
>> +     .sdma_reqs_cnt  = ARRAY_SIZE(omap2430_mmc2_sdma_reqs),
>> +     .opt_clks       = omap2430_mmc2_opt_clks,
>> +     .opt_clks_cnt   = ARRAY_SIZE(omap2430_mmc2_opt_clks),
>> +     .main_clk       = "mmchs2_fck",
>> +     .prcm           = {
>> +             .omap2 = {
>> +                     .module_offs = CORE_MOD,
>> +                     .prcm_reg_id = 2,
>> +                     .module_bit  = OMAP2430_EN_MMCHS2_SHIFT,
>> +                     .idlest_reg_id = 2,
>> +                     .idlest_idle_bit = OMAP2430_ST_MMCHS2_SHIFT,
>> +             },
>> +     },
>> +     .slaves         = omap2430_mmc2_slaves,
>> +     .slaves_cnt     = ARRAY_SIZE(omap2430_mmc2_slaves),
>> +     .class          = &omap2430_mmc_class,
>> +     .omap_chip      = OMAP_CHIP_INIT(CHIP_IS_OMAP2430),
>
> This needs to have HWMOD_CONTROL_OPT_CLKS_IN_RESET set in the flags,
> otherwise the softreset will not be possible.  See the OMAP2430 TRM Rev Z
> [SWPU090Z] Table 23-10 "MMCHS_SYSSTATUS", the RESETDONE bit.
>

Ok, will update

<snip>

Thanks,
Kishore
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux