[haiku-bugs] Re: [Haiku] #7525: Setting backlight brightness doesn't work.

  • From: "mmlr" <trac@xxxxxxxxxxxx>
  • Date: Mon, 23 May 2011 15:48:47 -0000

#7525: Setting backlight brightness doesn't work.
----------------------------+----------------------------
   Reporter:  pulkomandy    |      Owner:  tqh
       Type:  bug           |     Status:  assigned
   Priority:  normal        |  Milestone:  R1
  Component:  Drivers/ACPI  |    Version:  R1/Development
 Resolution:                |   Keywords:
 Blocked By:                |   Blocking:
Has a Patch:  1             |   Platform:  All
----------------------------+----------------------------

Comment (by mmlr):

 So the error is there in either case...

 Replying to [comment:12 luroh]:
 > Observing the output of the "ints" command didn't yield much in my case,
 too many numbers fluctuating with or without the patch to make any sense
 to me.

 The only thing I'm interested in is whether or not there are any handled
 and/or unhandled interrupts on int 9, where the ACPI interrupt handler
 should be installed. The thing is that the polarity and trigger mode were
 previously unset when only using the PIC (and when applying this patch, as
 the info will get lost because there's no IO-APIC structure to hold it).
 It's therefore possible that ACPI SCIs simply didn't work before and
 therefore SMIs were generated that the BIOS then handled (adjusting the
 brigthness for example). With IO-APIC support on and with the source
 override in place that, in your case, specifies the SCI interrupt to 9,
 active high, level triggered it might actually work now, shutting out the
 BIOS and therefore leaving the buttons to the OS.

 What you could try to see if the source override is the "problem" would be
 to comment out the line:
 {{{
 acpi_configure_source_overrides(madt);
 }}}

 In browser:haiku/trunk/src/system/kernel/arch/x86/ioapic.cpp#L743 and see
 if that changes things.

 I have checked FreeBSD again and they only force the SCI to level/low if
 there is no interrupt source override or if the user explicitly demanded
 it via settings. So I don't think doing the configuration this way is
 wrong per se.

-- 
Ticket URL: <http://dev.haiku-os.org/ticket/7525#comment:15>
Haiku <http://dev.haiku-os.org>
Haiku - the operating system.

Other related posts: