On 04/15/2009 06:49 PM, Matthew Garrett wrote: > Does this help? > > diff --git a/drivers/gpu/drm/i915/i915_opregion.c b/drivers/gpu/drm/i915/i915_opregion.c > index 6942772..8dc1fd3 100644 > --- a/drivers/gpu/drm/i915/i915_opregion.c > +++ b/drivers/gpu/drm/i915/i915_opregion.c > @@ -370,11 +370,8 @@ int intel_opregion_init(struct drm_device *dev, int resume > Hi Matthew, Yes please, after your patch the brightness can be adjusted again. :) The behaviour seems slightly different from before your 74a365b3f354 commit in that /sys/class/backlight remains empty until i915 gets loaded for the first time, the contents then persist even if i915 is unloaded. Perhaps the directory was populated by a different kernel module, since when I did the bisect yesterday the directory would be there on boot before I started X or loaded i915. I'm not sure if that matters to any other user-space app. (or if ideally one is supposed to be able to control brightness in the console without having to load i915), but it's fine for controlling brightness in Xorg. Thanks, Niel -- 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
