From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Sat, 16 Nov 2013 08:39:50 +0900 From: Greg Kroah-Hartman To: Marcel Holtmann Cc: Dan Carpenter , Geert Uytterhoeven , devel@driverdev.osuosl.org, Johan Hedberg , Yu-Chen Cho , "linux-kernel@vger.kernel.org list" , "linux-bluetooth@vger.kernel.org development" , jay.hung@mediatek.com Subject: Re: [PATCH] Staging: btmtk_usb: Add hdev parameter to hdev->send driver callback Message-ID: <20131115233950.GA17814@kroah.com> References: <1384517231-1605-1-git-send-email-geert@linux-m68k.org> <20131115130820.GS5302@mwanda> <20131115212936.GC14897@kroah.com> <6C5AC1E9-D9A0-473A-B47A-345018DFAC3C@holtmann.org> <324E99F5-B9B7-4E56-BE91-42964AFF5BA8@holtmann.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <324E99F5-B9B7-4E56-BE91-42964AFF5BA8@holtmann.org> List-ID: On Sat, Nov 16, 2013 at 07:36:31AM +0900, Marcel Holtmann wrote: > Hi Greg, > > >>>>> while this is patch is correct, I do not really care about staging drivers that actually bluntly violate my copyright. > >>>>> > >>>> > >>>> That's very cryptic. > >>>> > >>>> What is going on here? I googled it and I wasn't able to find what you > >>>> are talking about. Care to give us a hint and what you want us to do > >>>> here? > >>> > >>> the last time I checked, the majority of drivers/bluetooth/btusb.c has been written by myself. Now go and compare btusb.c to btmtk_usb.[ch]. > >>> > >>>> I have also added Johan Hedberg to the CC list because he also helped > >>>> break the build. Don't do that. > >>> > >>> Yes, we are doing exactly that. It is a staging driver. I could not care less if a staging drivers breaks the build or not. > >>> > >>> If anybody cares about this driver, then take the time to merge it upstream. It has never been submitted to linux-bluetooth mailing list. > >>> > >>> There are drivers that should have never been merged into staging. > >>> This is one of them. Look for yourself and explain to me why this > >>> driver is part of staging in the first place. > >> > >> Because it was sent to me by a developer? > > > > it is a problem when staging just becomes a dumping ground for drivers that the distributions find somewhere on the Internet or CD-ROMs. And then nobody has any intentions to clean up and integrate properly. This one did not even go through linux-bluetooth mailing list once. It was submitted right to staging. And then the submitter walked away. > > and if I quote the TODO file: > > TODO: > - checkpatch.pl clean > - determine if the driver should not be using a duplicate > version of the usb-bluetooth interface code, but should > be merged into the drivers/bluetooth/ directory and > infrastructure instead. > - review by the bluetooth developer community > > Please send any patches for this driver to Yu-Chen, Cho and > jay.hung@mediatek.com > > So from the submission we can assume that the submitter knew that this > was duplicated code. The code also never got submitted for review to > linux-bluetooth. And now 6 month later, none of the TODO items have > been actually worked on. > > I do not know what your timeline is for removing drivers from staging, > but this one seems to be a good candidate to get removed next. 6 months without any active contribution to getting it cleaned up and merged is the timeline. Which this one fits, so yes, I will remove it for 3.14, unless Jay or Cho is going to start doing work on this. thanks, greg k-h