Andrew Morton napsal(a): > On Wed, 11 Apr 2007 15:04:35 +0200 (CEST) > Jiri Slaby <jirislaby@xxxxxxxxx> wrote: > >> @@ -706,6 +718,8 @@ static int get_lcd_state(void) >> { >> int lcd = 0; >> >> + BUG_ON(!hotk->methods->lcd_status); > > mutter. We could just warn-and-bale here. > > Hopefully this is just an is-jiri-sane assertion and it should never happen, > no matter how busted the BIOS tables are? Exactly :). Only if somebody changes the code around, so that it is called with the method unset, it triggers and in that case it's definitely a bug. Side note, as Danny wrote, there is an asus-laptop driver sitting in -mm, which supersedes this one -- if you plan to push it upstream in the near future, don't bother with this patch. thanks, -- http://www.fi.muni.cz/~xslaby/ Jiri Slaby faculty of informatics, masaryk university, brno, cz e-mail: jirislaby gmail com, gpg pubkey fingerprint: B674 9967 0407 CE62 ACC8 22A0 32CC 55C3 39D4 7A7E - 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