Hi Simon,

On Wed, Mar 2, 2016 at 2:43 AM, Simon Horman <[email protected]> wrote:
> Make use of ARCH_RENESAS in place of ARCH_SHMOBILE.
>
> This is part of an ongoing process to migrate from ARCH_SHMOBILE to
> ARCH_RENESAS the motivation for which being that RENESAS seems to be a more
> appropriate name than SHMOBILE for the majority of Renesas ARM based SoCs.
>
> ARCH_RENESAS should cover all cases where both CONFIG_OF and
> ARCH_SHMOBILE are enabled.
>
> Signed-off-by: Simon Horman <[email protected]>

Acked-by: Geert Uytterhoeven <[email protected]>

If you intend to drop ARCH_SHMOBILE from arch/arm/mach-shmobile/Kconfig before
dropping the whole "if (...) { ... }" block below" (cfr. "drivers: sh: Stop
using the legacy clock domain on ARM",
http://www.spinics.net/lists/linux-renesas-soc/msg00869.html).

Note that the SH-people may resurrect (a variant of) the block when they start
migrating to DT and CCF.

> ---
>  drivers/sh/pm_runtime.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
>  Based on v4.5-rc6
>
> diff --git a/drivers/sh/pm_runtime.c b/drivers/sh/pm_runtime.c
> index a9bac3bf20de..aa2ce227e3eb 100644
> --- a/drivers/sh/pm_runtime.c
> +++ b/drivers/sh/pm_runtime.c
> @@ -34,7 +34,7 @@ static struct pm_clk_notifier_block platform_bus_notifier = 
> {
>
>  static int __init sh_pm_runtime_init(void)
>  {
> -       if (IS_ENABLED(CONFIG_OF) && IS_ENABLED(CONFIG_ARCH_SHMOBILE)) {
> +       if (IS_ENABLED(CONFIG_ARCH_RENESAS)) {
>                 if (!of_find_compatible_node(NULL, NULL,
>                                              "renesas,cpg-mstp-clocks"))
>                         return 0;

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

Reply via email to