http://bugzilla.kernel.org/show_bug.cgi?id=12233





------- Comment #53 from jwoi...@physics.adelaide.edu.au  2009-03-12 15:53 
-------
I was able to test 2.6.29-rc6 with your patch overnight on the S7020.  After
loading the video module (without fujitsu-laptop loaded and having not supplied
"acpi_backlight=vendor"), /sys/class/backlight/ was completely empty.  This
shows that the patch has not caused video to take over the backlight control on
the S7020 (something it shouldn't do because the S7020 only has a
vendor-specific method available).  So at least for the S7020 this patch does
not cause any regression and can go forward.

With fujitsu-laptop loaded (with video already loaded), /sys/class/backlight/
was populated.

  > grep . /sys/class/backlight/*/*
  /sys/class/backlight/fujitsu-laptop/actual_brightness:7
  /sys/class/backlight/fujitsu-laptop/bl_power:0
  /sys/class/backlight/fujitsu-laptop/brightness:7
  /sys/class/backlight/fujitsu-laptop/max_brightness:7

This was the same regardless of whether "acpi_backlight=vendor" was specified. 
On the S7020 this is to be expected though.

It would be good if this patch could now be tested on a machine which does have
the buggy _BQC method.  In the absence of this the patch should probably be
applied to video anyway since at least in theory it seems worthwhile.  I guess
the only issue is ensuring it doesn't create regressions for other laptops.


-- 
Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

------------------------------------------------------------------------------
Apps built with the Adobe(R) Flex(R) framework and Flex Builder(TM) are
powering Web 2.0 with engaging, cross-platform capabilities. Quickly and
easily build your RIAs with Flex Builder, the Eclipse(TM)based development
software that enables intelligent coding and step-through debugging.
Download the free 60 day trial. http://p.sf.net/sfu/www-adobe-com
_______________________________________________
acpi-bugzilla mailing list
acpi-bugzilla@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla

Reply via email to