From patchwork Thu Oct 2 07:46:52 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jani Nikula X-Patchwork-Id: 5016671 Return-Path: X-Original-To: patchwork-intel-gfx@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork1.web.kernel.org (Postfix) with ESMTP id 9F09D9FC0B for ; Thu, 2 Oct 2014 07:47:01 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id B2D8A20279 for ; Thu, 2 Oct 2014 07:47:00 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) by mail.kernel.org (Postfix) with ESMTP id 88F9A20270 for ; Thu, 2 Oct 2014 07:46:59 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 267716E473; Thu, 2 Oct 2014 00:46:58 -0700 (PDT) X-Original-To: intel-gfx@lists.freedesktop.org Delivered-To: intel-gfx@lists.freedesktop.org Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by gabe.freedesktop.org (Postfix) with ESMTP id 88DBF6E457; Thu, 2 Oct 2014 00:46:56 -0700 (PDT) Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga103.jf.intel.com with ESMTP; 02 Oct 2014 00:44:36 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.04,637,1406617200"; d="scan'208";a="582332856" Received: from jnikula-mobl1.fi.intel.com (HELO localhost) ([10.237.72.185]) by orsmga001.jf.intel.com with ESMTP; 02 Oct 2014 00:46:53 -0700 From: Jani Nikula To: Randy Dunlap , Jim Davis , Stephen Rothwell , linux-next , linux-kernel , David Airlie , daniel.vetter@intel.com, intel-gfx@lists.freedesktop.org, dri-devel In-Reply-To: <542C46B3.2020901@infradead.org> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo References: <87fvf73c6g.fsf@intel.com> <542C46B3.2020901@infradead.org> User-Agent: Notmuch/0.18.1+110~gd9ec63c (http://notmuchmail.org) Emacs/23.4.1 (x86_64-pc-linux-gnu) Date: Thu, 02 Oct 2014 10:46:52 +0300 Message-ID: <87eguq6hhv.fsf@intel.com> MIME-Version: 1.0 Subject: Re: [Intel-gfx] randconfig build error with next-20141001, in drivers/gpu/drm/i915 X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" X-Spam-Status: No, score=-5.2 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_MED, RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP On Wed, 01 Oct 2014, Randy Dunlap wrote: > On 10/01/14 10:57, Jani Nikula wrote: >> On Wed, 01 Oct 2014, Jim Davis wrote: >>> Building with the attached random configuration file, >>> >>> warning: (VIDEO_TIMBERDALE) selects TIMB_DMA which has unmet direct >>> dependencies (DMADEVICES && MFD_TIMBERDALE) >>> warning: (USB_OTG_FSM && FSL_USB2_OTG && USB_MV_OTG) selects USB_OTG >>> which has unmet direct dependencies (USB_SUPPORT && USB && PM_RUNTIME) >>> >>> drivers/built-in.o: In function `intel_panel_setup_backlight': >>> (.text+0x11855d): undefined reference to `backlight_device_register' >>> drivers/built-in.o: In function `intel_panel_destroy_backlight': >>> (.text+0x118624): undefined reference to `backlight_device_unregister' >>> make: *** [vmlinux] Error 1 >> >> It's the combination: >> >>> CONFIG_DRM_I915=y >>> CONFIG_BACKLIGHT_CLASS_DEVICE=m >> >> Not sure what the Kconfig magic spell needs to be when we don't want to >> depend on CONFIG_BACKLIGHT_CLASS_DEVICE but we'll use it if it's enabled >> (y or m). > > Something like: > > depends on BACKLIGHT_CLASS_DEVICE || BACKLIGHT_CLASS_DEVICE=n > > is what we usually use. Oh boy this gets ugly. Trying leads to more fun scripts/kconfig/conf --olddefconfig Kconfig drivers/video/fbdev/Kconfig:5:error: recursive dependency detected! drivers/video/fbdev/Kconfig:5: symbol FB is selected by DRM_KMS_FB_HELPER drivers/gpu/drm/Kconfig:34: symbol DRM_KMS_FB_HELPER is selected by DRM_I915_FBDEV drivers/gpu/drm/i915/Kconfig:40: symbol DRM_I915_FBDEV depends on DRM_I915 drivers/gpu/drm/i915/Kconfig:1: symbol DRM_I915 depends on ACPI_VIDEO drivers/acpi/Kconfig:130: symbol ACPI_VIDEO is selected by BACKLIGHT_CLASS_DEVICE drivers/video/backlight/Kconfig:156: symbol BACKLIGHT_CLASS_DEVICE is selected by FB_BACKLIGHT drivers/video/fbdev/Kconfig:188: symbol FB_BACKLIGHT is selected by PMAC_BACKLIGHT drivers/macintosh/Kconfig:135: symbol PMAC_BACKLIGHT depends on FB and the rabbit hole gets deeper than I thought. I guess it always does. At this point the question is whether "depends on ((ACPI && ACPI_VIDEO) || ACPI=n)" is worth pursuing and makes more sense than the "select foo if ACPI" lines, or do we need to build hacks upon hacks to get this fixed. BR, Jani. diff --git a/drivers/gpu/drm/i915/Kconfig b/drivers/gpu/drm/i915/Kconfig index 4e39ab34eb1c..daa453c62aa6 100644 --- a/drivers/gpu/drm/i915/Kconfig +++ b/drivers/gpu/drm/i915/Kconfig @@ -3,6 +3,7 @@ config DRM_I915 depends on DRM depends on X86 && PCI depends on (AGP || AGP=n) + depends on (BACKLIGHT_CLASS_DEVICE || BACKLIGHT_CLASS_DEVICE=n) select INTEL_GTT select AGP_INTEL if AGP select INTERVAL_TREE leads to scripts/kconfig/conf --olddefconfig Kconfig drivers/gpu/drm/i915/Kconfig:1:error: recursive dependency detected! drivers/gpu/drm/i915/Kconfig:1: symbol DRM_I915 depends on BACKLIGHT_CLASS_DEVICE drivers/video/backlight/Kconfig:156: symbol BACKLIGHT_CLASS_DEVICE is selected by DRM_I915 because we depend on ACPI_VIDEO when ACPI is enabled, originally done in commit ecb4aed78dcf09e48c8c34c8c2fa7f5c69344be6 Author: Len Brown Date: Fri Apr 24 11:33:47 2009 -0400 ACPI, i915: build fix Now, trying to fix that with what seems to me like the obvious dependency, conveying what we really want diff --git a/drivers/gpu/drm/i915/Kconfig b/drivers/gpu/drm/i915/Kconfig index 4e39ab34eb1c..f5a1ee37603e 100644 --- a/drivers/gpu/drm/i915/Kconfig +++ b/drivers/gpu/drm/i915/Kconfig @@ -3,6 +3,7 @@ config DRM_I915 depends on DRM depends on X86 && PCI depends on (AGP || AGP=n) + depends on ((ACPI && ACPI_VIDEO) || ACPI=n) select INTEL_GTT select AGP_INTEL if AGP select INTERVAL_TREE @@ -11,13 +12,6 @@ config DRM_I915 select SHMEM select TMPFS select DRM_KMS_HELPER - # i915 depends on ACPI_VIDEO when ACPI is enabled - # but for select to work, need to select ACPI_VIDEO's dependencies, ick - select BACKLIGHT_LCD_SUPPORT if ACPI - select BACKLIGHT_CLASS_DEVICE if ACPI - select INPUT if ACPI - select ACPI_VIDEO if ACPI - select ACPI_BUTTON if ACPI help Choose this option if you have a system that has "Intel Graphics Media Accelerator" or "HD Graphics" integrated graphics,