[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20140805041325.GA27507@rhlx01.hs-esslingen.de>
Date: Tue, 5 Aug 2014 06:13:25 +0200
From: Andreas Mohr <andi@...as.de>
To: Greg KH <gregkh@...uxfoundation.org>
Cc: Kristina Martšenko
<kristina.martsenko@...il.com>,
Linus Torvalds <torvalds@...ux-foundation.org>,
Andrew Morton <akpm@...ux-foundation.org>,
linux-kernel@...r.kernel.org, devel@...uxdriverproject.org
Subject: Re: [GIT PULL] Staging driver patches for 3.17-rc1
> Lots of things in here, over 2000 patches, but the best part is this:
> 1480 files changed, 39070 insertions(+), 254659 deletions(-)
>
> Thanks to the great work of Kristina Martšenko, 14 different staging
> drivers have been removed from the tree as they were obsolete and no
> one
> was willing to work on cleaning them up.
-ENOPARSE.
Define "as they were obsolete and no one was willing to work on cleaning
them up."?
*Either* they're obsolete (due to better alternatives available, in the
standard case), in which case they indeed can be removed on the spot,
*or* they are expected to become full-service drivers and thus need cleanup.
Which is it?
In the case of keucr (a driver where I happen to own hardware,
and where AFAIK this driver is required),
I really don't see "nobody was willing to work on cleaning them up.",
as git log tells. Admittedly there probably was no substantial rework,
but it did have many janitorial changes, and some changes
only 4 months ago even (a time frame which IMHO does not justify
removing a driver wholesale, with the effect of certain users to not be able to
use them then).
Oh well, yet another driver where it became more difficult rather than easier
to make forward progress.
But I guess that change was done very intentionally, likely to induce pain
in non-cooperative areas ;-P
(which may or may not be a good thing...)
Andreas Mohr
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists