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

[Kevin Hilman <khilman@ti.com>]

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