Re: [PATCH 1/2] staging: octeon: delete driver

[gregkh@xxxxxxxxxxxxxxxxxxx]

On Tue, Feb 04, 2020 at 12:31:16PM -0800, Matthew Wilcox wrote:
> On Tue, Feb 04, 2020 at 08:06:14PM +0000, Chris Packham wrote:
> > On Tue, 2020-02-04 at 07:09 +0000, gregkh@xxxxxxxxxxxxxxxxxxx wrote:
> > > On Tue, Feb 04, 2020 at 04:02:15AM +0000, Chris Packham wrote:
> > > > I'll pipe up on this thread too
> > > > 
> > > > On Tue, 2019-12-10 at 02:42 -0800, Guenter Roeck wrote:
> > > > > On 12/10/19 1:15 AM, Greg Kroah-Hartman wrote:
> > > > > > This driver has been in the tree since 2009 with no real movement to get
> > > > > > it out.  Now it is starting to cause build issues and other problems for
> > > > > > people who want to fix coding style problems, but can not actually build
> > > > > > it.
> > > > > > 
> > > > > > As nothing is happening here, just delete the module entirely.
> > > > > > 
> > > > > > Reported-by: Guenter Roeck <linux@xxxxxxxxxxxx>
> > > > > > Cc: David Daney <ddaney@xxxxxxxxxxxxxxxxxx>
> > > > > > Cc: "David S. Miller" <davem@xxxxxxxxxxxxx>
> > > > > > Cc: "Matthew Wilcox (Oracle)" <willy@xxxxxxxxxxxxx>
> > > > > > Cc: Guenter Roeck <linux@xxxxxxxxxxxx>
> > > > > > Cc: YueHaibing <yuehaibing@xxxxxxxxxx>
> > > > > > Cc: Aaro Koskinen <aaro.koskinen@xxxxxx>
> > > > > > Cc: Wambui Karuga <wambui.karugax@xxxxxxxxx>
> > > > > > Cc: Julia Lawall <julia.lawall@xxxxxxx>
> > > > > > Cc: Florian Westphal <fw@xxxxxxxxx>
> > > > > > Cc: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>
> > > > > > Cc: Branden Bonaby <brandonbonaby94@xxxxxxxxx>
> > > > > > Cc: "Petr Åtetiar" <ynezz@xxxxxxx>
> > > > > > Cc: Sandro Volery <sandro@xxxxxxxxxx>
> > > > > > Cc: Paul Burton <paulburton@xxxxxxxxxx>
> > > > > > Cc: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
> > > > > > Cc: Giovanni Gherdovich <bobdc9664@xxxxxxxxx>
> > > > > > Cc: Valery Ivanov <ivalery111@xxxxxxxxx>
> > > > > > Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
> > > > > 
> > > > > Acked-by: Guenter Roeck <linux@xxxxxxxxxxxx>
> > > > 
> > > > Please can we keep this driver. We do have platforms using it and we
> > > > would like it to stay around.
> > > > 
> > > > Clearly we'll need to sort things out to a point where they build
> > > > successfully. We've been hoping to see this move out of staging ever
> > > > since we selected Cavium as a vendor.
> > > 
> > > Great, can you send me a patchset that reverts this and fixes the build
> > > issues and accept maintainership of the code?
> > > 
> > 
> > Yep will do.
> > 
> > On Tue, 2020-02-04 at 10:21 +0300, Dan Carpenter wrote:
> > > My advice is to delete all the COMPILE_TEST code.  That stuff was a
> > > constant source of confusion and headaches.
> > 
> > I was also going to suggest this. Since the COMPILE_TEST has been a
> > source of trouble I was going to propose dropping the || COMPILE_TEST
> > from the Kconfig for the octeon drivers.
> 
> Not having it also causes problems.  I didn't originally add it for
> shits and giggles.

Yes, without this option, the code bit-rotted horribly.  It needs to be
able to be built otherwise people will change the code and it will break
and no one will notice except Guenter's build bots and then no one will
fix it :(

i.e. exactly what was happening before...

So this needs to all be fixed up properly, and really, should be merged
to the "real" part of the kernel...

thanks,

greg k-h