From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753985AbaJGFcs (ORCPT ); Tue, 7 Oct 2014 01:32:48 -0400 Received: from cantor2.suse.de ([195.135.220.15]:38666 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753639AbaJGFcU (ORCPT ); Tue, 7 Oct 2014 01:32:20 -0400 Date: Tue, 07 Oct 2014 07:32:18 +0200 Message-ID: From: Takashi Iwai To: Andreas Mohr Cc: Ondrej Zary , alsa-devel@alsa-project.org, Kernel development list Subject: Re: Workable vintage driver support mechanism? (Re: [alsa-devel] [PATCH v3] ES938 support for ES18xx driver) In-Reply-To: <20141006184150.GA2465@rhlx01.hs-esslingen.de> References: <1412027079-15876-1-git-send-email-linux@rainbow-software.org> <201410061555.19007.linux@rainbow-software.org> <20141006184150.GA2465@rhlx01.hs-esslingen.de> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI/1.14.6 (Maruoka) FLIM/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL/10.8 Emacs/24.3 (x86_64-suse-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org At Mon, 6 Oct 2014 20:41:50 +0200, Andreas Mohr wrote: > > On Mon, Oct 06, 2014 at 04:13:12PM +0200, Takashi Iwai wrote: > > At Mon, 6 Oct 2014 15:55:18 +0200, > > Ondrej Zary wrote: > > > ES938 does not depend on ES18xx and can be connected to any device with MIDI > > > interface. Maybe there are some other cards with this chip. > > > > Please prove it :) > > :( > > > That said, I'm not willing to merge the patch in the current form. > > One of the reason is that this can be implemented pretty easily in > > user-space. Another is that it's a deadly old device, and likely only > > handful boards are still running in the world, thus no much motivation > > to bloat the kernel code for that. We're even discussing to get rid > > of ISA codes from the kernel tree nowadays. > > > > Sorry, the patch was submitted 10 years too late. > > :(( > > Given that I hate the forces that are increasingly coming into play here > (but of course I do see the bigger picture, namely of kernel tree bloat), > I think I have an idea which might be useful to accept: > for every piece of sufficiently "vintage" submission, > people would be tasked with offering (or somehow ensuring) > a sufficiently closely time-related cleanup in other places. > > Thus, given a "diffstat penalty" (lines added minus lines removed) > of the vintage support patch: > > Additionally offer a cleanup patch > which gets rid of redundantly implemented kernel tree functionality: > - for 15 year old devices: 0.5 * diffstat_penalty > - for 20 year old devices: 1.0 * diffstat_penalty > - for 25 year old devices: 1.5 * diffstat_penalty > - for 30 year old devices: whaaa? The kernel bloat is one side, and your proposal would help to keep figure. But another side is the maintenance, as Clemens pointed. It's obviously more difficult for exotic hardware devices. So, it's really case-by-case. Speaking of this particular patch, however, the reason for NAK isn't (only) the vintage. There are other concerns as I raised, especially the ugliness of the interface it uses, and this doesn't have to be a kernel stuff at all. You could implement it as a user-space alsa-lib plugin as well. Takashi