Messages in this thread | | | Subject | Re: [PATCH] drivers: omap2: Kconfig: make FB_OMAP2_DSS_INIT depend on OF | From | Randy Dunlap <> | Date | Thu, 3 May 2018 08:48:56 -0700 |
| |
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 <anders.roxell@linaro.org> > --- > 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 <rdunlap@infradead.org> Tested-by: Randy Dunlap <rdunlap@infradead.org>
OTOH, I would rather see the Kconfig dependencies all expressed in drivers/video/fbdev/omap2/omapfb/Kconfig instead of them being split between 2 Kconfig files.
thanks, -- ~Randy
| |