linux-kernel - Re: mbp_nvidia_bl: Remove DMI dependency (was: Re: [PATCH,RESEND] backlight: mbp_nvidia_bl

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <AANLkTinovDucigOX+kidh+KwekbZP65moy7P2pYNn25L@mail.gmail.com>
Date:	Tue, 11 Jan 2011 17:05:11 +0000
From:	Richard Hughes <hughsient@...il.com>
To:	Matthew Garrett <mjg59@...f.ucam.org>
Cc:	Mourad De Clerck <mourad@...azul.com>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Richard Purdie <rpurdie@...ux.intel.com>,
	linux-kernel@...r.kernel.org, Richard Hughes <richard@...hsie.com>
Subject: Re: mbp_nvidia_bl: Remove DMI dependency (was: Re: [PATCH,RESEND]
 backlight: mbp_nvidia_bl - add support for MacBookPro7,1)

On 11 January 2011 16:48, Matthew Garrett <mjg59@...f.ucam.org> wrote:
> On Tue, Jan 11, 2011 at 05:42:12PM +0100, Mourad De Clerck wrote:
>> A bit late, but I tested this patch series and it works for me.
>>
>> The only slight change is that gnome-power-manager needs to be updated
>> to use /sys/class/backlight/apple_backlight instead of mbp_backlight.
>
> gnome-power-manager needs not to have a hardcoded list of backlights.

It's only uses the PolicyKit-enabled utility if there is no XBACKLIGHT
driver support. The list is put there as for a long time people had
multiple backlights, where, for instance thinkpad_screen worked but
not acpi_video1.

If the named backlight file isn't present in the override list, then
we just use the first one in /sys/class/backlight

Mourad, how many devices do you have in /sys/class/backlight?

Richard.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/