Re: [PATCH 3/5] OMAP3: hwmod data: Add HSMMC

Kevin Hilman
2011-02-08T22:30:38Z

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

> From: Paul Walmsley <paul@pwsan.com>
>
> Update the omap3 hwmod data with the HSMMC info.
>
> Signed-off-by: Paul Walmsley <paul@pwsan.com>
> Signed-off-by: Kevin Hilman <khilman@deeprootsystems.com>
> Signed-off-by: Rajendra Nayak <rnayak@ti.com>
> Signed-off-by: Kishore Kadiyala <kishore.kadiyala@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