RFC: arch/powerpc Kconfig cleanup

[Kumar Gala]

On Mar 15, 2007, at 7:34 PM, Arnd Bergmann wrote:

> On Friday 16 March 2007, Kumar Gala wrote:
>> +
>> +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