| Message ID | 1258927311-4340-7-git-send-email-albert_herranz@yahoo.es (mailing list archive) |
|---|---|
| State | Changes Requested |
| Headers | show |
On Sun, Nov 22, 2009 at 3:01 PM, Albert Herranz <albert_herranz@yahoo.es> wrote: > Add a config option GAMECUBE_COMMON to be used as a dependency for all > options common to the Nintendo GameCube and Wii video game consoles. > > Signed-off-by: Albert Herranz <albert_herranz@yahoo.es> This could just be rolled into the first patch that uses it. But otherwise; ACK. g. > --- > arch/powerpc/platforms/embedded6xx/Kconfig | 4 ++++ > 1 files changed, 4 insertions(+), 0 deletions(-) > > diff --git a/arch/powerpc/platforms/embedded6xx/Kconfig b/arch/powerpc/platforms/embedded6xx/Kconfig > index 291ac9d..97a2dbc 100644 > --- a/arch/powerpc/platforms/embedded6xx/Kconfig > +++ b/arch/powerpc/platforms/embedded6xx/Kconfig > @@ -90,3 +90,7 @@ config MPC10X_OPENPIC > config MPC10X_STORE_GATHERING > bool "Enable MPC10x store gathering" > depends on MPC10X_BRIDGE > + > +config GAMECUBE_COMMON > + bool > + > -- > 1.6.3.3 > > _______________________________________________ > Linuxppc-dev mailing list > Linuxppc-dev@lists.ozlabs.org > https://lists.ozlabs.org/listinfo/linuxppc-dev >
Grant Likely wrote: > On Sun, Nov 22, 2009 at 3:01 PM, Albert Herranz <albert_herranz@yahoo.es> wrote: >> Add a config option GAMECUBE_COMMON to be used as a dependency for all >> options common to the Nintendo GameCube and Wii video game consoles. >> >> Signed-off-by: Albert Herranz <albert_herranz@yahoo.es> > > This could just be rolled into the first patch that uses it. But > otherwise; ACK. > > g. > Thanks. This one I prefer in a separate patch as a logical step, if that's not a problem. >> --- >> arch/powerpc/platforms/embedded6xx/Kconfig | 4 ++++ >> 1 files changed, 4 insertions(+), 0 deletions(-) >> >> diff --git a/arch/powerpc/platforms/embedded6xx/Kconfig b/arch/powerpc/platforms/embedded6xx/Kconfig >> index 291ac9d..97a2dbc 100644 >> --- a/arch/powerpc/platforms/embedded6xx/Kconfig >> +++ b/arch/powerpc/platforms/embedded6xx/Kconfig >> @@ -90,3 +90,7 @@ config MPC10X_OPENPIC >> config MPC10X_STORE_GATHERING >> bool "Enable MPC10x store gathering" >> depends on MPC10X_BRIDGE >> + >> +config GAMECUBE_COMMON >> + bool >> + Cheers, Albert
> Add a config option GAMECUBE_COMMON to be used as a dependency for all > options common to the Nintendo GameCube and Wii video game consoles. Maybe something like GAMECUBE_OR_WII instead? "COMMON" is so common it's meaningless. Segher
Segher Boessenkool wrote: >> Add a config option GAMECUBE_COMMON to be used as a dependency for all >> options common to the Nintendo GameCube and Wii video game consoles. > > Maybe something like GAMECUBE_OR_WII instead? "COMMON" is so > common it's meaningless. > I don't like either GAMECUBE_OR_WII. It looks as if it can be used to match options for the GAMECUBE _or_ WII. But that's not the meaning of it. This option should be used only for options applicable to both the GAMECUBE and WII, i.e. basically those options in the WII which where "inherited" from the GAMECUBE to make it retro-compatible. If GAMECUBE_COMMON is unacceptable, maybe GAMECUBE_LEGACY or GAMECUBE_COMPAT? Thanks, Albert
>>> Add a config option GAMECUBE_COMMON to be used as a dependency >>> for all >>> options common to the Nintendo GameCube and Wii video game consoles. >> >> Maybe something like GAMECUBE_OR_WII instead? "COMMON" is so >> common it's meaningless. >> > > I don't like either GAMECUBE_OR_WII. > It looks as if it can be used to match options for the GAMECUBE > _or_ WII. But that's not the meaning of it. > > This option should be used only for options applicable to both the > GAMECUBE and WII, i.e. basically those options in the WII which > where "inherited" from the GAMECUBE to make it retro-compatible. > > If GAMECUBE_COMMON is unacceptable, maybe GAMECUBE_LEGACY or > GAMECUBE_COMPAT? Maybe it's best to write GAMECUBE || WII in the places that use it, then? Segher
On Tuesday 24 November 2009, Segher Boessenkool wrote: > > This option should be used only for options applicable to both the > > GAMECUBE and WII, i.e. basically those options in the WII which > > where "inherited" from the GAMECUBE to make it retro-compatible. > > > > If GAMECUBE_COMMON is unacceptable, maybe GAMECUBE_LEGACY or > > GAMECUBE_COMPAT? > > Maybe it's best to write GAMECUBE || WII in the places that use it, > then? Or just use CONFIG_NINTENDO? If we ever want to port to a future Nintendo system that is not compatible, we can still change it. Arnd <><
Segher Boessenkool wrote: >> If GAMECUBE_COMMON is unacceptable, maybe GAMECUBE_LEGACY or >> GAMECUBE_COMPAT? > > Maybe it's best to write GAMECUBE || WII in the places that use it, then? > That gets messy in conditionals, like for example: config PCI bool "PCI support" if PPC_PCI_CHOICE default y if !40x && !CPM2 && !8xx && !PPC_83xx \ - && !PPC_85xx && !PPC_86xx + && !PPC_85xx && !PPC_86xx && !(GAMECUBE || WII) default PCI_PERMEDIA if !4xx && !CPM2 && !8xx default PCI_QSPAN if !4xx && !CPM2 && 8xx select ARCH_SUPPORTS_MSI I'd prefer to have a Kconfig option for that, if we can come up with an agreed name :) Thanks, Albert
On Tue, 2009-11-24 at 19:03 +0100, Albert Herranz wrote: > Segher Boessenkool wrote: > >> Add a config option GAMECUBE_COMMON to be used as a dependency for all > >> options common to the Nintendo GameCube and Wii video game consoles. > > > > Maybe something like GAMECUBE_OR_WII instead? "COMMON" is so > > common it's meaningless. > > > > I don't like either GAMECUBE_OR_WII. > It looks as if it can be used to match options for the GAMECUBE _or_ WII. But that's not the meaning of it. > > This option should be used only for options applicable to both the GAMECUBE and WII, i.e. basically those options in the WII which where "inherited" from the GAMECUBE to make it retro-compatible. > > If GAMECUBE_COMMON is unacceptable, maybe GAMECUBE_LEGACY or GAMECUBE_COMPAT? I don't have any strong opinion. Maybe NINTENDO_COMMON ? Cheers, Ben.
Benjamin Herrenschmidt wrote: > On Tue, 2009-11-24 at 19:03 +0100, Albert Herranz wrote: >> Segher Boessenkool wrote: >>>> Add a config option GAMECUBE_COMMON to be used as a dependency for all >>>> options common to the Nintendo GameCube and Wii video game consoles. >>> Maybe something like GAMECUBE_OR_WII instead? "COMMON" is so >>> common it's meaningless. >>> >> I don't like either GAMECUBE_OR_WII. >> It looks as if it can be used to match options for the GAMECUBE _or_ WII. But that's not the meaning of it. >> >> This option should be used only for options applicable to both the GAMECUBE and WII, i.e. basically those options in the WII which where "inherited" from the GAMECUBE to make it retro-compatible. >> >> If GAMECUBE_COMMON is unacceptable, maybe GAMECUBE_LEGACY or GAMECUBE_COMPAT? > > I don't have any strong opinion. Maybe NINTENDO_COMMON ? > We should avoid using NINTENDO just to refer to the Nintendo GameCube or Wii. They have other consoles too :) There's even a port of Linux to the Nintendo DS... I'd stick to GAMECUBE_COMMON or GAMECUBE_COMPAT. > Cheers, > Ben. > Thanks, Albert
>>>> Add a config option GAMECUBE_COMMON to be used as a dependency >>>> for all >>>> options common to the Nintendo GameCube and Wii video game >>>> consoles. >>> >>> Maybe something like GAMECUBE_OR_WII instead? "COMMON" is so >>> common it's meaningless. >> >> I don't like either GAMECUBE_OR_WII. >> It looks as if it can be used to match options for the GAMECUBE >> _or_ WII. But that's not the meaning of it. >> >> This option should be used only for options applicable to both the >> GAMECUBE and WII, i.e. basically those options in the WII which >> where "inherited" from the GAMECUBE to make it retro-compatible. >> >> If GAMECUBE_COMMON is unacceptable, maybe GAMECUBE_LEGACY or >> GAMECUBE_COMPAT? > > I don't have any strong opinion. Maybe NINTENDO_COMMON ? That is much worse than GAMECUBE_COMMON. Can you call the "common" option GAMECUBE, and have WII be for support for whatever is new on the Wii? Or is there some significant code that is only needed on the old gamecube? Segher
diff --git a/arch/powerpc/platforms/embedded6xx/Kconfig b/arch/powerpc/platforms/embedded6xx/Kconfig index 291ac9d..97a2dbc 100644 --- a/arch/powerpc/platforms/embedded6xx/Kconfig +++ b/arch/powerpc/platforms/embedded6xx/Kconfig @@ -90,3 +90,7 @@ config MPC10X_OPENPIC config MPC10X_STORE_GATHERING bool "Enable MPC10x store gathering" depends on MPC10X_BRIDGE + +config GAMECUBE_COMMON + bool +
Add a config option GAMECUBE_COMMON to be used as a dependency for all options common to the Nintendo GameCube and Wii video game consoles. Signed-off-by: Albert Herranz <albert_herranz@yahoo.es> --- arch/powerpc/platforms/embedded6xx/Kconfig | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-)