linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
* RE: [HACK] add sandpoint + flattened dt support to arch/powerpc/b oot
@ 2006-05-18 10:24 Zang Roy-r61911
  2006-05-18 10:36 ` Benjamin Herrenschmidt
  0 siblings, 1 reply; 3+ messages in thread
From: Zang Roy-r61911 @ 2006-05-18 10:24 UTC (permalink / raw)
  To: Benjamin Herrenschmidt, Mark A. Greer; +Cc: linuxppc-dev


> > diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
> > index 6729c98..de09eac 100644
> > --- a/arch/powerpc/Kconfig
> > +++ b/arch/powerpc/Kconfig
> > @@ -323,7 +323,10 @@ config PPC_ISERIES
> >  
> >  config EMBEDDED6xx
> >  	bool "Embedded 6xx/7xx/7xxx-based board"
> > -	depends on PPC32 && BROKEN
> > +	depends on PPC32
> > +	select PPC_UDBG_16550
> > +	select MPIC
> > +	select MPIC_SERIAL
> 
> Not totally related to your patch but I'm tempted to turn that into a
> "Generic 6xx/7xx/7xxx" rather than "embedded" if we manage to always
> avoid board specific code most of the time, but instead add necessary
> bits in the device-tree. We still need a per-board Kconfig option I
> think that will just select the necessary bits and pieces 
> (and more than
> one can be selected at one time). Also, I think right now, 
> the embedded
> stuff is +/- exclusive from the MULTIPLATFORM stuff, that 
> must be fixed
> asap. We are all living in the same kernel now :)

config Embedded6xx is not accurate. It can not cover all the 
board with 7xx/7xxx based processor.

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

* RE: [HACK] add sandpoint + flattened dt support to arch/powerpc/b oot
  2006-05-18 10:24 [HACK] add sandpoint + flattened dt support to arch/powerpc/b oot Zang Roy-r61911
@ 2006-05-18 10:36 ` Benjamin Herrenschmidt
  0 siblings, 0 replies; 3+ messages in thread
From: Benjamin Herrenschmidt @ 2006-05-18 10:36 UTC (permalink / raw)
  To: Zang Roy-r61911; +Cc: linuxppc-dev

On Thu, 2006-05-18 at 18:24 +0800, Zang Roy-r61911 wrote:
> > > diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
> > > index 6729c98..de09eac 100644
> > > --- a/arch/powerpc/Kconfig
> > > +++ b/arch/powerpc/Kconfig
> > > @@ -323,7 +323,10 @@ config PPC_ISERIES
> > >  
> > >  config EMBEDDED6xx
> > >  	bool "Embedded 6xx/7xx/7xxx-based board"
> > > -	depends on PPC32 && BROKEN
> > > +	depends on PPC32
> > > +	select PPC_UDBG_16550
> > > +	select MPIC
> > > +	select MPIC_SERIAL
> > 
> > Not totally related to your patch but I'm tempted to turn that into a
> > "Generic 6xx/7xx/7xxx" rather than "embedded" if we manage to always
> > avoid board specific code most of the time, but instead add necessary
> > bits in the device-tree. We still need a per-board Kconfig option I
> > think that will just select the necessary bits and pieces 
> > (and more than
> > one can be selected at one time). Also, I think right now, 
> > the embedded
> > stuff is +/- exclusive from the MULTIPLATFORM stuff, that 
> > must be fixed
> > asap. We are all living in the same kernel now :)
> 
> config Embedded6xx is not accurate. It can not cover all the 
> board with 7xx/7xxx based processor.

Forget about CONFIG_EMBEDDED6xx, just do your own CONFIG_BOARDNAME for
now, the processor is irrelevant as long as it's a 6xx/7xx/7xxx
derivative.

Ben.

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

* RE: [HACK] add sandpoint + flattened dt support to arch/powerpc/b oot
@ 2006-05-18 10:47 Zang Roy-r61911
  0 siblings, 0 replies; 3+ messages in thread
From: Zang Roy-r61911 @ 2006-05-18 10:47 UTC (permalink / raw)
  To: Benjamin Herrenschmidt; +Cc: linuxppc-dev

 
> On Thu, 2006-05-18 at 18:24 +0800, Zang Roy-r61911 wrote:
> > > > diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
> > > > index 6729c98..de09eac 100644
> > > > --- a/arch/powerpc/Kconfig
> > > > +++ b/arch/powerpc/Kconfig
> > > > @@ -323,7 +323,10 @@ config PPC_ISERIES
> > > >  
> > > >  config EMBEDDED6xx
> > > >  	bool "Embedded 6xx/7xx/7xxx-based board"
> > > > -	depends on PPC32 && BROKEN
> > > > +	depends on PPC32
> > > > +	select PPC_UDBG_16550
> > > > +	select MPIC
> > > > +	select MPIC_SERIAL
> > > 
> > > Not totally related to your patch but I'm tempted to turn 
> that into a
> > > "Generic 6xx/7xx/7xxx" rather than "embedded" if we 
> manage to always
> > > avoid board specific code most of the time, but instead 
> add necessary
> > > bits in the device-tree. We still need a per-board 
> Kconfig option I
> > > think that will just select the necessary bits and pieces 
> > > (and more than
> > > one can be selected at one time). Also, I think right now, 
> > > the embedded
> > > stuff is +/- exclusive from the MULTIPLATFORM stuff, that 
> > > must be fixed
> > > asap. We are all living in the same kernel now :)
> > 
> > config Embedded6xx is not accurate. It can not cover all the 
> > board with 7xx/7xxx based processor.
> 
> Forget about CONFIG_EMBEDDED6xx, just do your own CONFIG_BOARDNAME for
> now, the processor is irrelevant as long as it's a 6xx/7xx/7xxx
> derivative.
> 
> Ben.
> 
> 
I can just do as your suggestion!

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

end of thread, other threads:[~2006-05-18 10:47 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-05-18 10:24 [HACK] add sandpoint + flattened dt support to arch/powerpc/b oot Zang Roy-r61911
2006-05-18 10:36 ` Benjamin Herrenschmidt
  -- strict thread matches above, loose matches on Subject: below --
2006-05-18 10:47 Zang Roy-r61911

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).