diff mbox

clk: renesas: cpg-mssr: Adjust r8a77980 ifdef

Message ID 152153161677.8310.9675462405281337962.sendpatchset@little-apple (mailing list archive)
State Accepted
Delegated to: Geert Uytterhoeven
Headers show

Commit Message

Magnus Damm March 20, 2018, 7:40 a.m. UTC
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(-)

Comments

Sergei Shtylyov March 20, 2018, 7:04 p.m. UTC | #1
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
Geert Uytterhoeven March 21, 2018, 9:55 a.m. UTC | #2
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
diff mbox

Patch

--- 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,