Re: [RFC PATCH 06/19] powerpc: gamecube/wii: introduce GAMECUBE_COMMON
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 ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [RFC PATCH 06/19] powerpc: gamecube/wii: introduce GAMECUBE_COMMON
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 ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [RFC PATCH 06/19] powerpc: gamecube/wii: introduce GAMECUBE_COMMON
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 ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [RFC PATCH 06/19] powerpc: gamecube/wii: introduce GAMECUBE_COMMON
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. ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [RFC PATCH 06/19] powerpc: gamecube/wii: introduce GAMECUBE_COMMON
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 ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [RFC PATCH 06/19] powerpc: gamecube/wii: introduce GAMECUBE_COMMON
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 ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [RFC PATCH 06/19] powerpc: gamecube/wii: introduce GAMECUBE_COMMON
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 ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [RFC PATCH 06/19] powerpc: gamecube/wii: introduce GAMECUBE_COMMON
Grant Likely wrote: On Sun, Nov 22, 2009 at 3:01 PM, Albert Herranz albert_herr...@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_herr...@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 ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [RFC PATCH 06/19] powerpc: gamecube/wii: introduce GAMECUBE_COMMON
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 ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
[RFC PATCH 06/19] powerpc: gamecube/wii: introduce GAMECUBE_COMMON
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_herr...@yahoo.es --- 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
Re: [RFC PATCH 06/19] powerpc: gamecube/wii: introduce GAMECUBE_COMMON
On Sun, Nov 22, 2009 at 3:01 PM, Albert Herranz albert_herr...@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_herr...@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, B.Sc., P.Eng. Secret Lab Technologies Ltd. ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev