From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ben Warren Date: Thu, 24 Jan 2008 12:28:38 -0500 Subject: [U-Boot-Users] [PATCH] [DRIVERS] is only available for ppc boards. In-Reply-To: <2acbd3e40801240850k6d7a5714n61479666d7cf9afe@mail.gmail.com> References: <000201c85ea6$cf35c9d0$9a4d010a@Emea.Arm.com> <2acbd3e40801240850k6d7a5714n61479666d7cf9afe@mail.gmail.com> Message-ID: <4798CAC6.6090308@gmail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Andy Fleming wrote: > On Jan 24, 2008 10:33 AM, Peter Pearse wrote: > >> +#if defined(CONFIG_MPC834X) || \ >> + defined(CONFIG_MPC8313) || \ >> + defined(CONFIG_MPC8315) || \ >> + defined(CONFIG_MPC837X) >> #include >> +#endif >> >> #ifdef CONFIG_HARD_SPI >> --- >> >> or should asm-ppc/mpc8xxx_spi.h be moved to drivers/spi? >> > > Hm. I'd prefer that, since that SPI driver will possibly propagate > through *many* variants, and it seems silly to change the driver every > time we make a new part. :) > > I'm not familiar enough with the device or driver to know whether the > header is truly ppc-specific, or just coincidentally so. > > Andy > The header is specific to this SPI controller, but not really PPC-specific. I don't know enough about how Freescale works to know if the different divisions such as PPC and Coldfire and whatever else share building blocks like these controllers, but conceivably (in my mind at least) this could be on anything Freescale makes. I'm no expert on Kconfig, but I think that once it's in play the Makefile for this driver determines whether to even compile the driver depending on whether a CONFIG is set or not. I probably forgot to define something like CONFIG_MPC8XXX_SPI. Maybe a change to the Makefile is needed here? Sorry for the seemingly random mumblings. regards, Ben