From: Greg KH <gregkh@suse.de>
To: "Brian G. Merrell" <bgmerrell@novell.com>
Cc: greg@kroah.com, linux-usb@vger.kernel.org,
linux-kernel@vger.kernel.org,
Takahiro Hirofuchi <hirofuchi@users.sourceforge.net>
Subject: Re: [patch 00/03] USB-IP patches
Date: Tue, 2 Sep 2008 10:12:36 -0700 [thread overview]
Message-ID: <20080902171236.GC9524@suse.de> (raw)
In-Reply-To: <20080830055007.GB3422@novell.com>
On Fri, Aug 29, 2008 at 11:50:08PM -0600, Brian G. Merrell wrote:
> On Thu, Aug 28, 2008 at 04:00:16PM -0700, greg@kroah.com wrote:
> > Here's 3 patches that adds USB-IP functionality code to the kernel tree.
> >
> > It is against 2.6.27-rc4, and has been added to the -staging tree. If
> > there are no major objections, I'll also queue it up in the usb tree for
> > submission into 2.6.28.
> >
> > The code is based on Takahiro's great work that has been sitting outside
> > of the kernel for many many years now. Brian Merrell took the most
> > recent version of the code, cleaned it up and sent it to me. I did more
> > cleanup and split the patch into 3 pieces, making it a bit easier to
> > work with.
> >
> > There are still a number of checkpatch.pl warnings/errors emitted, and
> > Brian said he would work on them in the upcoming weeks, but they don't
> > distract from the basic functionality and a good review would be nice to
> > have.
>
> I will indeed continue to work on getting rid of the checkpatch.pl
> warnings/errors during the upcoming weeks. I already eliminated the
> ones I could figure out without too much effort; I am sure the
> remaining ones are trivial to many of you out there, so I welcome
> any suggestions.
The kthread one looks like you will have to change a bit of the code,
all of the other remaining ones should be self-explanatory. If not,
please let us know and we'll be glad to help out.
Can you also work to address all of the review comments made on the code
so far as well? I don't know how much time you have to work on this,
but for now I'll let you send me changes and not do anything on the code
myself. Sound good?
thanks,
greg k-h
next prev parent reply other threads:[~2008-09-02 17:17 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20080828225248.758800945@mini.kroah.org>
2008-08-28 23:00 ` [patch 00/03] USB-IP patches greg
2008-08-28 23:00 ` [patch 01/03] USB: USB/IP: add common functions needed greg
2008-08-29 7:29 ` Oliver Neukum
2008-08-29 7:52 ` Alexander E. Patrakov
2008-08-29 14:32 ` Greg KH
2008-08-30 0:14 ` Andrew Morton
2008-08-28 23:00 ` [patch 02/03] USB: USB/IP: add client driver greg
2008-08-29 8:23 ` Oliver Neukum
2008-08-29 16:02 ` Alan Stern
2008-08-28 23:00 ` [patch 03/03] USB: USB/IP: add host driver greg
2008-08-29 5:43 ` Willy Tarreau
2008-08-28 23:06 ` [patch 00/03] USB-IP patches Arnaldo Carvalho de Melo
2008-08-30 5:50 ` Brian G. Merrell
2008-09-02 17:12 ` Greg KH [this message]
2008-09-24 1:28 ` Brian G. Merrell
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=20080902171236.GC9524@suse.de \
--to=gregkh@suse.de \
--cc=bgmerrell@novell.com \
--cc=greg@kroah.com \
--cc=hirofuchi@users.sourceforge.net \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb@vger.kernel.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