diff mbox

ARM: shmobile: Remove Koelsch DT reference legacy clock bits

Message ID 20140217072128.27086.81905.sendpatchset@w520 (mailing list archive)
State New, archived
Headers show

Commit Message

Magnus Damm Feb. 17, 2014, 7:21 a.m. UTC
From: Magnus Damm <damm@opensource.se>

Koelsch DT reference is these days built for multiplatform
only which means that CCF comes with the package. Remove
unused legacy code ifdefs to clean up the code.

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

 Written against renesas-devel-v3.14-rc3-20140217

 arch/arm/mach-shmobile/board-koelsch-reference.c |    5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

Comments

Laurent Pinchart Feb. 17, 2014, 11:07 a.m. UTC | #1
Hi Magnus,

Thank you for the patch.

On Monday 17 February 2014 16:21:28 Magnus Damm wrote:
> From: Magnus Damm <damm@opensource.se>
> 
> Koelsch DT reference is these days built for multiplatform
> only which means that CCF comes with the package. Remove
> unused legacy code ifdefs to clean up the code.
> 
> Signed-off-by: Magnus Damm <damm@opensource.se>

Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

> ---
> 
>  Written against renesas-devel-v3.14-rc3-20140217
> 
>  arch/arm/mach-shmobile/board-koelsch-reference.c |    5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)
> 
> --- 0001/arch/arm/mach-shmobile/board-koelsch-reference.c
> +++ work/arch/arm/mach-shmobile/board-koelsch-reference.c	2014-02-17
> 16:10:07.000000000 +0900 @@ -84,7 +84,6 @@ static void __init
> koelsch_add_du_device
> 
>  static void __init koelsch_add_standard_devices(void)
>  {
> -#ifdef CONFIG_COMMON_CLK
>  	/*
>  	 * This is a really crude hack to provide clkdev support to the CMT and
>  	 * DU devices until they get moved to DT.
> @@ -125,9 +124,7 @@ static void __init koelsch_add_standard_
>  			clk_put(clk);
>  		}
>  	}
> -#else
> -	r8a7791_clock_init();
> -#endif
> +
>  	r8a7791_add_dt_devices();
>  	of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-sh" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
Simon Horman Feb. 18, 2014, 12:11 a.m. UTC | #2
On Mon, Feb 17, 2014 at 04:21:28PM +0900, Magnus Damm wrote:
> From: Magnus Damm <damm@opensource.se>
> 
> Koelsch DT reference is these days built for multiplatform
> only which means that CCF comes with the package. Remove
> unused legacy code ifdefs to clean up the code.
> 
> Signed-off-by: Magnus Damm <damm@opensource.se>

Thanks, I have queued this up.
diff mbox

Patch

--- 0001/arch/arm/mach-shmobile/board-koelsch-reference.c
+++ work/arch/arm/mach-shmobile/board-koelsch-reference.c	2014-02-17 16:10:07.000000000 +0900
@@ -84,7 +84,6 @@  static void __init koelsch_add_du_device
 
 static void __init koelsch_add_standard_devices(void)
 {
-#ifdef CONFIG_COMMON_CLK
 	/*
 	 * This is a really crude hack to provide clkdev support to the CMT and
 	 * DU devices until they get moved to DT.
@@ -125,9 +124,7 @@  static void __init koelsch_add_standard_
 			clk_put(clk);
 		}
 	}
-#else
-	r8a7791_clock_init();
-#endif
+
 	r8a7791_add_dt_devices();
 	of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);