acpi_video0 backlight vs. intel_backlight

[monochromec]

acpi_video0 backlight vs. intel_backlight

[bytbox]

You need to modify the bck_driver enum at the top of that file, as well as the selection mechanism (relating to sysfs_backlight_probe). (If you don't want to take this on, I can probably get to it later this week and hand you a patch to test.)

[monochromec]

Ok, here's the complete patch including the enum fix:

--- sysfs_backlight.c 2014-02-22 18:32:50.552133444 +0100
+++ sysfs_backlight.c 2014-02-26 23:51:33.034948626 +0100
@@ -46,6 +46,7 @@
SYSFS_DRIVER_APPLE,
SYSFS_DRIVER_NVIDIA,
SYSFS_DRIVER_NOUVEAU,

• SYSFS_DRIVER_INTEL,
  SYSFS_DRIVER_ACPI,
  #endif
  SYSFS_DRIVER_MAX
  @@ -70,6 +71,7 @@
  "/sys/class/backlight/apple_backlight/actual_brightness",
  "/sys/class/backlight/nvidia_backlight/actual_brightness",
  "/sys/class/backlight/nv_backlight/actual_brightness",
• "/sys/class/backlight/intel_backlight/actual_brightness",
  "/sys/class/backlight/acpi_video0/actual_brightness",
  #endif
  };
  @@ -89,6 +91,7 @@
  "/sys/class/backlight/apple_backlight/brightness",
  "/sys/class/backlight/nvidia_backlight/brightness",
  "/sys/class/backlight/nv_backlight/brightness",
• "/sys/class/backlight/intel_backlight/brightness",
  "/sys/class/backlight/acpi_video0/brightness",
  #endif
  };
  @@ -108,6 +111,7 @@
  "/sys/class/backlight/apple_backlight/max_brightness",
  "/sys/class/backlight/nvidia_backlight/max_brightness",
  "/sys/class/backlight/nv_backlight/max_brightness",
• "/sys/class/backlight/intel_backlight/max_brightness",
  "/sys/class/backlight/acpi_video0/max_brightness",
  #endif
  };

Not sure what should be changed in sysfs_backlight_probe() or friends as they seem to work OOTB - even after closer inspection :-). But maybe I'm missing something here..

[bytbox]

Does this patch work without acpi_backlight=vendor? I was guessing that even with the enum fix, you'd need to change some part of the selection mechanism. However, if this works correctly for you, it's almost certainly correct, and I'll merge it in.

[monochromec]

Managed to find some time to give it another spin. After fixing the enum it also now works w/o the kernel boot parameter. Just don't use a step value of 1 in the pommed.conf file with the Intel driver -); otherwise you will be waiting forever as possible values for the backlight driver range from 1 to 1808. A step value of 10 to 20 does the trick though.

[bytbox]

Ok, I've pushed this patch, so I'm closing this issue. Thanks loads!