From mboxrd@z Thu Jan 1 00:00:00 1970 From: Borislav Petkov Subject: Re: [PATCH v2] drm/nouveau: always select ACPI_VIDEO if ACPI is enabled. Date: Tue, 5 Feb 2013 17:22:06 +0100 Message-ID: <20130205162206.GJ4827@pd.tnic> References: <20130204152344.GG13909@pd.tnic> <510FD6A2.80101@canonical.com> <20130205145232.GI4827@pd.tnic> <5111277B.5020607@canonical.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Return-path: Content-Disposition: inline In-Reply-To: <5111277B.5020607@canonical.com> Sender: linux-kernel-owner@vger.kernel.org To: Maarten Lankhorst Cc: dri-devel@lists.freedesktop.org, lkml , Michal Marek List-Id: dri-devel@lists.freedesktop.org On Tue, Feb 05, 2013 at 04:38:35PM +0100, Maarten Lankhorst wrote: > Argh, next attempt, based on i915's Kconfig. > > It seems that not only I have to select ACPI_VIDEO, I also have to select all the dependencies. > Is this a Kconfig bug or working as intended? i915 seems to have a workaround, so I copied it from > there. Except it's currently missing select THERMAL, so I guess it didn't get updated when that got > added. > > ---->8 > Having nouveau builtin would still allow ACPI_VIDEO to be used as external module > if some of the deps for acpi_video have not been met, which would result in a linking > failure. Solve this by selecting all dependencies as well. > > Signed-off-by: Maarten Lankhorst Yep, this takes care of all deps, Tested-by: Borislav Petkov Thanks. -- Regards/Gruss, Boris. Sent from a fat crate under my desk. Formatting is fine. --