Thanks Sam, I've meanwhile merged the patch with our R-b.

Am 26.10.21 um 20:46 schrieb Sam Ravnborg:
Hi Thomas,

On Tue, Oct 26, 2021 at 07:57:00PM +0200, Thomas Zimmermann wrote:
Linking the CMA frambuffer helpers into a CMA helper library in
commit 4b2b5e142ff4 ("drm: Move GEM memory managers into modules")
results in linker errors:

   arm-linux-gnueabihf-ld: drivers/gpu/drm/drm_fb_cma_helper.o: \
          in function `drm_fb_cma_get_gem_obj': \
          drivers/gpu/drm/drm_fb_cma_helper.c:46: undefined reference \
          to `drm_gem_fb_get_obj'
   arm-linux-gnueabihf-ld: drivers/gpu/drm/drm_fb_cma_helper.c:46: \
          undefined reference to `drm_gem_fb_get_obj'
   arm-linux-gnueabihf-ld: drivers/gpu/drm/drm_fb_cma_helper.c:46: \
          undefined reference to `drm_gem_fb_get_obj'
   arm-linux-gnueabihf-ld: drivers/gpu/drm/drm_fb_cma_helper.o: in \
          function `drm_fb_cma_sync_non_coherent': \
          drivers/gpu/drm/drm_fb_cma_helper.c:133: undefined reference \
          to `drm_atomic_helper_damage_iter_init'
   arm-linux-gnueabihf-ld: drivers/gpu/drm/drm_fb_cma_helper.c:135: \
          undefined reference to `drm_atomic_helper_damage_iter_next'

Link the CMA framebuffer helpers into the KMS helper library to
fix the problem.

Signed-off-by: Thomas Zimmermann <[email protected]>
Fixes: 4b2b5e142ff4 ("drm: Move GEM memory managers into modules")
Reported-by: Naresh Kamboju <[email protected]>
Reported-by: Linux Kernel Functional Testing <[email protected]>
Cc: Daniel Vetter <[email protected]>
Cc: Maarten Lankhorst <[email protected]>
Cc: Maxime Ripard <[email protected]>
Cc: David Airlie <[email protected]>
Cc: Daniel Vetter <[email protected]>
Cc: [email protected]

Patch looks fine,
Reviewed-by: Sam Ravnborg <[email protected]>

        Sam


--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Maxfeldstr. 5, 90409 Nürnberg, Germany
(HRB 36809, AG Nürnberg)
Geschäftsführer: Felix Imendörffer

Attachment: OpenPGP_signature
Description: OpenPGP digital signature

Reply via email to