[PATCH] clk: renesas: cpg-mssr: Adjust r8a77980 ifdef

[PATCH] clk: renesas: cpg-mssr: Adjust r8a77980 ifdef

[Magnus Damm]

From: Magnus Damm <damm+renesas@opensource.se>

Adjust the R8A77980-specific #ifdefs to use CLK instead of ARCH
to follow same style as other SoCs.

Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
---

 Noticed in renesas-drivers-2018-03-13-v4.16-rc5
 
 drivers/clk/renesas/renesas-cpg-mssr.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--- 0001/drivers/clk/renesas/renesas-cpg-mssr.c
+++ work/drivers/clk/renesas/renesas-cpg-mssr.c	2018-03-20 11:57:19.320607110 +0900
@@ -705,7 +705,7 @@ static const struct of_device_id cpg_mss
 		.data = &r8a77970_cpg_mssr_info,
 	},
 #endif
-#ifdef CONFIG_ARCH_R8A77980
+#ifdef CONFIG_CLK_R8A77980
 	{
 		.compatible = "renesas,r8a77980-cpg-mssr",
 		.data = &r8a77980_cpg_mssr_info,

Re: [PATCH] clk: renesas: cpg-mssr: Adjust r8a77980 ifdef

[Sergei Shtylyov]

Hello!

On 03/20/2018 10:40 AM, Magnus Damm wrote:

> From: Magnus Damm <damm+renesas@opensource.se>
> 
> Adjust the R8A77980-specific #ifdefs to use CLK instead of ARCH
> to follow same style as other SoCs.

   I guess we should add:

Fixes: ce15783c510a ("clk: renesas: cpg-mssr: add R8A77980 support")

   Hopefully, it would be an immutable SHA1...

> Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
> ---
> 
>  Noticed in renesas-drivers-2018-03-13-v4.16-rc5
>  
>  drivers/clk/renesas/renesas-cpg-mssr.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> --- 0001/drivers/clk/renesas/renesas-cpg-mssr.c
> +++ work/drivers/clk/renesas/renesas-cpg-mssr.c	2018-03-20 11:57:19.320607110 +0900
> @@ -705,7 +705,7 @@ static const struct of_device_id cpg_mss
>  		.data = &r8a77970_cpg_mssr_info,
>  	},
>  #endif
> -#ifdef CONFIG_ARCH_R8A77980
> +#ifdef CONFIG_CLK_R8A77980

   Oops, sorry about that overlook. :-<

>  	{
>  		.compatible = "renesas,r8a77980-cpg-mssr",
>  		.data = &r8a77980_cpg_mssr_info,

MBR, Sergei

Re: [PATCH] clk: renesas: cpg-mssr: Adjust r8a77980 ifdef

[Geert Uytterhoeven]

On Tue, Mar 20, 2018 at 8:04 PM, Sergei Shtylyov
<sergei.shtylyov@cogentembedded.com> wrote:
> On 03/20/2018 10:40 AM, Magnus Damm wrote:
>> From: Magnus Damm <damm+renesas@opensource.se>
>>
>> Adjust the R8A77980-specific #ifdefs to use CLK instead of ARCH
>> to follow same style as other SoCs.
>
>    I guess we should add:
>
> Fixes: ce15783c510a ("clk: renesas: cpg-mssr: add R8A77980 support")
>
>    Hopefully, it would be an immutable SHA1...
>
>> Signed-off-by: Magnus Damm <damm+renesas@opensource.se>

Thank you, will queue in clk-renesas-for-v4.17.

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

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