Hi, It works as expected. 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"), >> + }, >> + }, >> {} >> }; >> >> > -- Cheers, Alex Hung -- 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
