From: Matthew Garrett <mjg59@srcf.ucam.org>
To: Pedro Ribeiro <pedrib@gmail.com>
Cc: Kamal Mostafa <kamal@canonical.com>,
linux-acpi@vger.kernel.org, intel-gfx@lists.freedesktop.org
Subject: Re: [Intel-gfx] [RFC PATCH 0/2] i915 brightness control
Date: Thu, 3 Jun 2010 15:19:41 +0100 [thread overview]
Message-ID: <20100603141941.GB768@srcf.ucam.org> (raw)
In-Reply-To: <AANLkTiktSDWqHg_A8V5_jQsCf1w4k5x9Q7ZOCjAIZTEH@mail.gmail.com>
On Thu, Jun 03, 2010 at 12:20:39AM +0100, Pedro Ribeiro wrote:
> Are you sure this is a good idea to enable it by default just because
> of a problem with one manufacturer (Dell)?
It should (a-ha ha) be entirely safe on systems using the IGD opregion
spec.
> My Lenovo laptop has fine brightness control with a GM45 and this may break it.
> Might be safer only to enable it for laptops with known problems.
You have a complete list of all laptops with problems in this area? The
ongoing situation with Windows 7 compatibility breaking machines that
previously worked suggests that there's something going on which we
don't understand or implement.
--
Matthew Garrett | mjg59@srcf.ucam.org
next prev parent reply other threads:[~2010-06-03 14:19 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-06-02 22:11 [RFC PATCH 0/2] i915 brightness control Kamal Mostafa
2010-06-02 22:11 ` [RFC PATCH 1/2] acpi/video: acpi_brightness_hook API Kamal Mostafa
2010-06-02 22:11 ` [RFC PATCH 2/2] drm/i915: override acpi brightness control Kamal Mostafa
2010-06-02 23:20 ` [Intel-gfx] [RFC PATCH 0/2] i915 " Pedro Ribeiro
2010-06-03 14:19 ` Matthew Garrett [this message]
2010-06-04 19:47 ` Jerone Young
2010-09-13 17:48 ` Matthew Garrett
2010-09-24 19:12 ` Kamal Mostafa
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=20100603141941.GB768@srcf.ucam.org \
--to=mjg59@srcf.ucam.org \
--cc=intel-gfx@lists.freedesktop.org \
--cc=kamal@canonical.com \
--cc=linux-acpi@vger.kernel.org \
--cc=pedrib@gmail.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox