From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lukas Wunner Subject: Re: linux-next: build failure after merge of the drm-misc tree Date: Wed, 10 Feb 2016 14:17:41 +0100 Message-ID: <20160210131741.GA15492@wunner.de> References: <20160210122451.55391e43@canb.auug.org.au> <20160210084138.GA15392@wunner.de> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: <20160210084138.GA15392@wunner.de> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Stephen Rothwell Cc: Daniel Vetter , intel-gfx@lists.freedesktop.org, "Rafael J. Wysocki" , linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-acpi@vger.kernel.org, linux-next@vger.kernel.org, Darren Hart List-Id: linux-acpi@vger.kernel.org SGksCgpPbiBXZWQsIEZlYiAxMCwgMjAxNiBhdCAwOTo0MTozOEFNICswMTAwLCBMdWthcyBXdW5u ZXIgd3JvdGU6Cj4gT24gV2VkLCBGZWIgMTAsIDIwMTYgYXQgMTI6MjQ6NTFQTSArMTEwMCwgU3Rl cGhlbiBSb3Rod2VsbCB3cm90ZToKPiA+IEhpIGFsbCwKPiA+IAo+ID4gQWZ0ZXIgbWVyZ2luZyB0 aGUgZHJtLW1pc2MgdHJlZSwgdG9kYXkncyBsaW51eC1uZXh0IGJ1aWxkIChhcm0KPiA+IG11bHRp X3Y3X2RlZmNvbmZpZykgZmFpbGVkIGxpa2UgdGhpczoKPiA+IAo+ID4gSW4gZmlsZSBpbmNsdWRl ZCBmcm9tIGRyaXZlcnMvZ3B1L2RybS9ub3V2ZWF1L25vdXZlYXVfZHJtLmM6MjU6MDoKPiA+IGlu Y2x1ZGUvbGludXgvYXBwbGUtZ211eC5oOiBJbiBmdW5jdGlvbiAnYXBwbGVfZ211eF9wcmVzZW50 JzoKPiA+IGluY2x1ZGUvbGludXgvYXBwbGUtZ211eC5oOjM2OjQyOiBlcnJvcjogaW1wbGljaXQg ZGVjbGFyYXRpb24gb2YgZnVuY3Rpb24gJ2FjcGlfZGV2X3ByZXNlbnQnIFstV2Vycm9yPWltcGxp Y2l0LWZ1bmN0aW9uLWRlY2xhcmF0aW9uXQo+ID4gICByZXR1cm4gSVNfRU5BQkxFRChDT05GSUdf QVBQTEVfR01VWCkgJiYgYWNwaV9kZXZfcHJlc2VudChHTVVYX0FDUElfSElEKTsKPiA+ICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIF4KPiA+IAo+ID4gQ2F1c2VkIGJ5 IGNvbW1pdAo+ID4gCj4gPiAgIDI0MTMzMDZjMjU2NiAoImFwcGxlLWdtdXg6IEFkZCBoZWxwZXIg Zm9yIHByZXNlbmNlIGRldGVjdCIpCj4gPiAKPiA+IEkgaGF2ZSB1c2VkIHRoZSBkcm0tbWlzYyB0 cmVlIGZyb20gbmV4dC0yMDE2MDIwOSBmb3IgdG9kYXkuCj4gCj4gVWdoLCBhcG9sb2dpZXMsIEkg ZGlkbid0IGhhdmUgYSBub24tQUNQSSBwbGF0Zm9ybSBhdmFpbGFibGUgdG8gdGVzdAo+IHRoaXMg b24uCj4gCj4gU29sdXRpb24gaXMgdG8gZWl0aGVyIGFkZCB0byBpbmNsdWRlL2xpbnV4L2FjcGku aAo+IAo+IHN0YXRpYyBpbmxpbmUgYm9vbCBhY3BpX2Rldl9wcmVzZW50KGNvbnN0IGNoYXIgKmhp ZCkKPiB7Cj4gCXJldHVybiBmYWxzZTsKPiB9Cj4gCj4gc29tZXdoZXJlIGJlbG93Cj4gCj4gI2Vs c2UJLyogIUNPTkZJR19BQ1BJICovCj4gCj4gb3IgYWx0ZXJuYXRpdmVseSB0byBhZGQgdG8gaW5j bHVkZS9saW51eC9hcHBsZS1nbXV4LmgKPiAKPiBJU19FTkFCTEVEKENPTkZJR19BQ1BJKQo+IAo+ IGluIGFwcGxlX2dtdXhfcHJlc2VudCgpLgo+IAo+IEknbGwgY2hlY2sgdGhlIG90aGVyIHVzZXJz IG9mIGFjcGlfZGV2X3ByZXNlbnQoKSB0byBzZWUgd2hpY2ggb2YKPiB0aGVzZSB0d28gc29sdXRp b25zIGlzIG1vcmUgYXBwcm9wcmlhdGUgYW5kIHdpbGwgcG9zdCBhIGZpeCBzaG9ydGx5LgoKVGhl IHBhdGNoIGluY2x1ZGVkIGJlbG93IGZpeGVzIHRoZSBidWlsZCBpZiBDT05GSUdfQUNQSSBpcyBu b3Qgc2V0LgoKQERhbmllbCBWZXR0ZXI6IFdvdWxkIGl0IGJlIHBvc3NpYmxlIHRvIHNxdWFzaCB0 aGlzIHdpdGggMjQxMzMwNmMyNTY2CigiYXBwbGUtZ211eDogQWRkIGhlbHBlciBmb3IgcHJlc2Vu Y2UgZGV0ZWN0Iikgb24gdG9waWMvZHJtLW1pc2Mgc28KYXMgdG8gYXZvaWQgYnVpbGQgYnJlYWth Z2UgZm9yIGFueW9uZSB0cnlpbmcgdG8gYmlzZWN0IGJldHdlZW4gdGhhdApjb21taXQgYW5kIHRo aXMgZml4PwoKSSBjaGVja2VkIGFsbCBvdGhlciB1c2VycyBvZiBhY3BpX2Rldl9wcmVzZW50KCkg YW5kIGFsbCBvZiB0aGVtIGFyZQpvbmx5IGNvbXBpbGVkIGlmIENPTkZJR19BQ1BJIGlzIHNldC4g SGVuY2UgSSBvcHRlZCB0byBmaXggdGhpcyBpbgo8bGludXgvYXBwbGUtZ211eC5oPiByYXRoZXIg dGhhbiBpbiA8bGludXgvYWNwaS5oPi4KClRoYW5rcyBhZ2FpbiBTdGVwaGVuIGZvciByZXBvcnRp bmcgdGhpcyBhdCBzdWNoIGFuIGVhcmx5IHN0YWdlLAp0aG91Z2ggZG91YnRsZXNzbHkgaXQgd291 bGQgaGF2ZSBiZWVuIGJldHRlciBpZiBJIGhhZCB0aG91Z2h0IG9mCnRoaXMgcG9zc2liaWxpdHkg d2hlbiBwcmVwYXJpbmcgdGhlIG9yaWdpbmFsIHBhdGNoLCBvciBpZiBJIGhhZApjb21waWxlLXRl c3RlZCB3aXRob3V0IENPTkZJR19BQ1BJLiA6LSgKCkx1a2FzCgotLSA+OCAtLQpTdWJqZWN0OiBb UEFUQ0hdIGFwcGxlLWdtdXg6IEZpeCBidWlsZCBicmVha2FnZSBpZiAhQ09ORklHX0FDUEkKClRo ZSBEUk0gZHJpdmVycyBpOTE1LCBub3V2ZWF1IGFuZCByYWRlb24gbWF5IGJlIGNvbXBpbGVkIHdp dGgKQ09ORklHX0FDUEkgbm90IHNldCwgaW4gd2hpY2ggY2FzZSBhY3BpX2Rldl9wcmVzZW50KCkg aXMgdW5kZWZpbmVkLgoKQWRkIGEgbm8tb3Agc3R1YiBmb3IgYXBwbGVfZ211eF9wcmVzZW50KCkg d2hpY2ggaXMgdXNlZCBpZgpDT05GSUdfQVBQTEVfR01VWCBpcyBub3QgZW5hYmxlZCB0byBhdm9p ZCBidWlsZCBicmVha2FnZS4KKENPTkZJR19BUFBMRV9HTVVYIGRlcGVuZHMgb24gQ09ORklHX0FD UEkuKQoKUmVwb3J0ZWQtYnk6IFN0ZXBoZW4gUm90aHdlbGwgPHNmckBjYW5iLmF1dWcub3JnLmF1 PgpTaWduZWQtb2ZmLWJ5OiBMdWthcyBXdW5uZXIgPGx1a2FzQHd1bm5lci5kZT4KLS0tCiBpbmNs dWRlL2xpbnV4L2FwcGxlLWdtdXguaCB8IDEzICsrKysrKysrKysrKy0KIDEgZmlsZSBjaGFuZ2Vk LCAxMiBpbnNlcnRpb25zKCspLCAxIGRlbGV0aW9uKC0pCgpkaWZmIC0tZ2l0IGEvaW5jbHVkZS9s aW51eC9hcHBsZS1nbXV4LmggYi9pbmNsdWRlL2xpbnV4L2FwcGxlLWdtdXguaAppbmRleCBmZWVi YzI4Li5iMmQzMmUwIDEwMDY0NAotLS0gYS9pbmNsdWRlL2xpbnV4L2FwcGxlLWdtdXguaAorKysg Yi9pbmNsdWRlL2xpbnV4L2FwcGxlLWdtdXguaApAQCAtMjIsNiArMjIsOCBAQAogCiAjZGVmaW5l IEdNVVhfQUNQSV9ISUQgIkFQUDAwMEIiCiAKKyNpZiBJU19FTkFCTEVEKENPTkZJR19BUFBMRV9H TVVYKQorCiAvKioKICAqIGFwcGxlX2dtdXhfcHJlc2VudCgpIC0gZGV0ZWN0IGlmIGdtdXggaXMg YnVpbHQgaW50byB0aGUgbWFjaGluZQogICoKQEAgLTMzLDcgKzM1LDE2IEBACiAgKi8KIHN0YXRp YyBpbmxpbmUgYm9vbCBhcHBsZV9nbXV4X3ByZXNlbnQodm9pZCkKIHsKLQlyZXR1cm4gSVNfRU5B QkxFRChDT05GSUdfQVBQTEVfR01VWCkgJiYgYWNwaV9kZXZfcHJlc2VudChHTVVYX0FDUElfSElE KTsKKwlyZXR1cm4gYWNwaV9kZXZfcHJlc2VudChHTVVYX0FDUElfSElEKTsKIH0KIAorI2Vsc2Ug IC8qICFDT05GSUdfQVBQTEVfR01VWCAqLworCitzdGF0aWMgaW5saW5lIGJvb2wgYXBwbGVfZ211 eF9wcmVzZW50KHZvaWQpCit7CisJcmV0dXJuIGZhbHNlOworfQorCisjZW5kaWYgLyogIUNPTkZJ R19BUFBMRV9HTVVYICovCisKICNlbmRpZiAvKiBMSU5VWF9BUFBMRV9HTVVYX0ggKi8KLS0gCjEu OC41LjIgKEFwcGxlIEdpdC00OCkKCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fCkludGVsLWdmeCBtYWlsaW5nIGxpc3QKSW50ZWwtZ2Z4QGxpc3RzLmZyZWVk ZXNrdG9wLm9yZwpodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZv L2ludGVsLWdmeAo= From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751826AbcBJNXt (ORCPT ); Wed, 10 Feb 2016 08:23:49 -0500 Received: from mailout2.hostsharing.net ([83.223.90.233]:55756 "EHLO mailout2.hostsharing.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751712AbcBJNXq (ORCPT ); Wed, 10 Feb 2016 08:23:46 -0500 X-Greylist: delayed 383 seconds by postgrey-1.27 at vger.kernel.org; Wed, 10 Feb 2016 08:23:45 EST Date: Wed, 10 Feb 2016 14:17:41 +0100 From: Lukas Wunner To: Stephen Rothwell Cc: 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: <20160210131741.GA15492@wunner.de> References: <20160210122451.55391e43@canb.auug.org.au> <20160210084138.GA15392@wunner.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20160210084138.GA15392@wunner.de> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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. :-( 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)