Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932325AbWEAXuY (ORCPT ); Mon, 1 May 2006 19:50:24 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S932326AbWEAXuY (ORCPT ); Mon, 1 May 2006 19:50:24 -0400 Received: from moutng.kundenserver.de ([212.227.126.188]:45807 "EHLO moutng.kundenserver.de") by vger.kernel.org with ESMTP id S932325AbWEAXuX (ORCPT ); Mon, 1 May 2006 19:50:23 -0400 From: Arnd Bergmann To: cbe-oss-dev@ozlabs.org Subject: Re: [Cbe-oss-dev] [PATCH 11/13] cell: split out board specific files Date: Tue, 2 May 2006 01:50:13 +0200 User-Agent: KMail/1.9.1 Cc: Geoff Levand , Segher Boessenkool , linux-kernel@vger.kernel.org, linuxppc-dev@ozlabs.org References: <20060429232812.825714000@localhost.localdomain> <445690F7.5050605@am.sony.com> In-Reply-To: <445690F7.5050605@am.sony.com> MIME-Version: 1.0 Content-Disposition: inline Message-Id: <200605020150.14152.arnd@arndb.de> Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit X-Provags-ID: kundenserver.de abuse@kundenserver.de login:c48f057754fc1b1a557605ab9fa6da41 Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 5335 Lines: 141 On Tuesday 02 May 2006 00:51, Geoff Levand wrote: > Seems CELL_IIC is never used. Must be some stale variable, > so I removed it. Arnd, could you ack this. Yes, I used it before when there was more code shared between Cell and pSeries, but that is no longer the case. > Segher, a problem with your suggestion is that our > makefiles don't have as rich a set of logical ops as the > config files. Its easy to express 'build A if B', but not > so easy to do 'build A if not C'. To make this work > cleanly I made PPC_CELL denote !SOME_HYPERVISOR_THING, > so I can have constructions like this in the makefile: > > obj-$(CONFIG_PPC_CELL) += ... > > I also got rid of SPUFS_PRIV1_MMIO, since SPUFS_PRIV1_MMIO > just meant spufs with !SOME_HYPERVISOR_THING. > I guess that one should really be (SPU_FS && CELL_NATIVE), using the option Segher suggested now. > =================================================================== > --- cell--alp--2.orig/arch/powerpc/Kconfig 2006-05-01 15:13:22.000000000 -0700 > +++ cell--alp--2/arch/powerpc/Kconfig 2006-05-01 15:16:38.000000000 -0700 > @@ -391,15 +391,20 @@ > For more informations, refer to > > config PPC_CELL > - bool " Cell Broadband Processor Architecture" > + bool > + default n > + > +config PPC_IBM_CELL_BLADE > + bool " IBM Cell Blade" > depends on PPC_MULTIPLATFORM && PPC64 > + select PPC_CELL > select PPC_RTAS > select MMIO_NVRAM > select PPC_UDBG_16550 > > config PPC_SYSTEMSIM > bool " IBM Full System Simulator (systemsim) support" > - depends on PPC_CELL || PPC_PSERIES || PPC_MAPLE > + depends on PPC_IBM_CELL_BLADE || PPC_PSERIES || PPC_MAPLE > help > Support booting resulting image under IBMs Full System Simulator. > If you enable this option, you are able to select device whoops, this one should not be there at all. Note that I updated your previous patch as well to fit into the series for submission, and that did not include systemsim. > Index: cell--alp--2/arch/powerpc/platforms/cell/Kconfig > =================================================================== > --- cell--alp--2.orig/arch/powerpc/platforms/cell/Kconfig 2006-05-01 15:13:22.000000000 -0700 > +++ cell--alp--2/arch/powerpc/platforms/cell/Kconfig 2006-05-01 15:13:23.000000000 -0700 > @@ -11,10 +11,15 @@ > or may crash other CPUs. > Say 'n' here unless you expect to run on DD2.0 only. > > +config SPU_BASE > + bool > + default n > + > config SPU_FS > tristate "SPU file system" > default m > depends on PPC_CELL > + select SPU_BASE > help > The SPU file system is used to access Synergistic Processing > Units on machines implementing the Broadband Processor > Index: cell--alp--2/arch/powerpc/platforms/cell/Makefile > =================================================================== > --- cell--alp--2.orig/arch/powerpc/platforms/cell/Makefile 2006-05-01 15:13:22.000000000 -0700 > +++ cell--alp--2/arch/powerpc/platforms/cell/Makefile 2006-05-01 15:17:58.000000000 -0700 > @@ -1,14 +1,14 @@ > -obj-y += interrupt.o iommu.o setup.o spider-pic.o > -obj-y += pervasive.o pci.o > - > -obj-$(CONFIG_SMP) += smp.o > +obj-$(CONFIG_PPC_CELL) += interrupt.o iommu.o setup.o \ > + spider-pic.o pervasive.o pci.o > +ifeq ($(CONFIG_SMP),y) > +obj-$(CONFIG_PPC_CELL) += smp.o > +endif > > # needed only when building loadable spufs.ko > -spufs-modular-$(CONFIG_SPU_FS) += spu_syscalls.o > -obj-y += $(spufs-modular-m) > - > -# always needed in kernel > -spufs-builtin-$(CONFIG_SPU_FS) += spu_callbacks.o spu_base.o spu_priv1.o > -obj-y += $(spufs-builtin-y) $(spufs-builtin-m) > - > -obj-$(CONFIG_SPU_FS) += spufs/ > +spufs-modular-$(CONFIG_SPU_FS) += spu_syscalls.o > +obj-$(CONFIG_SPU_BASE) += spu_callbacks.o spu_base.o \ > + $(spufs-modular-m) > +ifdef CONFIG_SPU_FS > +obj-$(CONFIG_PPC_CELL) += spu_priv1_mmio.o > +endif I guess this could then become something like spu-priv1-$(CONFIG_PPC_CELL_NATIVE) += spu_priv1_mmio.o spufs-modular-$(CONFIG_SPU_FS) += spu_syscalls.o obj-$(CONFIG_SPU_BASE) += spu_callbacks.o spu_base.o \ $(spufs-modular-m) \ $(spu-priv1-y) > Index: cell--alp--2/drivers/net/Kconfig > =================================================================== > --- cell--alp--2.orig/drivers/net/Kconfig 2006-05-01 15:13:22.000000000 -0700 > +++ cell--alp--2/drivers/net/Kconfig 2006-05-01 15:13:23.000000000 -0700 > @@ -2179,7 +2179,7 @@ > > config SPIDER_NET > tristate "Spider Gigabit Ethernet driver" > - depends on PCI && PPC_CELL > + depends on PCI && PPC_IBM_CELL_BLADE > select FW_LOADER > help > This driver supports the Gigabit Ethernet chips present on the Hmm, I'm also no longer sure if this is right. In theory, spidernet could be used in all sorts of products, wether they are using the same bridge chip or just the gigabit ethernet macro from it. For now, I guess you can just leave this one alone if you respin the patch another time. It's disabled by default, so the dependency can be updated the next time we get a user in _addition_ to PPC_CELL. Arnd <>< - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/