diff mbox

drm: drm_drv: Only call drm_agp_clear() in __OS_HAS_AGP case

Message ID 1375999210-26044-1-git-send-email-festevam@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Fabio Estevam Aug. 8, 2013, 10 p.m. UTC
From: Fabio Estevam <fabio.estevam@freescale.com>

Commit 28ec711 (drm/agp: move AGP cleanup paths to drm_agpsupport.c) causes the 
following link error on ARM (imx_v6_v7_defconfig):

drivers/built-in.o: In function `drm_lastclose':
:(.text+0x588a0): undefined reference to `drm_agp_clear'
make: *** [vmlinux] Error 1

drm_agp_clear() is defined only '#if __OS_HAS_AGP' is true, so add this 'if'
logic when calling it.

Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
 drivers/gpu/drm/drm_drv.c | 2 ++
 1 file changed, 2 insertions(+)

Comments

David Herrmann Aug. 12, 2013, 10:26 a.m. UTC | #1
Hi

On Fri, Aug 9, 2013 at 12:00 AM, Fabio Estevam <festevam@gmail.com> wrote:
> From: Fabio Estevam <fabio.estevam@freescale.com>
>
> Commit 28ec711 (drm/agp: move AGP cleanup paths to drm_agpsupport.c) causes the
> following link error on ARM (imx_v6_v7_defconfig):
>
> drivers/built-in.o: In function `drm_lastclose':
> :(.text+0x588a0): undefined reference to `drm_agp_clear'
> make: *** [vmlinux] Error 1
>
> drm_agp_clear() is defined only '#if __OS_HAS_AGP' is true, so add this 'if'
> logic when calling it.
>
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>

This has already been fixed 4 days ago in the drm-next tree. You can
find the patch here:
  http://cgit.freedesktop.org/~airlied/linux/commit/?h=drm-next&id=00fd78e5279aec3aa504307ff2db892d3efb555d

Thanks
David

> ---
>  drivers/gpu/drm/drm_drv.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/drivers/gpu/drm/drm_drv.c b/drivers/gpu/drm/drm_drv.c
> index dddd799..58f7f39 100644
> --- a/drivers/gpu/drm/drm_drv.c
> +++ b/drivers/gpu/drm/drm_drv.c
> @@ -195,7 +195,9 @@ int drm_lastclose(struct drm_device * dev)
>
>         mutex_lock(&dev->struct_mutex);
>
> +#if __OS_HAS_AGP
>         drm_agp_clear(dev);
> +#endif
>
>         if (drm_core_check_feature(dev, DRIVER_SG) && dev->sg &&
>             !drm_core_check_feature(dev, DRIVER_MODESET)) {
> --
> 1.8.1.2
>
diff mbox

Patch

diff --git a/drivers/gpu/drm/drm_drv.c b/drivers/gpu/drm/drm_drv.c
index dddd799..58f7f39 100644
--- a/drivers/gpu/drm/drm_drv.c
+++ b/drivers/gpu/drm/drm_drv.c
@@ -195,7 +195,9 @@  int drm_lastclose(struct drm_device * dev)
 
 	mutex_lock(&dev->struct_mutex);
 
+#if __OS_HAS_AGP
 	drm_agp_clear(dev);
+#endif
 
 	if (drm_core_check_feature(dev, DRIVER_SG) && dev->sg &&
 	    !drm_core_check_feature(dev, DRIVER_MODESET)) {