On 11/13/2013 01:04 AM, Alex Hung wrote: > Hi, > > It works as expected. Thanks for the confirm! Now I look at the DMI table for initial backlight broken systems, in addition to the "HP Pavilion m4" and "HP 1000 Notebook PC", the following systems are also candidates for removal: HP Pavilion g6 Notebook PC - bug 56661 HP Pavilion dm4 - no bug entry Fujitsu E753 - bug 60161 HP Folio 13-2000 - bug 51141 I've checked the acpidump of all the above laptops except HP dm4, which doesn't have a bug page, they all have the same problem we are talking about here. The HP dm4 is very suspicious according to its corresponding commit message and its brand. I've asked in the individual bug page for test, once confirmed, I'll remove them all. Hi Gustavo, Can you please test this? I've prepared a branch for you: https://github.com/aaronlu/linux acpi_video_dmi It is based on Rafael's linux-next branch, with only one patch on top: remove all the above systems from the DMI table. Thanks, Aaron > > > > On Tue, Nov 12, 2013 at 10:03 AM, Aaron Lu <aaron.lu@xxxxxxxxx> wrote: >> On 05/06/2013 04:23 PM, Alex Hung wrote: >>> On HP 1000 lapops, BIOS reports minimum backlight on boot and >>> causes backlight to dim completely. This ignores the initial backlight >>> values and set to max brightness. >>> >>> Buglink: https://bugs.launchpad.net/bugs/1167760 >> >> I've checked the acpi dump from the bug page, it should be the same >> firmware problem which the following patch is meant to solve: >> http://www.spinics.net/lists/linux-acpi/msg47212.html >> >> commit 2c62333a408f5badd2d2ffd7177f95deeccc5ca4 >> Author: Aaron Lu <aaron.lu@xxxxxxxxx> >> Date: Wed Nov 6 09:07:10 2013 +0800 >> >> ACPI / video: Quirk initial backlight level 0 >> >> Possible to give it a test? It's in Rafael's linux-next branch: >> http://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git >> >> If confirmed, we can save an entry in the DMI table. >> >> Thanks, >> Aaron >> >>> >>> Signed-off-by: Alex Hung <alex.hung@xxxxxxxxxxxxx> >>> --- >>> drivers/acpi/video.c | 8 ++++++++ >>> 1 file changed, 8 insertions(+) >>> >>> diff --git a/drivers/acpi/video.c b/drivers/acpi/video.c >>> index c3932d0..5b32e15 100644 >>> --- a/drivers/acpi/video.c >>> +++ b/drivers/acpi/video.c >>> @@ -456,6 +456,14 @@ static struct dmi_system_id video_dmi_table[] __initdata = { >>> DMI_MATCH(DMI_PRODUCT_NAME, "HP Pavilion dm4 Notebook PC"), >>> }, >>> }, >>> + { >>> + .callback = video_ignore_initial_backlight, >>> + .ident = "HP 1000 Notebook PC", >>> + .matches = { >>> + DMI_MATCH(DMI_BOARD_VENDOR, "Hewlett-Packard"), >>> + DMI_MATCH(Drjw@sisk.plMI_PRODUCT_NAME, "HP 1000 Notebook PC"), >>> + }, >>> + }, >>> {} >>> }; >>> >>> >> > > > -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
