I demand that Corentin Chary may or may not have written... > On Tue, Feb 10, 2009 at 4:45 PM, Matthew Garrett <mjg59@xxxxxxxxxxxxx> > wrote: [snip; long delay when initialising eeepc-laptop] >>>> BIOS bug. There's an explicit delay in the eee bios for some reason, and >>>> I haven't found any straightforward way to avoid it. >>> BIOS bug or no, the fact remains that this is (AFAICS) a regression. >> You don't get properly working hotkeys otherwise, to the best of my >> recollection. There's an entry on the kernel bugzilla about this >> somewhere. > http://bugzilla.kernel.org/show_bug.cgi?id=12243 > If I understand things correctly, this was already the case before, but the > kernel boot was slower, so we couldn't see that. If it's not the case, > someone with a 901 should try bisecting (I only have a 701). I have some timings... >From 2.6.28.4: [ 0.898903] eeepc: Eee PC Hotkey Driver [ 0.902414] eeepc: Hotkey init (getting ACPI bus status) [ 0.905913] eeepc: Hotkey init (init flags) [ 2.620193] eeepc: Hotkey init flags 0x41 [ 2.626667] eeepc: Get control methods supported: 0x101713 [ 2.630215] input: Asus EeePC extra buttons as /class/input/input4 >From 2.6.29-rc4: [ 2.141971] eeepc: Eee PC Hotkey Driver [ 2.145325] eeepc: Hotkey init (getting ACPI bus status) [ 2.148555] eeepc: Hotkey init (init flags) [ 2.286445] usb 5-1: new full speed USB device using uhci_hcd and address 2 [... USB Bluetooth device; Elantech test ...] [ 2.780187] input: ETPS/2 Elantech Touchpad as /class/input/input5 [ 28.088399] eeepc: Hotkey init flags 0x41 [ 28.094880] eeepc: Get control methods supported: 0x101713 [ 28.098425] input: Asus EeePC extra buttons as /class/input/input6 Config is as in my first posting in this thread; kernel logs are attached. The following patch is applied: --- ./drivers/platform/x86/eeepc-laptop.c~ 2009-02-10 16:40:18.000000000 +0000 +++ ./drivers/platform/x86/eeepc-laptop.c 2009-02-10 16:40:18.000000000 +0000 @@ -452,10 +452,15 @@ struct acpi_buffer buffer = { ACPI_ALLOCATE_BUFFER, NULL }; int result; + printk(EEEPC_NOTICE "Hotkey init (getting ACPI bus status)\n"); result = acpi_bus_get_status(ehotk->device); if (result) + { + printk(EEEPC_ERR "Hotkey init aborted (%d)\n", result); return result; + } if (ehotk->device->status.present) { + printk(EEEPC_NOTICE "Hotkey init (init flags)\n"); if (write_acpi_int(ehotk->handle, "INIT", ehotk->init_flag, &buffer)) { printk(EEEPC_ERR "Hotkey initialization failed\n"); -- | Darren Salt | linux or ds at | nr. Ashington, | Toon | RISC OS, Linux | youmustbejoking,demon,co,uk | Northumberland | Army | + Buy less and make it last longer. INDUSTRY CAUSES GLOBAL WARMING. If at first you don't succeed, give up. No use being a damn fool.
Attachment:
dmesg-28.4.txt.gz
Description: GNU Zip compressed data
Attachment:
dmesg-29-rc4.txt.gz
Description: GNU Zip compressed data
