Thanks for the patch.
On Thu, Jun 16, 2016 at 12:27 PM, Geert Uytterhoeven
<geert+renesas@xxxxxxxxx> wrote:
> Let rcar_sysc_init() trigger initialization of the SYSC PM domains from
> DT if called before the early_initcall.
> On failure, it falls back to mapping the passed register block, as
> before.
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
> ---
> v4:
> - New.
> ---
> drivers/soc/renesas/rcar-sysc.c | 20 +++++++++++---------
> 1 file changed, 11 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/soc/renesas/rcar-sysc.c b/drivers/soc/renesas/rcar-sysc.c
> index 59a8152681a59a5c..68d6856c9d3cd192 100644
> --- a/drivers/soc/renesas/rcar-sysc.c
> +++ b/drivers/soc/renesas/rcar-sysc.c
> @@ -164,15 +164,6 @@ static bool rcar_sysc_power_is_off(const struct rcar_sysc_ch *sysc_ch)
> return false;
> }
>
> -void __iomem *rcar_sysc_init(phys_addr_t base)
> -{
> - rcar_sysc_base = ioremap_nocache(base, PAGE_SIZE);
> - if (!rcar_sysc_base)
> - panic("unable to ioremap R-Car SYSC hardware block\n");
Is this check no longer necessary?
> -
> - return rcar_sysc_base;
> -}
> -
> struct rcar_sysc_pd {
> struct generic_pm_domain genpd;
> struct rcar_sysc_ch ch;
> @@ -328,6 +319,9 @@ static int __init rcar_sysc_pd_init(void)
> unsigned int i;
> int error;
>
> + if (rcar_sysc_base)
> + return 0;
> +
> np = of_find_matching_node_and_match(NULL, rcar_sysc_matches, &match);
> if (!np)
> return -ENODEV;
> @@ -405,3 +399,11 @@ out_put:
> return error;
> }
> early_initcall(rcar_sysc_pd_init);
> +
> +void __iomem * __init rcar_sysc_init(phys_addr_t base)
> +{
> + if (rcar_sysc_pd_init())
> + rcar_sysc_base = ioremap_nocache(base, PAGE_SIZE);
> +
> + return rcar_sysc_base;
> +}
> --
> 1.9.1
>
CU
Uli
