linux-kernel - Re: [PATCH v4 0/2] Add driver for Apple gmux device

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] [day] [month] [year] [list]

Message-ID: <20120321143842.GB23643@thinkpad-t410>
Date:	Wed, 21 Mar 2012 09:38:42 -0500
From:	Seth Forshee <seth.forshee@...onical.com>
To:	Shea Levy <shea@...alevy.com>
Cc:	Matthew Garrett <mjg@...hat.com>,
	platform-driver-x86@...r.kernel.org,
	Richard Purdie <rpurdie@...ys.net>,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v4 0/2] Add driver for Apple gmux device

On Wed, Mar 21, 2012 at 08:36:47AM -0400, Shea Levy wrote:
> Hi Seth,
> 
> On 03/16/2012 03:41 PM, Seth Forshee wrote:
> >Apple laptops with hybrid graphics have a device named gmux that is used
> >for switching between GPUs and backlight control. On many models this is
> >the only reliable method for controlling the backlight.
> >
> >This series adds initial support for the gmux device, along with
> >anciallary support for disabling apple_bl when the gmux device is
> >detected. Initially only backlight control is supported.
> >
> >
> 
> I've done some basic testing on my MBP 5,3 (just setting a few
> brightness levels, nothing fancy), and it all works, so feel free to
> add my basic
> 
> Tested-by: Shea Levy <shea@...alevy.com>
> 
> if you wish. Please let me know if you need testing for future gmux
> features.

Thanks for testing! Matthew has already merged the patch to his tree
though, so I won't be able to add your tag.

> A few questions:
>     * I'm using nouveau, and with the gmux I have both
> gmux_backlight and nv_backlight (which never worked) in
> /sys/class/backlight, is this expected?

That's a better question for the nouveau developers. nouveau has some
criteria it uses to determine whether or not it should supply a
backlight device, and apparently that's all passing even though the
backlight control doesn't work.

>     * Is there anything I can do to test the backlight interface in
> addition to setting brightness levels?

The testing you've already done is sufficient.

>     * Besides the sysfs backlight interface, are there any other
> userspace-visible changes your driver makes that I should test?

Not at this time, no.

Thanks,
Seth

--
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/