Return-path: Received: from mx1.suse.de ([195.135.220.2]:38121 "EHLO mx1.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753728AbYJPIpD (ORCPT ); Thu, 16 Oct 2008 04:45:03 -0400 Date: Thu, 16 Oct 2008 10:43:33 +0200 Message-ID: (sfid-20081016_104510_450642_E6B0CF1F) From: Takashi Iwai To: Adrian Bunk Cc: Andrew Morton , Dan Williams , linux-ext4@vger.kernel.org, netdev@vger.kernel.org, linux-ide@vger.kernel.org, Bartlomiej Zolnierkiewicz , marek.vasut@gmail.com, David Woodhouse , Mark Fasheh , Ralf Baechle , Mauro Carvalho Chehab , linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org, linuxppc-dev@ozlabs.org Subject: Re: powerpc allmodconfig In-Reply-To: <20081016082157.GC23630@cs181140183.pp.htv.fi> References: <20081015213337.a99a9595.akpm@linux-foundation.org> <20081016073836.GB23630@cs181140183.pp.htv.fi> <20081016082157.GC23630@cs181140183.pp.htv.fi> MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Sender: linux-wireless-owner@vger.kernel.org List-ID: At Thu, 16 Oct 2008 11:21:57 +0300, Adrian Bunk wrote: > > On Thu, Oct 16, 2008 at 09:57:11AM +0200, Takashi Iwai wrote: > > At Thu, 16 Oct 2008 10:38:36 +0300, > > Adrian Bunk wrote: > > > > > > On Thu, Oct 16, 2008 at 07:57:29AM +0200, Takashi Iwai wrote: > > > > At Wed, 15 Oct 2008 21:33:37 -0700, > > > > Andrew Morton wrote: > > > > > > > > > > sound/soc/soc-dapm.c:1029: warning: 'snd_soc_dapm_connect_input' is deprecated (declared at sound/soc/soc-dapm.c:1026) > > > > > sound/soc/soc-dapm.c:1029: warning: 'snd_soc_dapm_connect_input' is deprecated (declared at sound/soc/soc-dapm.c:1026) > > > > > > > > These are definitions of deprecated interfaces. > > > > We can remove it in 2.6.29. If we don't want to be conservative, it > > > > can be removed in 2.6.28, too. > > > >... > > > > > > Since it's an in-kernel API there's no reason to keep it once there are > > > no users left. > > > > Right. But, IMO, now is no suitable time. > > A thing like API removal should have been tested in linux-next, and we > > had plenty of time indeed for 2.6.28. > >... > > A grep through the tree and one test compile that covers > sound/soc/soc-dapm.c should be enough testing. > > And having it then in -next once should be enough to discover if someone > wrongly added a new user. My point is the time for removal. The API changes should have been done in the merge window, and it should have been tested *before* the merge window. > I have removed many functions in the kernel, and there isn't much that > can go wrong - even adding a PCI ID to a driver has a bigger risk of > introducing a regression. Yeah, IMHO, adding PCI IDs blindly at the late stage should be avoided, too, although many people love that. thanks, Takashi