From: Takashi Iwai Subject: Re: powerpc allmodconfig Date: Thu, 16 Oct 2008 10:43:33 +0200 Message-ID: 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 Cc: Andrew Morton , Dan Williams , linux-ext4-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-ide-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Bartlomiej Zolnierkiewicz , marek.vasut-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, David Woodhouse , Mark Fasheh , Ralf Baechle , Mauro Carvalho Chehab , linux-wireless-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linuxppc-dev-mnsaURCQ41sdnm+yROfE0A@public.gmane.org To: Adrian Bunk Return-path: In-Reply-To: <20081016082157.GC23630-re2QNgSbS3j4D6uPqz5PAwR5/fbUUdgG@public.gmane.org> Sender: linux-wireless-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-ext4.vger.kernel.org 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 -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html