diff mbox

[V2] powerpc: remove the PPC_CLOCK dependency

Message ID 1362561364-16073-1-git-send-email-Yuantian.Tang@freescale.com (mailing list archive)
State Superseded, archived
Delegated to: Kumar Gala
Headers show

Commit Message

tang yuantian March 6, 2013, 9:16 a.m. UTC
From: Tang Yuantian <Yuantian.Tang@freescale.com>

config FSL_SOC and CPM do not really depend on PPC_CLOCK. So remove it.
PPC_CLOCK also keeps powerpc archtecture from supporting COMMON_CLK.

Signed-off-by: Tang Yuantian <Yuantian.Tang@freescale.com>
---
v2: correct the title

 arch/powerpc/Kconfig           |    1 -
 arch/powerpc/platforms/Kconfig |    1 -
 2 files changed, 0 insertions(+), 2 deletions(-)

Comments

Kumar Gala March 7, 2013, 4:32 p.m. UTC | #1
On Mar 6, 2013, at 3:16 AM, <Yuantian.Tang@freescale.com> <Yuantian.Tang@freescale.com> wrote:

> From: Tang Yuantian <Yuantian.Tang@freescale.com>
> 
> config FSL_SOC and CPM do not really depend on PPC_CLOCK. So remove it.
> PPC_CLOCK also keeps powerpc archtecture from supporting COMMON_CLK.
> 
> Signed-off-by: Tang Yuantian <Yuantian.Tang@freescale.com>
> ---
> v2: correct the title
> 
> arch/powerpc/Kconfig           |    1 -
> arch/powerpc/platforms/Kconfig |    1 -
> 2 files changed, 0 insertions(+), 2 deletions(-)

Does the Flexcan driver use this?

- k

> 
> diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
> index 352f416..383485b 100644
> --- a/arch/powerpc/Kconfig
> +++ b/arch/powerpc/Kconfig
> @@ -655,7 +655,6 @@ config SBUS
> config FSL_SOC
> 	bool
> 	select HAVE_CAN_FLEXCAN if NET && CAN
> -	select PPC_CLOCK
> 
> config FSL_PCI
>  	bool
> diff --git a/arch/powerpc/platforms/Kconfig b/arch/powerpc/platforms/Kconfig
> index e7a896a..aba81cd 100644
> --- a/arch/powerpc/platforms/Kconfig
> +++ b/arch/powerpc/platforms/Kconfig
> @@ -343,7 +343,6 @@ config FSL_ULI1575
> 
> config CPM
> 	bool
> -	select PPC_CLOCK
> 
> config OF_RTC
> 	bool
> -- 
> 1.7.0.4
>
tang yuantian March 8, 2013, 2:57 a.m. UTC | #2
> Subject: Re: [PATCH][V2] powerpc: remove the PPC_CLOCK dependency
> 
> 
> On Mar 6, 2013, at 3:16 AM, <Yuantian.Tang@freescale.com>
> <Yuantian.Tang@freescale.com> wrote:
> 
> > From: Tang Yuantian <Yuantian.Tang@freescale.com>
> >
> > config FSL_SOC and CPM do not really depend on PPC_CLOCK. So remove it.
> > PPC_CLOCK also keeps powerpc archtecture from supporting COMMON_CLK.
> >
> > Signed-off-by: Tang Yuantian <Yuantian.Tang@freescale.com>
> > ---
> > v2: correct the title
> >
> > arch/powerpc/Kconfig           |    1 -
> > arch/powerpc/platforms/Kconfig |    1 -
> > 2 files changed, 0 insertions(+), 2 deletions(-)
> 
> Does the Flexcan driver use this?
> 

In SDK, Flexcan did select the PPC_CLOCK, in upstream, it didn't.
Whatever it selects PPC_CLOCK or not, it doesn't use it.

Added the patch author to this loop.

- Yuantian

> - k
> 
> >
> > diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig index
> > 352f416..383485b 100644
> > --- a/arch/powerpc/Kconfig
> > +++ b/arch/powerpc/Kconfig
> > @@ -655,7 +655,6 @@ config SBUS
> > config FSL_SOC
> > 	bool
> > 	select HAVE_CAN_FLEXCAN if NET && CAN
> > -	select PPC_CLOCK
> >
> > config FSL_PCI
> >  	bool
> > diff --git a/arch/powerpc/platforms/Kconfig
> > b/arch/powerpc/platforms/Kconfig index e7a896a..aba81cd 100644
> > --- a/arch/powerpc/platforms/Kconfig
> > +++ b/arch/powerpc/platforms/Kconfig
> > @@ -343,7 +343,6 @@ config FSL_ULI1575
> >
> > config CPM
> > 	bool
> > -	select PPC_CLOCK
> >
> > config OF_RTC
> > 	bool
> > --
> > 1.7.0.4
> >
>
diff mbox

Patch

diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
index 352f416..383485b 100644
--- a/arch/powerpc/Kconfig
+++ b/arch/powerpc/Kconfig
@@ -655,7 +655,6 @@  config SBUS
 config FSL_SOC
 	bool
 	select HAVE_CAN_FLEXCAN if NET && CAN
-	select PPC_CLOCK
 
 config FSL_PCI
  	bool
diff --git a/arch/powerpc/platforms/Kconfig b/arch/powerpc/platforms/Kconfig
index e7a896a..aba81cd 100644
--- a/arch/powerpc/platforms/Kconfig
+++ b/arch/powerpc/platforms/Kconfig
@@ -343,7 +343,6 @@  config FSL_ULI1575
 
 config CPM
 	bool
-	select PPC_CLOCK
 
 config OF_RTC
 	bool