Re: [PATCH 0/2]: the patch set to weaken the dependency between ACPI video driver and i915 driver

[yakui_zhao <yakui.zhao@intel.com>]

On Thu, 2009-04-16 at 00:54 +0800, Matthew Garrett wrote:
> Sorry, I should have been clearer. ACPI is required if the hardware 
> supports opregion. Now that I've thought about it more, I think the 
> patch I've just sent should remove the need to change the #ifdefs in
> the 
> video driver
Agree.    If another patch from you is applied, it is unnecessary to
change the #ifdef in the acpi video driver.
    Please go ahead. Thanks.

Another issue is about the ACPI opregion. For the boxes that supports
ACPI opregion the brightness still can't be controlled even when the
acpi video driver is loaded correctly. Only after the i915 driver is
loaded, the ACPI backlight I/F can work well. In such case whether the
ACPI video driver can work well depends on the i915 driver.
    But whether the i915 driver can work well doesn't depend on the acpi
video driver. Right? 

At the same time a lot of people complain fail in the kernel compilation
if the i915 is compiled as builtin module and acpi video is compiled as
module. 

So we had better make some changes so that there is no failure about
kernel compilation.

thanks.