Acer Aspire 5710 suffers from a similar ACPI problem reported at:
http://bugzilla.kernel.org/show_bug.cgi?id=13121
like the 5710Z and 7720.
Signed-off-by: Ozan Çağlayan <[email protected]>
---
drivers/acpi/video.c | 8 ++++++++
1 files changed, 8 insertions(+), 0 deletions(-)
diff --git a/drivers/acpi/video.c b/drivers/acpi/video.c
index 9865d46..e2436d1 100644
--- a/drivers/acpi/video.c
+++ b/drivers/acpi/video.c
@@ -571,6 +571,14 @@ static struct dmi_system_id video_dmi_table[] __initdata = {
},
{
.callback = video_set_bqc_offset,
+ .ident = "Acer Aspire 5710",
+ .matches = {
+ DMI_MATCH(DMI_BOARD_VENDOR, "Acer"),
+ DMI_MATCH(DMI_PRODUCT_NAME, "Aspire 5710"),
+ },
+ },
+ {
+ .callback = video_set_bqc_offset,
.ident = "Acer Aspire 5710Z",
.matches = {
DMI_MATCH(DMI_BOARD_VENDOR, "Acer"),
--
1.7.0.4
On Mon, Jun 14, 2010 at 10:17:44AM +0300, Ozan Çağlayan wrote:
> Acer Aspire 5710 suffers from a similar ACPI problem reported at:
> http://bugzilla.kernel.org/show_bug.cgi?id=13121
Bleah. Is this really necessary? If we set maximum brightness and then
read back something other than maximum brightness, then that gives us
the offset.
--
Matthew Garrett | [email protected]
Matthew Garrett wrote:
> On Mon, Jun 14, 2010 at 10:17:44AM +0300, Ozan Çağlayan wrote:
>> Acer Aspire 5710 suffers from a similar ACPI problem reported at:
>> http://bugzilla.kernel.org/show_bug.cgi?id=13121
>
> Bleah. Is this really necessary? If we set maximum brightness and then
> read back something other than maximum brightness, then that gives us
> the offset.
By *this* do you mean the specific quirk that I sent or the whole offset workaround
suggested and upstreamed in the bug report?
Actually I sync'ed the video driver from linux-2.6 on top of 2.6.31.13 and applied this patch.
The user reported that it fixed the issue but it is possible that this quirk
is NOOP and the bug is already fixed with what I've backported.
I'll let him try without the patch, and report back.
Thanks,
Regards.
On Mon, Jun 14, 2010 at 07:19:16PM +0300, Ozan Çağlayan wrote:
> By *this* do you mean the specific quirk that I sent or the whole offset workaround
> suggested and upstreamed in the bug report?
Sorry, yeah, the entire workaround. I'll look into producing something
more generic. Right now the quirk is the only thing that'll work.
--
Matthew Garrett | [email protected]