Kishore Kadiyala <kishore.kadiy...@ti.com> writes:

> From: Paul Walmsley <p...@pwsan.com>
>
> Update the omap3 hwmod data with the HSMMC info.
>
> Signed-off-by: Paul Walmsley <p...@pwsan.com>
> Signed-off-by: Kevin Hilman <khil...@deeprootsystems.com>
> Signed-off-by: Rajendra Nayak <rna...@ti.com>
> Signed-off-by: Kishore Kadiyala <kishore.kadiy...@ti.com>

[...]

Just notcied a minor issue below, which was probably inherited from my
original hacks on this code...

> +static struct omap_hwmod omap3xxx_mmc1_hwmod = {
> +     .name           = "mmc1_hwmod",
> +     .mpu_irqs       = mmc1_mpu_irqs,
> +     .mpu_irqs_cnt   = ARRAY_SIZE(mmc1_mpu_irqs),
> +     .sdma_reqs      = mmc1_sdma_reqs,
> +     .sdma_reqs_cnt  = ARRAY_SIZE(mmc1_sdma_reqs),
> +     .opt_clks       = mmc1_opt_clks,
> +     .opt_clks_cnt   = ARRAY_SIZE(mmc1_opt_clks),
> +     .main_clk       = "mmchs1_fck",
> +     .prcm           = {
> +             .omap2 = {
> +                     .module_offs = CORE_MOD,
> +                     .prcm_reg_id = 1,
> +                     .module_bit = OMAP3430_GRPSEL_MMC1_SHIFT,

Thee GRPSEL bit isn't the right bit here, and should be the
EN_MMC1_SHIFT value.  I realize, they're probably the same bitfield, but
this should be consistent with the other hwmods.

Same issue with other MMC hwmods.

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

Reply via email to