diff mbox series

[7/8] drm/bridge: fix dependency for lvds-encoder

Message ID 20180525155030.3667352-7-arnd@arndb.de
State New
Headers show
Series [1/8] drm/sun4i: mark PM functions as __maybe_unused | expand

Commit Message

Arnd Bergmann May 25, 2018, 3:50 p.m. UTC
The DRM panel bridge code is built into the kms helpers module, so we
get a link error when trying to use it from a built-in driver while the
kms helper is a loadable module:

drivers/gpu/drm/bridge/lvds-encoder.o: In function `lvds_encoder_probe':
lvds-encoder.c:(.text+0x124): undefined reference to `devm_drm_panel_bridge_add'

This adds a the same dependency in the lvds-encoder that we use for all
the other users of the panel bridge. I did not bisect the problem, but
from inspection it seems to date back to the patch that separated out
the panel bridge from lvds encoder.

Fixes: 13dfc0540a57 ("drm/bridge: Refactor out the panel wrapper from the lvds-encoder bridge.")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>

---
 drivers/gpu/drm/bridge/Kconfig | 1 +
 1 file changed, 1 insertion(+)

-- 
2.9.0

Comments

Linus Walleij May 28, 2018, 7:47 a.m. UTC | #1
On Fri, May 25, 2018 at 5:50 PM, Arnd Bergmann <arnd@arndb.de> wrote:

> The DRM panel bridge code is built into the kms helpers module, so we

> get a link error when trying to use it from a built-in driver while the

> kms helper is a loadable module:

>

> drivers/gpu/drm/bridge/lvds-encoder.o: In function `lvds_encoder_probe':

> lvds-encoder.c:(.text+0x124): undefined reference to `devm_drm_panel_bridge_add'

>

> This adds a the same dependency in the lvds-encoder that we use for all

> the other users of the panel bridge. I did not bisect the problem, but

> from inspection it seems to date back to the patch that separated out

> the panel bridge from lvds encoder.

>

> Fixes: 13dfc0540a57 ("drm/bridge: Refactor out the panel wrapper from the lvds-encoder bridge.")

> Signed-off-by: Arnd Bergmann <arnd@arndb.de>


Reviewed-by: Linus Walleij <linus.walleij@linaro.org>


Yours,
Linus Walleij
diff mbox series

Patch

diff --git a/drivers/gpu/drm/bridge/Kconfig b/drivers/gpu/drm/bridge/Kconfig
index 6caa47834194..cf47bfa7a050 100644
--- a/drivers/gpu/drm/bridge/Kconfig
+++ b/drivers/gpu/drm/bridge/Kconfig
@@ -46,6 +46,7 @@  config DRM_DUMB_VGA_DAC
 config DRM_LVDS_ENCODER
 	tristate "Transparent parallel to LVDS encoder support"
 	depends on OF
+	select DRM_KMS_HELPER
 	select DRM_PANEL_BRIDGE
 	help
 	  Support for transparent parallel to LVDS encoders that don't require