From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner+w=401wt.eu-S1760110AbYEXRds@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1760110AbYEXRds (ORCPT <rfc822;w@1wt.eu>);
	Sat, 24 May 2008 13:33:48 -0400
Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1755456AbYEXRdj
	(ORCPT <rfc822;linux-kernel-outgoing>);
	Sat, 24 May 2008 13:33:39 -0400
Received: from an-out-0708.google.com ([209.85.132.250]:52815 "EHLO
	an-out-0708.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1751617AbYEXRdi (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Sat, 24 May 2008 13:33:38 -0400
Message-ID: <fa686aa40805241033x128c30b0v826717cc879a712e@mail.gmail.com>
Date: Sat, 24 May 2008 11:33:35 -0600
From: "Grant Likely" <grant.likely@secretlab.ca>
To: "Jochen Friedrich" <jochen@scram.de>
Subject: Re: [PATCH 3/4] spi: Add OF binding support for SPI busses
Cc: "David Brownell" <david-b@pacbell.net>, fabrizio.garetto@gmail.com,
       linuxppc-dev@ozlabs.org, linux-kernel@vger.kernel.org,
       spi-devel-general@lists.sourceforge.net
In-Reply-To: <48384D13.6010608@scram.de>
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
References: <20080516193054.28030.35126.stgit@trillian.secretlab.ca>
	 <20080516193613.28030.13950.stgit@trillian.secretlab.ca>
	 <20080521151928.GA28857@polina.dev.rtsoft.ru>
	 <200805221905.32288.david-b@pacbell.net>
	 <fa686aa40805232326w35f455d1s274899160d47eccb@mail.gmail.com>
	 <fa686aa40805240950ocd16b97y308a54c68efa28ef@mail.gmail.com>
	 <48384D13.6010608@scram.de>
X-Google-Sender-Auth: 7d724dbec80b68aa
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Sat, May 24, 2008 at 11:14 AM, Jochen Friedrich <jochen@scram.de> wrote:
> Grant Likely schrieb:
>> On Sat, May 24, 2008 at 12:26 AM, Grant Likely
>> <grant.likely@secretlab.ca> wrote:
>>> On Thu, May 22, 2008 at 8:05 PM, David Brownell <david-b@pacbell.net> wrote:
>>>> On Wednesday 21 May 2008, Anton Vorontsov wrote:
>>>>>> +++ b/drivers/spi/spi_of.c
>>>>> I think better placement for this is drivers/of, no?
>>>> Yes please.
>>> Okay, I wasn't sure.  Will do.
>>
>> I'm having second thoughts about this.  I think this code is more SPI
>> centric than it is OF centric.  ie. it is usable by all spi masters in
>> an OF enabled system, but it is not usable by all OF devices in an SPI
>> enabled system.  Or, in other words; it adds OF support to SPI, not
>> the other way around.  I think drivers/spi is the right place for this
>> to live.
>
> Isn't the same true for drivers/of/gpio.c or drivers/of/of_i2c.c, as well?

I would argue 'yes!'

g.

-- 
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.