Skip to content

Conversation

@MaxKrummenacher
Copy link
Contributor

When using imx_v8_defconfig but changing CONFIG_DRM_IMX95_LDB=m the build fails as the module cannot link against the functions provided by imx-ldb-helper.

Fix that by reverting the revert of the mainline fix for this.

ERROR: modpost: missing MODULE_LICENSE() in drivers/gpu/drm/bridge/imx/imx-ldb-helper.o
ERROR: modpost: "ldb_remove_bridge_helper" [drivers/gpu/drm/bridge/imx/imx95-ldb.ko] undefined!
ERROR: modpost: "ldb_bridge_atomic_check_helper" [drivers/gpu/drm/bridge/imx/imx95-ldb.ko] undefined!
ERROR: modpost: "ldb_channel_is_split_link" [drivers/gpu/drm/bridge/imx/imx95-ldb.ko] undefined!
ERROR: modpost: "ldb_bridge_mode_set_helper" [drivers/gpu/drm/bridge/imx/imx95-ldb.ko] undefined!
ERROR: modpost: "ldb_channel_is_single_link" [drivers/gpu/drm/bridge/imx/imx95-ldb.ko] undefined!
ERROR: modpost: "ldb_init_helper" [drivers/gpu/drm/bridge/imx/imx95-ldb.ko] undefined!
ERROR: modpost: "ldb_find_next_bridge_helper" [drivers/gpu/drm/bridge/imx/imx95-ldb.ko] undefined!
ERROR: modpost: "ldb_add_bridge_helper" [drivers/gpu/drm/bridge/imx/imx95-ldb.ko] undefined!
ERROR: modpost: "ldb_bridge_disable_helper" [drivers/gpu/drm/bridge/imx/imx95-ldb.ko] undefined!
ERROR: modpost: "ldb_bridge_enable_helper" [drivers/gpu/drm/bridge/imx/imx95-ldb.ko] undefined!
WARNING: modpost: suppressed 1 unresolved symbol warnings because there were too many)

commit 7eb790e ("Revert "drm/bridge: imx: fix mixed module-builtin
object"") reverted that the imx-ldb-helper can be used as module and
with later commits stopped using the file in downstream for i.MX
8QXP/8QM).

The commit 0b608ae ("LF-10583-2: drm: bridge: imx: Add i.MX95 LVDS
Display Bridge(LDB) driver") used the file again for i.MX 95 without
readding the posibility to use it as a module.

Revert the Revert so that imx-ldb-helper can be used from a module
independent of it being built as a module or built in.

Signed-off-by: Max Krummenacher <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant