From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ralf Baechle Date: Thu, 21 Nov 2013 17:39:33 +0000 Subject: Re: [PATCH 24/34] PCI: use weak functions for MSI arch-specific functions Message-Id: <20131121173933.GT10382@linux-mips.org> List-Id: References: <1384915853-31006-1-git-send-email-r65037@freescale.com> <1384915853-31006-24-git-send-email-r65037@freescale.com> In-Reply-To: <1384915853-31006-24-git-send-email-r65037@freescale.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Richard Zhu Cc: Thomas Petazzoni , Benjamin Herrenschmidt , Paul Mackerras , linuxppc-dev@lists.ozlabs.org, Martin Schwidefsky , Heiko Carstens , linux390@de.ibm.com, linux-s390@vger.kernel.org, Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , x86@kernel.org, Russell King , Tony Luck , Fenghua Yu , linux-ia64@vger.kernel.org, linux-mips@linux-mips.org, "David S. Miller" , sparclinux@vger.kernel.org, Chris Metcalf , Jason Cooper On Wed, Nov 20, 2013 at 10:50:43AM +0800, Richard Zhu wrote: Looking good, Acked-by: Ralf Baechle Nevertheless I'd again like to express that I'm not that fond of of the increasing number of weak functions in the kernel. In the old days things were such that when an a platform didn't provice a platform hook or enable a default hook function, one would get a build error - an unmistakable sign to the maintainer that something needs attention. Weak functions mean default functions may result in subtly incorrect operation. Been there, got bitten. Ralf