Hi Uwe, On Fri, Jun 28, 2013 at 4:29 AM, Uwe Kleine-König <u.kleine-koe...@pengutronix.de> wrote: > On Fri, Jun 28, 2013 at 12:32:01AM -0300, Fabio Estevam wrote: >> From: Fabio Estevam <fabio.este...@freescale.com> >> >> When building imx_v6_v7_defconfig with imx-drm drivers selected as modules, >> we >> get the following build error: >> >> ERROR: "imx_drm_encoder_get_mux_id" [drivers/staging/imx-drm/imx-ldb.ko] >> undefined! > So imx-ldb.c needs a symbol from imx-drm-core.c right? The alternative > to exporting this symbol for a single user(?) is to link these two files > into a single module. Does this make sense? If yes, that would IMHO be > the better fix.
The Makefile for these files is: imxdrm-objs := imx-drm-core.o imx-fb.o obj-$(CONFIG_DRM_IMX) += imxdrm.o obj-$(CONFIG_DRM_IMX_PARALLEL_DISPLAY) += parallel-display.o obj-$(CONFIG_DRM_IMX_TVE) += imx-tve.o obj-$(CONFIG_DRM_IMX_LDB) += imx-ldb.o obj-$(CONFIG_DRM_IMX_FB_HELPER) += imx-fbdev.o obj-$(CONFIG_DRM_IMX_IPUV3_CORE) += ipu-v3/ obj-$(CONFIG_DRM_IMX_IPUV3) += ipuv3-crtc.o so imx-drm-core is always built and imx-ldb is only built when CONFIG_DRM_IMX_LDB is selected. In this case, we should keep the two modules separate and fix this error using the export. Regards, Fabio Estevam -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
