One minor comment here:
On Tue, 16 Nov 2010, Rajendra Nayak wrote:
> Put infrastructure in place, so arch specific func pointers
> can be hooked up to the platform-independent part of the
> framework.
>
> Signed-off-by: Rajendra Nayak <rna...@ti.com>
> Cc: Paul Walmsley <p...@pwsan.com>
> Cc: Benoit Cousson <b-cous...@ti.com>
> Cc: Kevin Hilman <khil...@deeprootsystems.com>
> ---
> arch/arm/mach-omap2/io.c | 2 +-
> arch/arm/plat-omap/include/plat/powerdomain.h | 22 +++++++++++++++++++++-
> arch/arm/plat-omap/powerdomain.c | 10 ++++++++--
> 3 files changed, 30 insertions(+), 4 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/io.c b/arch/arm/mach-omap2/io.c
> index 40562dd..76c531a 100644
> --- a/arch/arm/mach-omap2/io.c
> +++ b/arch/arm/mach-omap2/io.c
> @@ -316,7 +316,7 @@ void __init omap2_init_common_hw(struct omap_sdrc_params
> *sdrc_cs0,
> {
> u8 skip_setup_idle = 0;
>
> - pwrdm_init(powerdomains_omap);
> + pwrdm_init(powerdomains_omap, NULL);
> clkdm_init(clockdomains_omap, clkdm_autodeps);
> if (cpu_is_omap242x())
> omap2420_hwmod_init();
> diff --git a/arch/arm/plat-omap/include/plat/powerdomain.h
> b/arch/arm/plat-omap/include/plat/powerdomain.h
> index 9ca420d..35e1ef7 100644
> --- a/arch/arm/plat-omap/include/plat/powerdomain.h
> +++ b/arch/arm/plat-omap/include/plat/powerdomain.h
> @@ -117,8 +117,28 @@ struct powerdomain {
> #endif
> };
>
> +struct pwrdm_functions {
> + int (*pwrdm_set_next_pwrst)(struct powerdomain *pwrdm, u8 pwrst);
> + int (*pwrdm_read_next_pwrst)(struct powerdomain *pwrdm);
> + int (*pwrdm_read_pwrst)(struct powerdomain *pwrdm);
> + int (*pwrdm_read_prev_pwrst)(struct powerdomain *pwrdm);
> + int (*pwrdm_set_logic_retst)(struct powerdomain *pwrdm, u8 pwrst);
> + int (*pwrdm_set_mem_onst)(struct powerdomain *pwrdm, u8 bank, u8
> pwrst);
> + int (*pwrdm_set_mem_retst)(struct powerdomain *pwrdm, u8 bank, u8
> pwrst);
> + int (*pwrdm_read_logic_pwrst)(struct powerdomain *pwrdm);
> + int (*pwrdm_read_prev_logic_pwrst)(struct powerdomain *pwrdm);
> + int (*pwrdm_read_logic_retst)(struct powerdomain *pwrdm);
> + int (*pwrdm_read_mem_pwrst)(struct powerdomain *pwrdm, u8 bank);
> + int (*pwrdm_read_prev_mem_pwrst)(struct powerdomain *pwrdm, u8
> bank);
> + int (*pwrdm_read_mem_retst)(struct powerdomain *pwrdm, u8 bank);
> + int (*pwrdm_clear_all_prev_pwrst)(struct powerdomain *pwrdm);
> + int (*pwrdm_enable_hdwr_sar)(struct powerdomain *pwrdm);
> + int (*pwrdm_disable_hdwr_sar)(struct powerdomain *pwrdm);
> + int (*pwrdm_set_lowpwrstchange)(struct powerdomain *pwrdm);
> + int (*pwrdm_wait_transition)(struct powerdomain *pwrdm);
> +};
>
> -void pwrdm_init(struct powerdomain **pwrdm_list);
> +void pwrdm_init(struct powerdomain **pwrdm_list, struct pwrdm_functions
> *custom_funcs);
>
> struct powerdomain *pwrdm_lookup(const char *name);
>
> diff --git a/arch/arm/plat-omap/powerdomain.c
> b/arch/arm/plat-omap/powerdomain.c
> index 9204799..9e2d712 100644
> --- a/arch/arm/plat-omap/powerdomain.c
> +++ b/arch/arm/plat-omap/powerdomain.c
> @@ -80,6 +80,8 @@ static u16 pwrstst_reg_offs;
> /* pwrdm_list contains all registered struct powerdomains */
> static LIST_HEAD(pwrdm_list);
>
> +static struct pwrdm_functions *arch_pwrdm;
> +
> /* Private functions */
>
> static struct powerdomain *_pwrdm_lookup(const char *name)
> @@ -218,7 +220,7 @@ static int _pwrdm_post_transition_cb(struct powerdomain
> *pwrdm, void *unused)
> * registered. No return value. XXX pwrdm_list is not really a
> * "list"; it is an array. Rename appropriately.
> */
> -void pwrdm_init(struct powerdomain **pwrdm_list)
> +void pwrdm_init(struct powerdomain **pwrdm_list, struct pwrdm_functions
> *custom_funcs)
> {
> struct powerdomain **p = NULL;
>
> @@ -234,6 +236,11 @@ void pwrdm_init(struct powerdomain **pwrdm_list)
> return;
> }
>
> + if (!custom_funcs)
> + WARN(1, "No custom pwrdm functions registered\n");
It's probably best to prefix this message with "powerdomain: "
> + else
> + arch_pwrdm = custom_funcs;
> +
> if (pwrdm_list) {
> for (p = pwrdm_list; *p; p++)
> _pwrdm_register(*p);
> @@ -1074,4 +1081,3 @@ int pwrdm_post_transition(void)
> pwrdm_for_each(_pwrdm_post_transition_cb, NULL);
> return 0;
> }
> -
> --
> 1.7.0.4
>
- Paul
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
