From: David Brown <davidb@codeaurora.org>
To: Greg KH <gregkh@suse.de>
Cc: linux-kernel@vger.kernel.org, Dima Zavin <dima@android.com>,
Daniel Walker <dwalker@fifo99.com>,
Kenneth Heitke <kheitke@codeaurora.org>,
linux-arm-msm@vger.kernel.org, tsoni@codeaurora.org,
linux-arm-kernel@lists.infradead.org, swetland@google.com
Subject: Re: [PATCH v2 2/2] msm: add single-wire serial bus interface (SSBI) driver
Date: Thu, 24 Feb 2011 20:06:11 -0800 [thread overview]
Message-ID: <8yar5awbvws.fsf@huya.qualcomm.com> (raw)
In-Reply-To: <20110225035857.GA9985@suse.de> (Greg KH's message of "Thu, 24 Feb 2011 19:58:57 -0800")
On Thu, Feb 24 2011, Greg KH wrote:
> On Thu, Feb 24, 2011 at 07:54:21PM -0800, David Brown wrote:
>> On Thu, Feb 24 2011, Greg KH wrote:
>>
>> > drivers/ssbi? What's keeping this from later moving off of the msm
>> > chips to run on others? USB started out only on one processor, as did a
>> > lot of other bus-specific drivers, before the hardware became present on
>> > other architectures. So no need to bury it under a msm specific
>> > location.
>>
>> Not sure if anything is stopping it from moving off of MSM chips, but it
>> doesn't seem very likely. There are too many similar interfaces that
>> are standard that others would be likely to use. For the most part, it
>> is the bus to communicate between the MSM and it's specialized
>> peripherals.
>>
>> We can still put it there, but it will likely be the only driver ever in
>> the directory.
>
> Ah, so it's just one driver? Then how about drivers/platform/msm/ ?
I like it. It keeps drivers from getting cluttered. Plus x86 is
getting lonely in there.
Ken, in addition to addressing the comments made so far and any others
that come in, let's put this under drivers/platform/msm.
Thanks,
David
--
Sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.
next prev parent reply other threads:[~2011-02-25 4:06 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1298585954-22422-1-git-send-email-kheitke@codeaurora.org>
2011-02-24 22:19 ` [PATCH v2 1/2] MAINTAINERS: Add drivers/msm to MSM subsystem Kenneth Heitke
2011-02-24 23:08 ` Nicolas Pitre
2011-02-24 22:19 ` [PATCH v2 2/2] msm: add single-wire serial bus interface (SSBI) driver Kenneth Heitke
2011-02-25 2:23 ` Daniel Walker
2011-02-25 2:31 ` David Brown
2011-02-25 2:35 ` Daniel Walker
2011-02-25 2:39 ` Dima Zavin
2011-02-25 3:20 ` David Brown
2011-02-25 3:28 ` Greg KH
2011-02-25 3:54 ` David Brown
2011-02-25 3:58 ` Greg KH
2011-02-25 4:06 ` David Brown [this message]
2011-02-25 19:00 ` David Brown
2011-02-25 20:21 ` Brian Swetland
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=8yar5awbvws.fsf@huya.qualcomm.com \
--to=davidb@codeaurora.org \
--cc=dima@android.com \
--cc=dwalker@fifo99.com \
--cc=gregkh@suse.de \
--cc=kheitke@codeaurora.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=swetland@google.com \
--cc=tsoni@codeaurora.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox