From: Fabio Estevam Subject: [PATCH] drm: drm_drv: Only call drm_agp_clear() in __OS_HAS_AGP case Date: Thu, 8 Aug 2013 19:00:10 -0300 Message-ID: <1375999210-26044-1-git-send-email-festevam@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-gh0-f174.google.com (mail-gh0-f174.google.com [209.85.160.174]) by gabe.freedesktop.org (Postfix) with ESMTP id 3E39FE5D0E for ; Thu, 8 Aug 2013 15:00:21 -0700 (PDT) Received: by mail-gh0-f174.google.com with SMTP id r17so688737ghr.19 for ; Thu, 08 Aug 2013 15:00:20 -0700 (PDT) List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org Errors-To: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org To: airlied@linux.ie Cc: Fabio Estevam , dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org From: Fabio Estevam 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 --- 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