mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Brian Swetland <swetland@google.com>
To: Pavel Machek <pavel@ucw.cz>
Cc: Greg KH <greg@kroah.com>,
	malchev@google.com, kernel list <linux-kernel@vger.kernel.org>,
	linux-arm-kernel <linux-arm-kernel@lists.arm.linux.org.uk>
Subject: Re: HTC Dream camera support for staging
Date: Wed, 8 Jul 2009 02:47:46 -0700	[thread overview]
Message-ID: <a55d774e0907080247x23a2f134p1ecdb079c158df5@mail.gmail.com> (raw)
In-Reply-To: <20090708084439.GC20020@elf.ucw.cz>

On Wed, Jul 8, 2009 at 1:44 AM, Pavel Machek<pavel@ucw.cz> wrote:
> On Thu 2009-07-02 13:08:24, Greg KH wrote:
>> On Mon, Jun 29, 2009 at 01:43:40PM +0200, Pavel Machek wrote:
>> >
>> > This patch adds driver for HTC Dream camera. I guess driver is
>> > slightly higher quality than usual for staging/ , but it is fairly big
>> > and I don't feel like doing all the cleanups myself. Also some parts
>> > can probably be removed, as they did not end up in shipping hardware..
>> >
>> > The driver is actually so big that I'll attach it gzipped, otherwise
>> > it would not pass maillist limits.
>> >
>> > Signed-off-by: Pavel Machek <pavel@ucw.cz>
>> > Cc: Brian Swetland <swetland@google.com>
>>
>> Can you resubmit it with the original author descriptions?  It's not
>> fair to them to have it shown up as coming from you.
>
> Yes, done... and thanks.
>
> Brian... I'd like to submit qdsp5 and smd code, too. Who should be
> listed as their authors?

smd is pretty much my fault at this point, though Arve hacks on the
PM-related bits.

The latest incarnation of the rpcrouter is me, with a lot of the
original work by Iliyan, based heavily on the work Qualcomm did (thus
their copyright on it as well as Google's).

qdsp5 is Iliyan.

Might be worth breaking these into at least three pieces (smd,
rpcrouter, qdsp5)?

Are you highly opposed to having smd and rpcrouter under
arch/arm/mach-msm?  They're (for better or worse) pretty integral to
the core operation of the platform (suspend/resume and a lot of other
core SoC functions require coordination with the modem core).

The qdsp5 stuff could exist elsewhere easily (though I'm not sure
where the logical place is), and stuff on top of it like the v4l2
camera stuff has obvious homes in the usual driver hierarchy.

Brian

  reply	other threads:[~2009-07-08  9:48 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-29 11:43 Pavel Machek
2009-06-29 18:28 ` Guennadi Liakhovetski
2009-06-29 18:52   ` Greg KH
2009-07-02 20:08 ` Greg KH
2009-07-08  8:44   ` Pavel Machek
2009-07-08  9:47     ` Brian Swetland [this message]
2009-07-09  9:53       ` Pavel Machek
2009-07-09 15:00         ` Greg KH
     [not found] ` <20090629121023.GD28046@google.com>
     [not found]   ` <20090630102457.GA25802@elf.ucw.cz>
     [not found]     ` <a55d774e0906300338l7743d566k5b463eed3efd1ea1@mail.gmail.com>
     [not found]       ` <20090709101315.GF10937@elf.ucw.cz>
     [not found]         ` <a55d774e0907090406u5e91b5c9r2303b098d0e44988@mail.gmail.com>
2009-07-13 20:27           ` Pavel Machek
2009-07-13 21:06             ` Pavel Machek
2009-07-13 22:05             ` Arve Hjønnevåg
2009-07-23 17:28               ` Pavel Machek
2009-07-08  8:40 Pavel Machek

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=a55d774e0907080247x23a2f134p1ecdb079c158df5@mail.gmail.com \
    --to=swetland@google.com \
    --cc=greg@kroah.com \
    --cc=linux-arm-kernel@lists.arm.linux.org.uk \
    --cc=linux-kernel@vger.kernel.org \
    --cc=malchev@google.com \
    --cc=pavel@ucw.cz \
    /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

all inboxes | Powered by JetHome®