From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751283AbeEDK7R (ORCPT ); Fri, 4 May 2018 06:59:17 -0400 Received: from bombadil.infradead.org ([198.137.202.133]:36522 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750820AbeEDK7Q (ORCPT ); Fri, 4 May 2018 06:59:16 -0400 Date: Fri, 4 May 2018 07:59:06 -0300 From: Mauro Carvalho Chehab To: Bartlomiej Zolnierkiewicz Cc: Randy Dunlap , Anders Roxell , tomi.valkeinen@ti.com, linux-omap@vger.kernel.org, linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] drivers: omap2: Kconfig: make FB_OMAP2_DSS_INIT depend on OF Message-ID: <20180504075906.02ecf32e@vento.lan> In-Reply-To: <44871995.YYHkG5ofpC@amdc3058> References: <20180420112551.27027-1-anders.roxell@linaro.org> <16e74acc-1265-faec-0c36-49681a7760e4@infradead.org> <44871995.YYHkG5ofpC@amdc3058> X-Mailer: Claws Mail 3.16.0 (GTK+ 2.24.32; x86_64-redhat-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id w44AxMYZ023220 Em Fri, 04 May 2018 12:48:46 +0200 Bartlomiej Zolnierkiewicz escreveu: > On Thursday, May 03, 2018 08:48:56 AM Randy Dunlap wrote: > > On 04/20/2018 04:25 AM, Anders Roxell wrote: > > > Commit 7378f1149884 ("media: omap2: omapfb: allow building it with > > > COMPILE_TEST") broke compilation without CONFIG_OF selected. > > > CC drivers/video/fbdev/core/fbmem.o > > > drivers/video/fbdev/omap2/omapfb/dss/omapdss-boot-init.c: In function ‘omapdss_update_prop’: > > > drivers/video/fbdev/omap2/omapfb/dss/omapdss-boot-init.c:68:2: error: implicit declaration of function ‘of_update_property’; did you mean ‘of_get_property’? [-Werror=implicit-function-declaration] > > > of_update_property(node, prop); > > > ^~~~~~~~~~~~~~~~~~ > > > of_get_property > > > cc1: some warnings being treated as errors > > > scripts/Makefile.build:312: recipe for target 'drivers/video/fbdev/omap2/omapfb/dss/omapdss-boot-init.o' failed > > > make[7]: *** [drivers/video/fbdev/omap2/omapfb/dss/omapdss-boot-init.o] Error 1 > > > scripts/Makefile.build:559: recipe for target 'drivers/video/fbdev/omap2/omapfb/dss' failed > > > make[6]: *** [drivers/video/fbdev/omap2/omapfb/dss] Error 2 > > > make[6]: *** Waiting for unfinished jobs.... > > > > > > Add OF dependency in order to make all configurations work again. > > > > > > of_update_property() has no inline stub, and that that could be added as > > > an alternative. > > > > > > Signed-off-by: Anders Roxell > > > --- > > > drivers/video/fbdev/omap2/Kconfig | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/drivers/video/fbdev/omap2/Kconfig b/drivers/video/fbdev/omap2/Kconfig > > > index 82008699d253..4de381f2452e 100644 > > > --- a/drivers/video/fbdev/omap2/Kconfig > > > +++ b/drivers/video/fbdev/omap2/Kconfig > > > @@ -1,4 +1,4 @@ > > > -if ARCH_OMAP2PLUS || COMPILE_TEST > > > +if OF && (ARCH_OMAP2PLUS || COMPILE_TEST) > > > > > > source "drivers/video/fbdev/omap2/omapfb/Kconfig" > > > > > > > > > > Hi, > > > > This patch certainly fixes the build errors that I have been seeing in > > linux-next for several days now, so: > > > > Acked-by: Randy Dunlap > > Tested-by: Randy Dunlap > > Thanks. > > Since there has been no follow up from Mauro on this ([1]) I guess > that the best thing I can do is simply merging both patches to fbdev > tree.. > > [1] https://www.spinics.net/lists/linux-media/msg132583.html Sorry, it seems I missed it. If you didn't merge them yet, I'll merge it on my tree today, if you think it would be best to have it together with the patch that enabled FB_OMAP builds with COMPILE_TEST. If you prefer, instead, feel free to merge via your tree, as this is FB stuff anyway. If you need/want: Acked-by: Mauro Carvalho Chehab Thanks, Mauro