From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hans de Goede Date: Thu, 22 May 2014 08:41:44 +0000 Subject: Re: [PATCH resend 1/4] nouveau: Don't check acpi_video_backlight_support() before registering backli Message-Id: <537DB848.3000203@redhat.com> List-Id: References: <1400679596-19663-1-git-send-email-hdegoede@redhat.com> <1400679596-19663-2-git-send-email-hdegoede@redhat.com> <2450872.0pLqjZcv9y@vostro.rjw.lan> In-Reply-To: <2450872.0pLqjZcv9y@vostro.rjw.lan> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: "Rafael J. Wysocki" Cc: Aaron Lu , Jingoo Han , Bryan Wu , Lee Jones , Jean-Christophe Plagniol-Villard , Tomi Valkeinen , Ben Skeggs , David Airlie , Zhang Rui , Len Brown , linux-acpi@vger.kernel.org, linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org Hi, On 05/22/2014 01:30 AM, Rafael J. Wysocki wrote: > On Wednesday, May 21, 2014 03:39:53 PM Hans de Goede wrote: >> acpi_video_backlight_support() is supposed to be called by other (vendor >> specific) firmware backlight controls, not by native / raw backlight controls >> like nv_backlight. >> >> Userspace will normally prefer firmware interfaces over raw interfaces, so >> if acpi_video backlight support is present it will use that even if >> nv_backlight is registered as well. >> >> Except when video.use_native_backlight is present on the kernel cmdline >> (or enabled through a dmi based quirk). As the name indicates the goal here >> is to make only the raw interface available to userspace so that it will use >> that (it only does this when it sees a win8 compliant bios). >> >> This is done by: >> 1) Not registering any acpi_video# backlight devices; and >> 2) Making acpi_video_backlight_support() return true so that other firmware >> drivers, ie acer_wmi, thinkpad_acpi, dell_laptop, etc. Don't register their >> own vender specific interfaces. >> >> Currently nouveau breaks this setup, as when acpi_video_backlight_support() >> returns true, it does not register itself, resulting in no backlight control >> at all. >> >> This is esp. going to be a problem with 3.16 which will default to >> video.use_native_backlight=1, and thus nouveau based laptops with a win8 bios >> will get no backlight control at all. >> >> This also likely explains why the previous attempt to make >> video.use_native_backlight=1 the default was not a success, as without this >> patch having a default of video.use_native_backlight=1 will cause regressions. >> >> Note this effectively reverts commit 5bead799 >> >> Also see: https://bugzilla.redhat.com/show_bug.cgi?id93171 >> >> Signed-off-by: Hans de Goede > > It would be good to have an ACK from the nouveau people for this one. Right, it could / should even go in through the drm tree I guess. Regards, Hans