From: Kumar Gala <hidden> Date: 2007-03-15 23:37:49
arch/powerpc/Kconfig has become something of a mess and platform support
is scattered between it and various arch/powerpc/platforms/*/Kconfig
files. I've got some ideas on some simple clean up that will introduce a
arch/powerpc/platforms/Kconfig and pull out platform config information
from arch/powerpc/Kconfig.
Beyond general cleanup, there are several config options that should
exist under the "Platform support" menu but end up getting duplicated
across sub-architectures (QE, CPM2, etc.)
By introducing a platforms/Kconfig we will have a place to put those
config options so we don't have to duplicate them.
One think I noticed is that various platforms/*/Kconfig create top level
menu choices for things like:
Cell Broadband Engine options
PS3 Platform Options
PA Semi PWRficient options
iSeries device drivers
Machine Type (for embedded 6xx)
I didn't know if people were ok with having these move under the 'Platform
support' menu or would prefer they stay as top level menu choices. Here's
a work up of a patch for Cell that moves things around and puts the 'Cell
Broarband Engine options' under 'Platform support'.
- k
From: Grant Likely <hidden> Date: 2007-03-15 23:45:17
On 3/15/07, Kumar Gala [off-list ref] wrote:
arch/powerpc/Kconfig has become something of a mess and platform support
is scattered between it and various arch/powerpc/platforms/*/Kconfig
files. I've got some ideas on some simple clean up that will introduce a
arch/powerpc/platforms/Kconfig and pull out platform config information
from arch/powerpc/Kconfig.
Beyond general cleanup, there are several config options that should
exist under the "Platform support" menu but end up getting duplicated
across sub-architectures (QE, CPM2, etc.)
By introducing a platforms/Kconfig we will have a place to put those
config options so we don't have to duplicate them.
I'm all for this, go for it!
g.
--
Grant Likely, B.Sc. P.Eng.
Secret Lab Technologies Ltd.
grant.likely@secretlab.ca
(403) 399-0195
From: Olof Johansson <hidden> Date: 2007-03-16 00:05:33
On Thu, Mar 15, 2007 at 06:34:25PM -0500, Kumar Gala wrote:
I didn't know if people were ok with having these move under the 'Platform
support' menu or would prefer they stay as top level menu choices. Here's
a work up of a patch for Cell that moves things around and puts the 'Cell
Broarband Engine options' under 'Platform support'.
Looks good to me. I rarely notice menu organization myself since I tend
to edit .config by hand and run menuconfig. Go for it.
-Olof
I'm not sure if these should really be part of cell/Kconfig, as they are
about platform code that is in other directories. I think it would make
more sense conceptually to have them in ps3/Kconfig and celleb/Kconfig.
At some point we should also separate the common code (mostly spufs)
from the cell native support that does not apply to ps3 and celleb.
Arnd <><
From: Kumar Gala <hidden> Date: 2007-03-16 01:17:06
On Mar 15, 2007, at 7:34 PM, Arnd Bergmann wrote:
On Friday 16 March 2007, Kumar Gala wrote:
quoted
+
+config PPC_PS3
+ bool "Sony PS3 (incomplete)"
+ depends on PPC_MULTIPLATFORM && PPC64
+ select PPC_CELL
+ select USB_ARCH_HAS_OHCI
+ select USB_OHCI_LITTLE_ENDIAN
+ select USB_OHCI_BIG_ENDIAN_MMIO
+ select USB_ARCH_HAS_EHCI
+ select USB_EHCI_BIG_ENDIAN_MMIO
+ help
+ This option enables support for the Sony PS3 game console
+ and other platforms using the PS3 hypervisor.
+ Support for this platform is not yet complete, so
+ enabling this will not result in a bootable kernel on a
+ PS3 system.
+
+config PPC_CELLEB
+ bool "Toshiba's Cell Reference Set 'Celleb' Architecture"
+ depends on PPC_MULTIPLATFORM && PPC64
+ select PPC_CELL
+ select PPC_OF_PLATFORM_PCI
+ select HAS_TXX9_SERIAL
+ select PPC_UDBG_BEAT
+ select USB_OHCI_BIG_ENDIAN_MMIO
+ select USB_EHCI_BIG_ENDIAN_MMIO
I'm not sure if these should really be part of cell/Kconfig, as
they are
about platform code that is in other directories. I think it would
make
more sense conceptually to have them in ps3/Kconfig and celleb/
Kconfig.
At some point we should also separate the common code (mostly spufs)
from the cell native support that does not apply to ps3 and celleb.
Fair point, I'll fix that up. I'm more interested if people are ok
with having the 'Cell Broadband Engine options' under 'Platform
support' is ok or if it should stay at the top level.
- k
From: Li Yang-r58472 <hidden> Date: 2007-03-16 06:58:31
arch/powerpc/Kconfig has become something of a mess and platform
support
is scattered between it and various arch/powerpc/platforms/*/Kconfig
files. I've got some ideas on some simple clean up that will
introduce a
arch/powerpc/platforms/Kconfig and pull out platform config
information
from arch/powerpc/Kconfig.
=20
Beyond general cleanup, there are several config options that should
exist under the "Platform support" menu but end up getting duplicated
across sub-architectures (QE, CPM2, etc.)
=20
By introducing a platforms/Kconfig we will have a place to put those
config options so we don't have to duplicate them.
At some point we should also separate the common code (mostly spufs)
from the cell native support that does not apply to ps3 and celleb.
=20
Fair point, I'll fix that up. =A0I'm more interested if people are ok =A0
with having the 'Cell Broadband Engine options' under 'Platform =A0
support' is ok or if it should stay at the top level.
I was never happy with having it at the top level, but I couldn't
figure out a better way at the time, and I think I just copied what
some other platforms did.
I think it would be good to turn them into menuconfig entries, so
that if platforms/Kconfig basically contains
source arch/powerpc/platforms/pseries/Kconfig
source arch/powerpc/platforms/powermac/Kconfig
source arch/powerpc/platforms/cell/Kconfig
source arch/powerpc/platforms/celleb/Kconfig
source arch/powerpc/platforms/ps3/Kconfig
=2E..
each of them can either be
config CELLEB
bool "support for celleb"
select FOO
select BAR
or
menuconfig PSERIES
bool "support for pseries"
select BAR
select BAZ
config PSERIES_FOO
bool "option foo for pseries"
depends on PSERIES
endmenu
Arnd <><