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

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

 



Ambresh K <ambresh@xxxxxx> 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@xxxxxx>

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
--
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