Re: [PATCH] OMAP3: CLK: FIX: warning SR fclk clk domain not found

[Kevin Hilman <khilman@deeprootsystems.com>]

Ambresh K <ambresh@ti.com> writes:

> Added clock domain name to sr[1,2] clk structure to fix warning
> while initialising sr hwmod during boot-up.
>
> Signed-off-by: Ambresh K <ambresh@ti.com>

Please add a better subject and changelog.  Something that would be
more useful when looking at this in git history down the road.
Something like "add SR clocks to WKUP clockdomain" for the suject is
more helpful.

Also, for the changelog, explain more about SmartReflex having its own
powerdomain but no clockdomain, thus why it's clocks are in the WKUP
clockdomain.

Kevin


> ---
>  arch/arm/mach-omap2/clock3xxx_data.c |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/clock3xxx_data.c b/arch/arm/mach-omap2/clock3xxx_data.c
> index d5153b6..8a49d8a 100644
> --- a/arch/arm/mach-omap2/clock3xxx_data.c
> +++ b/arch/arm/mach-omap2/clock3xxx_data.c
> @@ -3100,6 +3100,7 @@ static struct clk sr1_fck = {
>  	.parent		= &sys_ck,
>  	.enable_reg	= OMAP_CM_REGADDR(WKUP_MOD, CM_FCLKEN),
>  	.enable_bit	= OMAP3430_EN_SR1_SHIFT,
> +	.clkdm_name	= "wkup_clkdm",
>  	.recalc		= &followparent_recalc,
>  };
>  
> @@ -3110,6 +3111,7 @@ static struct clk sr2_fck = {
>  	.parent		= &sys_ck,
>  	.enable_reg	= OMAP_CM_REGADDR(WKUP_MOD, CM_FCLKEN),
>  	.enable_bit	= OMAP3430_EN_SR2_SHIFT,
> +	.clkdm_name	= "wkup_clkdm",
>  	.recalc		= &followparent_recalc,
>  };
>  
> -- 
> 1.6.0.4