linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
* [POWERPC] MPC52xx is a 6xx variant, remove PPC_52xx config option
@ 2006-10-26  3:52 Grant Likely
  2006-10-27  9:56 ` Andrey Volkov
  0 siblings, 1 reply; 4+ messages in thread
From: Grant Likely @ 2006-10-26  3:52 UTC (permalink / raw)
  To: Benjamin Herrenschmidt, linuxppc-embedded

Signed-off-by: Grant Likely <grant.likely@secretlab.ca>
---
 arch/powerpc/Kconfig |    5 -----
 1 files changed, 0 insertions(+), 5 deletions(-)

diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
index f9b11c1..6bff8b7 100644
--- a/arch/powerpc/Kconfig
+++ b/arch/powerpc/Kconfig
@@ -126,11 +126,6 @@ config CLASSIC32
 	  systems, 64 bit IBM RS/6000 or an Apple G5, choose 6xx.
 	  Note that the kernel runs in 32-bit mode even on 64-bit chips.
 
-config PPC_52xx
-	bool "Freescale 52xx"
-	select 6xx
-	select PPC_FPU
-	
 config PPC_82xx
 	bool "Freescale 82xx"
 	select 6xx
-- 
1.4.3.rc2.g0503

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [POWERPC] MPC52xx is a 6xx variant, remove PPC_52xx config option
  2006-10-26  3:52 [POWERPC] MPC52xx is a 6xx variant, remove PPC_52xx config option Grant Likely
@ 2006-10-27  9:56 ` Andrey Volkov
  2006-10-27 10:24   ` Benjamin Herrenschmidt
  0 siblings, 1 reply; 4+ messages in thread
From: Andrey Volkov @ 2006-10-27  9:56 UTC (permalink / raw)
  To: Grant Likely; +Cc: linuxppc-embedded

Grant Likely wrote:
> Signed-off-by: Grant Likely <grant.likely@secretlab.ca>
> ---
>  arch/powerpc/Kconfig |    5 -----
>  1 files changed, 0 insertions(+), 5 deletions(-)
> 
> diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
> index f9b11c1..6bff8b7 100644
> --- a/arch/powerpc/Kconfig
> +++ b/arch/powerpc/Kconfig
> @@ -126,11 +126,6 @@ config CLASSIC32
>  	  systems, 64 bit IBM RS/6000 or an Apple G5, choose 6xx.
>  	  Note that the kernel runs in 32-bit mode even on 64-bit chips.
>  
> -config PPC_52xx
> -	bool "Freescale 52xx"
> -	select 6xx
> -	select PPC_FPU
> -	
>  config PPC_82xx
>  	bool "Freescale 82xx"
>  	select 6xx
Please _don't_ do it, since mpc5200 platform drivers depend from this
option.

--
Regards
Andrey Volkov

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [POWERPC] MPC52xx is a 6xx variant, remove PPC_52xx config option
  2006-10-27  9:56 ` Andrey Volkov
@ 2006-10-27 10:24   ` Benjamin Herrenschmidt
  2006-10-27 13:21     ` Grant Likely
  0 siblings, 1 reply; 4+ messages in thread
From: Benjamin Herrenschmidt @ 2006-10-27 10:24 UTC (permalink / raw)
  To: Andrey Volkov; +Cc: linuxppc-embedded

On Fri, 2006-10-27 at 13:56 +0400, Andrey Volkov wrote:
> Grant Likely wrote:
> > Signed-off-by: Grant Likely <grant.likely@secretlab.ca>
> > ---
> >  arch/powerpc/Kconfig |    5 -----
> >  1 files changed, 0 insertions(+), 5 deletions(-)
> > 
> > diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
> > index f9b11c1..6bff8b7 100644
> > --- a/arch/powerpc/Kconfig
> > +++ b/arch/powerpc/Kconfig
> > @@ -126,11 +126,6 @@ config CLASSIC32
> >  	  systems, 64 bit IBM RS/6000 or an Apple G5, choose 6xx.
> >  	  Note that the kernel runs in 32-bit mode even on 64-bit chips.
> >  
> > -config PPC_52xx
> > -	bool "Freescale 52xx"
> > -	select 6xx
> > -	select PPC_FPU
> > -	
> >  config PPC_82xx
> >  	bool "Freescale 82xx"
> >  	select 6xx

> Please _don't_ do it, since mpc5200 platform drivers depend from this
> option.

And that should be fixed. We are yet discussing the proper way of
integrating 52xx in arch/powerpc, but that definitely doesn't involve
making it an exclusive processor type. Besides, there will probably be
enough changes in the drivers to adapt them to of_platform_device type
probing that we might just well fork them all anyway.

Ben.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [POWERPC] MPC52xx is a 6xx variant, remove PPC_52xx config option
  2006-10-27 10:24   ` Benjamin Herrenschmidt
@ 2006-10-27 13:21     ` Grant Likely
  0 siblings, 0 replies; 4+ messages in thread
From: Grant Likely @ 2006-10-27 13:21 UTC (permalink / raw)
  To: Benjamin Herrenschmidt; +Cc: linuxppc-embedded

On 10/27/06, Benjamin Herrenschmidt <benh@kernel.crashing.org> wrote:
> On Fri, 2006-10-27 at 13:56 +0400, Andrey Volkov wrote:
> > Please _don't_ do it, since mpc5200 platform drivers depend from this
> > option.
>
> And that should be fixed. We are yet discussing the proper way of
> integrating 52xx in arch/powerpc, but that definitely doesn't involve
> making it an exclusive processor type. Besides, there will probably be
> enough changes in the drivers to adapt them to of_platform_device type
> probing that we might just well fork them all anyway.

Besides; this is only arch/powerpc.  52xx doesn't event boot on
arch/powerpc yet.

g.

-- 
Grant Likely, B.Sc. P.Eng.
Secret Lab Technologies Ltd.
grant.likely@secretlab.ca
(403) 399-0195

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2006-10-27 13:21 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-10-26  3:52 [POWERPC] MPC52xx is a 6xx variant, remove PPC_52xx config option Grant Likely
2006-10-27  9:56 ` Andrey Volkov
2006-10-27 10:24   ` Benjamin Herrenschmidt
2006-10-27 13:21     ` Grant Likely

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).