From: Greg KH <gregkh@linuxfoundation.org>
To: Pavel Machek <pavel@ucw.cz>
Cc: cmroliv@gmail.com, marcel@holtmann.org, pali.rohar@gmail.com,
kernel list <linux-kernel@vger.kernel.org>,
Linus Torvalds <torvalds@linux-foundation.org>
Subject: Re: patch "staging: remove nokia_hp4p driver
Date: Sat, 30 Aug 2014 15:04:17 -0700 [thread overview]
Message-ID: <20140830220417.GA15000@kroah.com> (raw)
In-Reply-To: <20140830213022.GA17147@amd>
On Sat, Aug 30, 2014 at 11:30:22PM +0200, Pavel Machek wrote:
> Hi!
>
> > This is a note to let you know that I've just added the patch titled
> >
> > staging: remove nokia_hp4p driver
> > -Re: [PATCH v6] Bluetooth: Add hci_h4p driver
> >
> > to my staging git tree which can be found at
> > git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git
> > in the staging-next branch.
> >
> > The patch will show up in the next release of the linux-next tree
> > (usually sometime within the next 24 hours during the week.)
> >
> > The patch will also be merged in the next major kernel release
> > during the merge window.
> >
> > If you have any questions about this process, please let me know.
>
> What is going on here? I get flamed for not cleaning up the driver,
> because I cleaned it up before merging to -staging. Ok, so I did more
> cleanups, sent 3 cleanup patches, no reaction on those, and now I got
> a note that you are going to remove the driver...?
For the 3 "cleanup" patches, the first one was rejected and you said to
not include it, so I couldn't apply the others.
> Please don't, I'd still like to clean the driver up and get included,
> as n900's are still under active use.
As the Bluetooth maintainer has said a number of times, he doesn't want
the driver in the tree as it is not doing the correct things. It's been
a long time in the tree with no work on it at all, and I follow the
suggestions of the maintainers of the subsystems that staging drivers
follow.
I suggest cleaning this up in your own tree, and then just submitting it
for inclusion in the "normal" part of the kernel. That way I'm not
standing in your way on applying patches, and you can work much quicker
to resolve the issues that Marcel has pointed out.
thanks,
greg k-h
next prev parent reply other threads:[~2014-08-30 22:04 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <14094295422869@kroah.com>
2014-08-30 21:30 ` patch "staging: remove nokia_hp4p driver Pavel Machek
2014-08-30 22:04 ` Greg KH [this message]
2014-08-30 22:44 ` Pavel Machek
2014-08-30 22:49 ` Greg KH
2014-08-31 9:28 ` Pavel Machek
2014-08-30 23:09 ` Marcel Holtmann
2014-08-31 8:23 ` Pali Rohár
2014-08-31 9:51 ` Pavel Machek
2014-09-01 22:16 ` Marcel Holtmann
2014-09-04 11:36 ` Pavel Machek
2014-12-03 12:33 ` 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=20140830220417.GA15000@kroah.com \
--to=gregkh@linuxfoundation.org \
--cc=cmroliv@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=marcel@holtmann.org \
--cc=pali.rohar@gmail.com \
--cc=pavel@ucw.cz \
--cc=torvalds@linux-foundation.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