From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753878AbbCJPby (ORCPT ); Tue, 10 Mar 2015 11:31:54 -0400 Received: from mga09.intel.com ([134.134.136.24]:41126 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752091AbbCJPbv (ORCPT ); Tue, 10 Mar 2015 11:31:51 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.11,375,1422950400"; d="scan'208";a="689852641" From: Jani Nikula To: Archit Taneja , robdclark@gmail.com, airlied@linux.ie, treding@nvidia.com, p.zabel@pengutronix.de, benjamin.gaignard@linaro.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, daniel@ffwll.ch Subject: Re: [RFC 1/6] drm: Add top level Kconfig option for DRM fbdev emulation In-Reply-To: <54FEC5F1.90208@codeaurora.org> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo References: <1425980493-27533-1-git-send-email-architt@codeaurora.org> <1425980493-27533-2-git-send-email-architt@codeaurora.org> <20150310094756.GE3800@phenom.ffwll.local> <54FEBEF1.5000103@codeaurora.org> <20150310100547.GG3800@phenom.ffwll.local> <54FEC5F1.90208@codeaurora.org> User-Agent: Notmuch/0.19+72~gd7c02cb (http://notmuchmail.org) Emacs/24.4.1 (x86_64-pc-linux-gnu) Date: Tue, 10 Mar 2015 17:33:13 +0200 Message-ID: <87zj7krhom.fsf@intel.com> MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 10 Mar 2015, Archit Taneja wrote: > On 03/10/2015 03:35 PM, Daniel Vetter wrote: >> On Tue, Mar 10, 2015 at 03:22:49PM +0530, Archit Taneja wrote: >>> >>> >>> On 03/10/2015 03:17 PM, Daniel Vetter wrote: >>>> On Tue, Mar 10, 2015 at 03:11:28PM +0530, Archit Taneja wrote: >>> DRM_KMS_FB_HELPER selects that for us, right? >> >> Hm right I've missed that. Reminds me that you need one more patch at the >> end to remove all the various select DRM_KMS_FB_HELPER from all drm >> drivers. Otherwise this knob here won't work by default if you e.g. select >> radeon. In general we can't mix explicit options with menu entries with a >> select. > > I was trying that out. Removing DRM_KMS_FB_HELPER and having > DRM_FBDEV_EMULATION disabled breaks drivers which use FB stuff > internally in their respective xyz_fbdev.c files. > > Are you saying that we should remove DRM_KMS_FB_HELPER for such drivers > and replace them with 'select DRM_FBDEV_EMULATION'? Slightly tangential: As a rule of thumb, you should not "select" anything that is visible in menuconfig or has dependencies [1]. This will break the build eventually, and attempts to fix it are troublesome [2]. BR, Jani. [1] Documentation/kbuild/kconfig-language.txt [2] http://mid.gmane.org/1413580403-16225-1-git-send-email-jani.nikula@intel.com -- Jani Nikula, Intel Open Source Technology Center