Dear Corentin, 2012/8/9 Corentin Chary <[email protected]>: > On Thu, Aug 9, 2012 at 5:29 AM, AceLan Kao <[email protected]> wrote: >> Dear Corentin, >> >> 2012/8/9 Carlos Alberto Lopez Perez <[email protected]>: >>> On 08/08/12 11:34, Corentin Chary wrote: >>>>> The problem is that the /sys/class/backlight/acpi_video0 directory still >>>>> exists >>>>> > and the gnome-settings-daemon will choose acpi_video0 for the >>>>> > brightness functions >>>> Are you sure acpi_video_unregister() was called in this case ? can you >>>> add some traces to video.c to see what happened and why >>>> backlight_device_unregister() wasn't called ? >> Yes, you are right, acpi_video_unregister() is not be called. >> #ifdef CONFIG_ACPI_VIDEO doesn't work as expected. >> In ubuntu kernel, it's config as module, CONFIG_ACPI_VIDEO=m. >> I will try to figure out what happened. >> >>>>> > But, if asus-wmi is loaded before video, then video won't call >>>>> > backlight_device_register(), >>>>> > since the flag is already be set. >>>> Honestly, I don't know if there is a way to do that, and asus-wmi >>>> should be loadable and working at any time, not only on boot. >>> >>> What about writing an udev rule to force using acpi_video0 for asus-wmi? >>> >>> Use udevadm to get the attributes for building the rule >>> >>> udevadm info --attribute-walk --path=/sys/class/backlight/acpi_video0 >>> > > Should be fixed by > https://github.com/iksaif/platform-drivers-x86/commit/3a00b9fd44ac59f5ded116478e750e883c869bb2 > (not yet in tree, was waiting for next merge window but will send it > in an RC if it fixes the issue) Great, thanks a lot.
> > -- > Corentin Chary > http://xf.iksaif.net > -- > To unsubscribe from this list: send the line "unsubscribe > platform-driver-x86" in > the body of a message to [email protected] > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Chia-Lin Kao(AceLan) http://blog.acelan.idv.tw/ E-Mail: acelan.kaoATcanonical.com (s/AT/@/) -- To unsubscribe from this list: send the line "unsubscribe platform-driver-x86" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
