diff mbox

ARM: mach-davinci: Remove driver CDCE949

Message ID 54BE5F3C.50907@ti.com (mailing list archive)
State New, archived
Headers show

Commit Message

Sekhar Nori Jan. 20, 2015, 1:59 p.m. UTC
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 <rickard_strandqvist@spectrumdigital.se>
> 
> 
>> 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 <mach/irqs.h>
>>  #include <mach/serial.h>
>>  #include <mach/clock.h>
>> -#include <mach/cdce949.h>
>>  
>>  #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<---

Comments

Rickard Strandqvist Jan. 20, 2015, 6:35 p.m. UTC | #1
2015-01-20 14:59 GMT+01:00 Sekhar Nori <nsekhar@ti.com>:
> 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 <rickard_strandqvist@spectrumdigital.se>
>>
>>
>>> 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 <mach/irqs.h>
>>>  #include <mach/serial.h>
>>>  #include <mach/clock.h>
>>> -#include <mach/cdce949.h>
>>>
>>>  #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"
>


Hello Sekhar

I had started a bit on this, but seeing that I missed the answer you
that I had started.
But then there is a solution now, and that was the important thing :)

Kind regards
Rickard Strandqvist
diff mbox

Patch

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"