From: Greg KH <greg@kroah.com>
To: Stefan Klug <stefan.klug@baslerweb.com>
Cc: linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org,
stern@rowland.harvard.edu
Subject: Re: [PATCH][RFC] USB: zerocopy support for usbfs
Date: Wed, 2 Jul 2014 10:55:26 -0700 [thread overview]
Message-ID: <20140702175526.GA21063@kroah.com> (raw)
In-Reply-To: <53B42B14.7010303@baslerweb.com>
On Wed, Jul 02, 2014 at 05:53:56PM +0200, Stefan Klug wrote:
> Hi everybody,
> in short: The attached patch adds zerocopy support to the usbfs driver.
> I tested it on x86 and on a globalscale mirabox ARM board. Until now it
> works
> quite nice and I'd love to get some comments and feedback on the patch.
Very nice.
Minor non-technical issues here, your patch is corrupted (linewrapped
and tabs converted to spaces) and can't be applied at all. Can you fix
your email client and try again? Look at
Documentation/email_clients.txt for hints on how to do this.
> --- a/drivers/usb/core/devio.c
> +++ b/drivers/usb/core/devio.c
> @@ -4,6 +4,7 @@
> * devio.c -- User space communication with USB devices.
> *
> * Copyright (C) 1999-2000 Thomas Sailer (sailer@ife.ee.ethz.ch)
> + * Copyright (C) 2014 Stefan Klug (stefan.klug@baslerweb.com)
According to legal advice given to me, this isn't needed / true, so can
you drop it? Or, if you have different legal advice, might I ask for
what the legal justification is that this needs to be added?
> * This program is free software; you can redistribute it and/or
> modify
> * it under the terms of the GNU General Public License as published
> by
> @@ -30,6 +31,7 @@
> * 04.01.2000 0.2 Turned into its own filesystem
> * 30.09.2005 0.3 Fix user-triggerable oops in async URB delivery
> * (CAN-2005-3055)
> + * 07.05.2014 0.4 Added zerocopy support
No need to do this, we use git now and that's where the changelog goes.
We should just delete this "fake changelog" entirely from this file.
thanks,
greg k-h
next prev parent reply other threads:[~2014-07-02 17:55 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-07-02 15:53 [PATCH][RFC] USB: zerocopy support for usbfs Stefan Klug
2014-07-02 17:55 ` Greg KH [this message]
2014-07-03 7:06 ` Stefan Klug
2014-07-02 18:24 ` Alan Stern
2014-07-02 18:49 ` Peter Stuge
2014-07-02 19:31 ` Alan Stern
2014-07-02 19:42 ` Peter Stuge
2014-07-02 20:40 ` Alan Stern
2014-07-03 7:48 ` Stefan Klug
2014-07-03 8:40 ` David Laight
2014-07-03 14:15 ` Alan Stern
2014-07-02 18:58 ` Oliver Neukum
2014-07-02 19:38 ` Alan Stern
2014-07-03 8:22 ` Stefan Klug
2014-07-04 8:55 ` Oliver Neukum
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=20140702175526.GA21063@kroah.com \
--to=greg@kroah.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=stefan.klug@baslerweb.com \
--cc=stern@rowland.harvard.edu \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.