Re: [PATCH 6/10] ACPI: register ACPI Video LCD as generic thermal cooling device

2008-01-21 Thread Zhang Rui
Hi, Matthew, On Fri, 2008-01-18 at 09:42 +0800, Matthew Garrett wrote: > On Fri, Jan 18, 2008 at 09:31:40AM +0800, Zhang Rui wrote: > > > Just like I don't think lcd should be used for ACPI thermal > management > > before I saw it is listed in _TZD and intel_menlow requires to > throttle > > it

Re: [PATCH 6/10] ACPI: register ACPI Video LCD as generic thermal cooling device

2008-01-21 Thread Zhang Rui
Hi, Matthew, On Fri, 2008-01-18 at 09:42 +0800, Matthew Garrett wrote: On Fri, Jan 18, 2008 at 09:31:40AM +0800, Zhang Rui wrote: Just like I don't think lcd should be used for ACPI thermal management before I saw it is listed in _TZD and intel_menlow requires to throttle it when

Re: [PATCH 6/10] ACPI: register ACPI Video LCD as generic thermal cooling device

2008-01-17 Thread Matthew Garrett
On Fri, Jan 18, 2008 at 09:31:40AM +0800, Zhang Rui wrote: > Just like I don't think lcd should be used for ACPI thermal management > before I saw it is listed in _TZD and intel_menlow requires to throttle > it when overheating, why not let the individual drivers implement the > callbacks if

Re: [PATCH 6/10] ACPI: register ACPI Video LCD as generic thermal cooling device

2008-01-17 Thread Zhang Rui
Hi, Mattew, thanks for your comments. On Thu, 2008-01-17 at 20:24 +0800, Matthew Garrett wrote: > On Thu, Jan 17, 2008 at 03:51:22PM +0800, Zhang Rui wrote: > > From: Zhang Rui <[EMAIL PROTECTED]> > > > > Register ACPI video device as thermal cooling devices as they may be > listed > > in _TZD

Re: [PATCH 6/10] ACPI: register ACPI Video LCD as generic thermal cooling device

2008-01-17 Thread Matthew Garrett
On Thu, Jan 17, 2008 at 03:51:22PM +0800, Zhang Rui wrote: > From: Zhang Rui <[EMAIL PROTECTED]> > > Register ACPI video device as thermal cooling devices as they may be listed > in _TZD method and the backlight control can be used for throttling. I'm worried to some extent about how ungeneric

Re: [PATCH 6/10] ACPI: register ACPI Video LCD as generic thermal cooling device

2008-01-17 Thread Matthew Garrett
On Thu, Jan 17, 2008 at 03:51:22PM +0800, Zhang Rui wrote: From: Zhang Rui [EMAIL PROTECTED] Register ACPI video device as thermal cooling devices as they may be listed in _TZD method and the backlight control can be used for throttling. I'm worried to some extent about how ungeneric this

Re: [PATCH 6/10] ACPI: register ACPI Video LCD as generic thermal cooling device

2008-01-17 Thread Zhang Rui
Hi, Mattew, thanks for your comments. On Thu, 2008-01-17 at 20:24 +0800, Matthew Garrett wrote: On Thu, Jan 17, 2008 at 03:51:22PM +0800, Zhang Rui wrote: From: Zhang Rui [EMAIL PROTECTED] Register ACPI video device as thermal cooling devices as they may be listed in _TZD method and

Re: [PATCH 6/10] ACPI: register ACPI Video LCD as generic thermal cooling device

2008-01-17 Thread Matthew Garrett
On Fri, Jan 18, 2008 at 09:31:40AM +0800, Zhang Rui wrote: Just like I don't think lcd should be used for ACPI thermal management before I saw it is listed in _TZD and intel_menlow requires to throttle it when overheating, why not let the individual drivers implement the callbacks if there is

[PATCH 6/10] ACPI: register ACPI Video LCD as generic thermal cooling device

2008-01-16 Thread Zhang Rui
From: Zhang Rui <[EMAIL PROTECTED]> Register ACPI video device as thermal cooling devices as they may be listed in _TZD method and the backlight control can be used for throttling. Signed-off-by: Zhang Rui <[EMAIL PROTECTED]> Signed-off-by: Thomas Sujith <[EMAIL PROTECTED]> ---

[PATCH 6/10] ACPI: register ACPI Video LCD as generic thermal cooling device

2008-01-16 Thread Zhang Rui
From: Zhang Rui [EMAIL PROTECTED] Register ACPI video device as thermal cooling devices as they may be listed in _TZD method and the backlight control can be used for throttling. Signed-off-by: Zhang Rui [EMAIL PROTECTED] Signed-off-by: Thomas Sujith [EMAIL PROTECTED] --- drivers/acpi/video.c |