Re: [PATCH 03/23] mfd/db8500-prcmu: initialize DB8500 PRCMU regs

[Samuel Ortiz <sameo@linux.intel.com>]

Hi Linus,

On Fri, Aug 12, 2011 at 10:27:50AM +0200, Linus Walleij wrote:
> From: Mattias Nilsson <mattias.i.nilsson@stericsson.com>
> 
> Some clocks may be force enabled when we probe the
> driver, but they need to be turned off by default so
> we have a known state. We call this the register
> initialization function if we need more stuff in
> there in the future.
> 
> Signed-off-by: Mattias Nilsson <mattias.i.nilsson@stericsson.com>
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> ---
>  drivers/mfd/db8500-prcmu.c |   12 ++++++++++++
>  1 files changed, 12 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/mfd/db8500-prcmu.c b/drivers/mfd/db8500-prcmu.c
> index dcc690e..e2c4a26 100644
> --- a/drivers/mfd/db8500-prcmu.c
> +++ b/drivers/mfd/db8500-prcmu.c
> @@ -1840,6 +1840,16 @@ void __init prcmu_early_init(void)
>  	}
>  }
>  
> +static void __init init_prcm_registers(void)
Could we have a less generic function name here ?

Cheers,
Samuel.

-- 
Intel Open Source Technology Centre
http://oss.intel.com/