From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Grant Likely" Subject: Re: [PATCH] Simple driver for Xilinx SPI controler. Date: Wed, 6 Jun 2007 13:00:38 -0600 Message-ID: References: <4666F598.2000004@ru.mvista.com> <20070606180618.C4D861268059@mail88-blu.bigfish.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: spi-devel-general@lists.sourceforge.net, Andrei Konovalov , Wolfgang Reissnegger , linuxppc-embedded@ozlabs.org To: "Stephen Neuendorffer" Return-path: In-Reply-To: <20070606180618.C4D861268059@mail88-blu.bigfish.com> Content-Disposition: inline List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linuxppc-embedded-bounces+glppe-linuxppc-embedded-2=m.gmane.org@ozlabs.org Errors-To: linuxppc-embedded-bounces+glppe-linuxppc-embedded-2=m.gmane.org@ozlabs.org List-Id: linux-spi.vger.kernel.org On 6/6/07, Stephen Neuendorffer wrote: > > Yes, except that microblazes systems run in non-virtex FPGAs... > > It may be that XILINX_EDK is not the right thing either, but it > seemed like the easiest way to have a superset of powerpc and microblaze > systems > > What I'm trying to do is (in concert with Grant's recent configuration > changes) > having an easy way for a new board configuration to select the Kconfig > options for > all of the Xilinx drivers. Namely: > > config XILINX_ML300 > bool "Xilinx-ML300" > select XILINX_VIRTEX_II_PRO > select EMBEDDEDBOOT > help > This option enables support for the Xilinx ML300 evaluation > board. > > config XILINX_VIRTEX_II_PRO > bool > select XILINX_VIRTEX > > config XILINX_EDK > bool > depends on XILINX_VIRTEX || XILINX_MICROBLAZE > default y I don't have any problem with adding XILINX_EDK (or whatever), but think particular layout is back-assward. Rather than XILINX_EDK depending on XILINX_VIRTEX/MICROBLAZE, XILINX_VIRTEX/MICROBLAZE should 'select' XILINX_EDK and you need to drop the 'default y' line. Otherwise the XILINX_EDK option shows up in non-edk config files. > > config XILINX_GPIO > tristate "Xilinx OPB GPIO Support" > depends on XILINX_EDK > help > This option enables support for Xilinx GPIO. > > > It seems like I'm using XILINX_EDK to mean something different than you > are. > Perhaps it sould be instead: > > config XILINX_DRIVERS > bool > depends on XILINX_VIRTEX || XILINX_MICROBLAZE > default y Hmmm, yes. XILINX_DRIVERS is probably better. > > config XILINX_GPIO_EDK > tristate "Xilinx EDK-based OPB GPIO Support" > depends on XILINX_DRIVERS > select XILINX_EDK > help > This option enables support for Xilinx GPIO using EDK OS > independent driver. Why still have XILINX_EDK? I thought you meant replace XILINX_EDK with XILINX_DRIVERS. Cheers, g. -- Grant Likely, B.Sc., P.Eng. Secret Lab Technologies Ltd. grant.likely@secretlab.ca (403) 399-0195