From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751502AbcBJNgk (ORCPT ); Wed, 10 Feb 2016 08:36:40 -0500 Received: from mail-wm0-f46.google.com ([74.125.82.46]:33268 "EHLO mail-wm0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751047AbcBJNgi (ORCPT ); Wed, 10 Feb 2016 08:36:38 -0500 Date: Wed, 10 Feb 2016 14:37:00 +0100 From: Daniel Vetter To: Lukas Wunner Cc: Stephen Rothwell , Daniel Vetter , intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, linux-next@vger.kernel.org, linux-kernel@vger.kernel.org, Darren Hart , "Rafael J. Wysocki" , linux-acpi@vger.kernel.org Subject: Re: linux-next: build failure after merge of the drm-misc tree Message-ID: <20160210133700.GZ11240@phenom.ffwll.local> Mail-Followup-To: Lukas Wunner , Stephen Rothwell , intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, linux-next@vger.kernel.org, linux-kernel@vger.kernel.org, Darren Hart , "Rafael J. Wysocki" , linux-acpi@vger.kernel.org References: <20160210122451.55391e43@canb.auug.org.au> <20160210084138.GA15392@wunner.de> <20160210131741.GA15492@wunner.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20160210131741.GA15492@wunner.de> X-Operating-System: Linux phenom 4.3.0-1-amd64 User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Feb 10, 2016 at 02:17:41PM +0100, Lukas Wunner wrote: > Hi, > > On Wed, Feb 10, 2016 at 09:41:38AM +0100, Lukas Wunner wrote: > > On Wed, Feb 10, 2016 at 12:24:51PM +1100, Stephen Rothwell wrote: > > > Hi all, > > > > > > After merging the drm-misc tree, today's linux-next build (arm > > > multi_v7_defconfig) failed like this: > > > > > > In file included from drivers/gpu/drm/nouveau/nouveau_drm.c:25:0: > > > include/linux/apple-gmux.h: In function 'apple_gmux_present': > > > include/linux/apple-gmux.h:36:42: error: implicit declaration of function 'acpi_dev_present' [-Werror=implicit-function-declaration] > > > return IS_ENABLED(CONFIG_APPLE_GMUX) && acpi_dev_present(GMUX_ACPI_HID); > > > ^ > > > > > > Caused by commit > > > > > > 2413306c2566 ("apple-gmux: Add helper for presence detect") > > > > > > I have used the drm-misc tree from next-20160209 for today. > > > > Ugh, apologies, I didn't have a non-ACPI platform available to test > > this on. > > > > Solution is to either add to include/linux/acpi.h > > > > static inline bool acpi_dev_present(const char *hid) > > { > > return false; > > } > > > > somewhere below > > > > #else /* !CONFIG_ACPI */ > > > > or alternatively to add to include/linux/apple-gmux.h > > > > IS_ENABLED(CONFIG_ACPI) > > > > in apple_gmux_present(). > > > > I'll check the other users of acpi_dev_present() to see which of > > these two solutions is more appropriate and will post a fix shortly. > > The patch included below fixes the build if CONFIG_ACPI is not set. > > @Daniel Vetter: Would it be possible to squash this with 2413306c2566 > ("apple-gmux: Add helper for presence detect") on topic/drm-misc so > as to avoid build breakage for anyone trying to bisect between that > commit and this fix? > > I checked all other users of acpi_dev_present() and all of them are > only compiled if CONFIG_ACPI is set. Hence I opted to fix this in > rather than in . > > Thanks again Stephen for reporting this at such an early stage, > though doubtlessly it would have been better if I had thought of > this possibility when preparing the original patch, or if I had > compile-tested without CONFIG_ACPI. :-( Since !ACPI and enabling APPLE_GMUX is a bit a fringe config I don't think this will hurt bisectability at all on a real system. So just applied this one on top of drm-misc. Thanks, Daniel > > Lukas > > -- >8 -- > Subject: [PATCH] apple-gmux: Fix build breakage if !CONFIG_ACPI > > The DRM drivers i915, nouveau and radeon may be compiled with > CONFIG_ACPI not set, in which case acpi_dev_present() is undefined. > > Add a no-op stub for apple_gmux_present() which is used if > CONFIG_APPLE_GMUX is not enabled to avoid build breakage. > (CONFIG_APPLE_GMUX depends on CONFIG_ACPI.) > > Reported-by: Stephen Rothwell > Signed-off-by: Lukas Wunner > --- > include/linux/apple-gmux.h | 13 ++++++++++++- > 1 file changed, 12 insertions(+), 1 deletion(-) > > diff --git a/include/linux/apple-gmux.h b/include/linux/apple-gmux.h > index feebc28..b2d32e0 100644 > --- a/include/linux/apple-gmux.h > +++ b/include/linux/apple-gmux.h > @@ -22,6 +22,8 @@ > > #define GMUX_ACPI_HID "APP000B" > > +#if IS_ENABLED(CONFIG_APPLE_GMUX) > + > /** > * apple_gmux_present() - detect if gmux is built into the machine > * > @@ -33,7 +35,16 @@ > */ > static inline bool apple_gmux_present(void) > { > - return IS_ENABLED(CONFIG_APPLE_GMUX) && acpi_dev_present(GMUX_ACPI_HID); > + return acpi_dev_present(GMUX_ACPI_HID); > } > > +#else /* !CONFIG_APPLE_GMUX */ > + > +static inline bool apple_gmux_present(void) > +{ > + return false; > +} > + > +#endif /* !CONFIG_APPLE_GMUX */ > + > #endif /* LINUX_APPLE_GMUX_H */ > -- > 1.8.5.2 (Apple Git-48) > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch