All of lore.kernel.org
 help / color / mirror / Atom feed
From: Aaron Sowry <aaron+nv-+1tCnOwpXBmzQB+pC5nmwQ@public.gmane.org>
To: Calvin Walton <calvin.walton-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
Subject: Re: nouveau exposes backlight controls in presence of	ACPI
Date: Tue, 2 Nov 2010 20:28:02 +0100	[thread overview]
Message-ID: <20101102192802.GA4197@aeneby.se> (raw)
In-Reply-To: <1288715822.11373.6.camel@ayu>


[-- Attachment #1.1: Type: text/plain, Size: 1853 bytes --]

> It was my understanding that the future direction the kernel was going
> to go with this is that all detected backlight devices would be
> exported, with a tag in sysfs describing the type of interface
> (platform, firmware, direct hardware, whichever). Then it would be up to
> userspace to pick the correct backlight device to use based on the tags.
> Nothing stopping a user from manipulating a different one directly, with
> possible interesting side-effects, of course.

Interesting. I guess this would be the best long-term solution, but short-term we can't proceed based on speculation. Nor can we really trust userspace to make the correct decisions at the moment, as the controls don't normally provide enough information to be able to do that reliably.

> Note that the thinkpad-acpi module at the moment is also checking for
> the ACPI video device to be loaded, and will disable itself in that
> case. So the case that causes problems right now is if you have e.g.
> both thinkpad-acpi and nouveau, but no ACPI video device?

The problem in my case is when both ACPI and nouveau controls are exposed - gnome-power-manager prioritises the nouveau control over ACPI (I've raised a bug for this too) and because my card isn't supported properly by the driver, I get a range of approximately useless brightness levels. As I see it, we have 3 related issues when it comes to backlight control:

0) Fixing NV50 backlight support in nouveau for those chipsets with max_brightness > 1025
1) Determining under which circumstances we should expose kernel-level controls (IMHO only when no better alternatives are available)
2) Determining how userspace should select an appropriate control when more than one is available

Matthew Garrett addresses some of those issues in a fairly decent rant here:
http://mjg59.livejournal.com/127103.html

/Aaron

[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 489 bytes --]

[-- Attachment #2: Type: text/plain, Size: 181 bytes --]

_______________________________________________
Nouveau mailing list
Nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
http://lists.freedesktop.org/mailman/listinfo/nouveau

      reply	other threads:[~2010-11-02 19:28 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-10-29 15:49 nouveau exposes backlight controls in presence of ACPI Aaron Sowry
     [not found] ` <20101029154934.GA2076-+1tCnOwpXBmzQB+pC5nmwQ@public.gmane.org>
2010-11-02  3:58   ` Ben Skeggs
2010-11-02 14:51     ` Aaron Sowry
     [not found]       ` <20101102145153.GA4075-+1tCnOwpXBmzQB+pC5nmwQ@public.gmane.org>
2010-11-02 22:37         ` Ben Skeggs
2010-11-02 16:37     ` Calvin Walton
2010-11-02 19:28       ` Aaron Sowry [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20101102192802.GA4197@aeneby.se \
    --to=aaron+nv-+1tcnowpxbmzqb+pc5nmwq@public.gmane.org \
    --cc=calvin.walton-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.