From mboxrd@z Thu Jan 1 00:00:00 1970 From: nsekhar@ti.com (Sekhar Nori) Date: Tue, 20 Jan 2015 19:29:24 +0530 Subject: [PATCH] ARM: mach-davinci: Remove driver CDCE949 In-Reply-To: <54BC8F2B.10509@ti.com> References: <1421624259-24127-1-git-send-email-rickard_strandqvist@spectrumdigital.se> <54BC8F2B.10509@ti.com> Message-ID: <54BE5F3C.50907@ti.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Monday 19 January 2015 10:29 AM, Sekhar Nori wrote: > Hi, > > On Monday 19 January 2015 05:07 AM, Rickard Strandqvist wrote: >> Remove driver CDCE949 because it is not used anywhere. >> >> Signed-off-by: Rickard Strandqvist > > >> diff --git a/arch/arm/mach-davinci/board-dm646x-evm.c b/arch/arm/mach-davinci/board-dm646x-evm.c >> index ae129bc..8be5aef 100644 >> --- a/arch/arm/mach-davinci/board-dm646x-evm.c >> +++ b/arch/arm/mach-davinci/board-dm646x-evm.c >> @@ -45,7 +45,6 @@ >> #include >> #include >> #include >> -#include >> >> #include "davinci.h" >> #include "clock.h" >> @@ -399,9 +398,6 @@ static struct i2c_board_info __initdata i2c_info[] = { >> { >> I2C_BOARD_INFO("cpld_video", 0x3b), >> }, >> - { >> - I2C_BOARD_INFO("cdce949", 0x6c), >> - }, >> }; > > There is cdce_clk_init() and associated data structures that also need > to be removed. In the interest of time, I made this change myself. I will be merging the attached patch with this one. Thanks, Sekhar ---8<--- diff --git a/arch/arm/mach-davinci/board-dm646x-evm.c b/arch/arm/mach-davinci/board-dm646x-evm.c index 8be5aef3aca9..846a84ddc28e 100644 --- a/arch/arm/mach-davinci/board-dm646x-evm.c +++ b/arch/arm/mach-davinci/board-dm646x-evm.c @@ -711,31 +711,6 @@ static void __init evm_init_i2c(void) evm_init_video(); } -#define CDCE949_XIN_RATE 27000000 - -/* CDCE949 support - "lpsc" field is overridden to work as clock number */ -static struct clk cdce_clk_in = { - .name = "cdce_xin", - .rate = CDCE949_XIN_RATE, -}; - -static struct clk_lookup cdce_clks[] = { - CLK(NULL, "xin", &cdce_clk_in), - CLK(NULL, NULL, NULL), -}; - -static void __init cdce_clk_init(void) -{ - struct clk_lookup *c; - struct clk *clk; - - for (c = cdce_clks; c->clk; c++) { - clk = c->clk; - clkdev_add(c); - clk_register(clk); - } -} - #define DM6467T_EVM_REF_FREQ 33000000 static void __init davinci_map_io(void) @@ -744,8 +719,6 @@ static void __init davinci_map_io(void) if (machine_is_davinci_dm6467tevm()) davinci_set_refclk_rate(DM6467T_EVM_REF_FREQ); - - cdce_clk_init(); } #define DM646X_EVM_PHY_ID "davinci_mdio-0:01" From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753639AbbATOAG (ORCPT ); Tue, 20 Jan 2015 09:00:06 -0500 Received: from devils.ext.ti.com ([198.47.26.153]:56848 "EHLO devils.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751242AbbATOAE (ORCPT ); Tue, 20 Jan 2015 09:00:04 -0500 Message-ID: <54BE5F3C.50907@ti.com> Date: Tue, 20 Jan 2015 19:29:24 +0530 From: Sekhar Nori User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.3.0 MIME-Version: 1.0 To: Rickard Strandqvist , Kevin Hilman CC: Russell King , , Subject: Re: [PATCH] ARM: mach-davinci: Remove driver CDCE949 References: <1421624259-24127-1-git-send-email-rickard_strandqvist@spectrumdigital.se> <54BC8F2B.10509@ti.com> In-Reply-To: <54BC8F2B.10509@ti.com> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Monday 19 January 2015 10:29 AM, Sekhar Nori wrote: > Hi, > > On Monday 19 January 2015 05:07 AM, Rickard Strandqvist wrote: >> Remove driver CDCE949 because it is not used anywhere. >> >> Signed-off-by: Rickard Strandqvist > > >> diff --git a/arch/arm/mach-davinci/board-dm646x-evm.c b/arch/arm/mach-davinci/board-dm646x-evm.c >> index ae129bc..8be5aef 100644 >> --- a/arch/arm/mach-davinci/board-dm646x-evm.c >> +++ b/arch/arm/mach-davinci/board-dm646x-evm.c >> @@ -45,7 +45,6 @@ >> #include >> #include >> #include >> -#include >> >> #include "davinci.h" >> #include "clock.h" >> @@ -399,9 +398,6 @@ static struct i2c_board_info __initdata i2c_info[] = { >> { >> I2C_BOARD_INFO("cpld_video", 0x3b), >> }, >> - { >> - I2C_BOARD_INFO("cdce949", 0x6c), >> - }, >> }; > > There is cdce_clk_init() and associated data structures that also need > to be removed. In the interest of time, I made this change myself. I will be merging the attached patch with this one. Thanks, Sekhar ---8<--- diff --git a/arch/arm/mach-davinci/board-dm646x-evm.c b/arch/arm/mach-davinci/board-dm646x-evm.c index 8be5aef3aca9..846a84ddc28e 100644 --- a/arch/arm/mach-davinci/board-dm646x-evm.c +++ b/arch/arm/mach-davinci/board-dm646x-evm.c @@ -711,31 +711,6 @@ static void __init evm_init_i2c(void) evm_init_video(); } -#define CDCE949_XIN_RATE 27000000 - -/* CDCE949 support - "lpsc" field is overridden to work as clock number */ -static struct clk cdce_clk_in = { - .name = "cdce_xin", - .rate = CDCE949_XIN_RATE, -}; - -static struct clk_lookup cdce_clks[] = { - CLK(NULL, "xin", &cdce_clk_in), - CLK(NULL, NULL, NULL), -}; - -static void __init cdce_clk_init(void) -{ - struct clk_lookup *c; - struct clk *clk; - - for (c = cdce_clks; c->clk; c++) { - clk = c->clk; - clkdev_add(c); - clk_register(clk); - } -} - #define DM6467T_EVM_REF_FREQ 33000000 static void __init davinci_map_io(void) @@ -744,8 +719,6 @@ static void __init davinci_map_io(void) if (machine_is_davinci_dm6467tevm()) davinci_set_refclk_rate(DM6467T_EVM_REF_FREQ); - - cdce_clk_init(); } #define DM646X_EVM_PHY_ID "davinci_mdio-0:01"