Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756534Ab0H3SXz (ORCPT ); Mon, 30 Aug 2010 14:23:55 -0400 Received: from mail-gy0-f174.google.com ([209.85.160.174]:51212 "EHLO mail-gy0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755592Ab0H3SXy convert rfc822-to-8bit (ORCPT ); Mon, 30 Aug 2010 14:23:54 -0400 MIME-Version: 1.0 In-Reply-To: <001401cb4814$fb4bdd70$66f8800a@maildom.okisemi.com> References: <4C77BD91.9070302@dsn.okisemi.com> <001401cb4814$fb4bdd70$66f8800a@maildom.okisemi.com> From: Grant Likely Date: Mon, 30 Aug 2010 12:23:33 -0600 X-Google-Sender-Auth: sQRasxekQ39hROhqU8WdWqacTBU Message-ID: Subject: Re: [MeeGo-Dev][PATCH] Topcliff: Update PCH_SPI driver to 2.6.35 To: Masayuki Ohtake Cc: Thomas Gleixner , meego-dev@meego.com, LKML , David Brownell , qi.wang@intel.com, yong.y.wang@intel.com, andrew.chih.howe.khor@intel.com, arjan@linux.intel.com, gregkh@suse.de, Tomoya MORINAGA , David Woodhouse , Alan Cox Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3405 Lines: 69 On Mon, Aug 30, 2010 at 1:28 AM, Masayuki Ohtake wrote: > ----- Original Message ----- > From: "Grant Likely" > To: "Thomas Gleixner" > Cc: "Masayuki Ohtak" ; ; "LKML" ; "David > Brownell" ; ; ; > ; ; ; "Tomoya MORINAGA" > ; "David Woodhouse" ; "Alan Cox" > Sent: Saturday, August 28, 2010 4:20 AM > Subject: Re: [MeeGo-Dev][PATCH] Topcliff: Update PCH_SPI driver to 2.6.35 > > >> On Fri, Aug 27, 2010 at 12:53 PM, Thomas Gleixner wrote: >> > B1;2401;0cOn Fri, 27 Aug 2010, Grant Likely wrote: >> >> [cc'ing Thomas Gleixner and David Woodhouse since this driver needs to >> >> get some data about the platform (to know what spi_devices are >> >> present) and I don't know how that is handled for x86 SoCs.] >> > >> > The best way to do all this platform specific configuration is to use >> > device tree. I really don't want to have x86/mach-xyz/board[A-Z] >> > horror, which is unavoidable when we don't get a sensible way to >> > configure the boards. >> >> I knew you were going to say that! ?:-) >> >> Ohtak-san, for this patch I'd like you to drop the separate driver >> that only registers the spi_devices and just submit the core driver. > > Sorry, I can't follow your discussion by lack of SPI knowledge. > Which the above mean that "spi_register_board_info" ?moves to our spi_pch or > delete for our driver ? I mean remove drivers/spi/spi_pch_device.c and the related Kconfig bits from this patch. The spi_device registration is a separate task which should be submitted in a separate patch, and would be better handled in either platform support code or with a device tree. >> (You can of course keep the spi_device registration in your own tree >> for debug purposes). ?I'll expect that the device will get >> instantiated using a device tree to determine which spi_devices are >> present. ?The parsing of spi device tree data will be moving into the >> core spi subsystem code in the next merge window most likely, so it >> can all be handled transparently. >> >> > SFI was meant to provide a lightweight ACPI, but >> > now that device tree is generic and more platforms are using it, I >> > really want to standartize on that and forget SFI. >> > >> > That makes even more sense, as all these AMBA peripherals which are >> > duct-taped to a x86 core can be found in other SoCs with different >> > cores as well. >> >> Indeed. ?BTW, Ohtak-san, is this spi bus device something brand new, >> or is it derived from an existing spi device? > > Yes, Intel Topcliff is new concept device. As Thomas already commented, what I'm really interested in is whether or not this spi controller is based on an older SPI controller. If it is, then it may be possible to add support for this chip to an existing driver instead of writing a whole new one. g. -- 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/