public inbox for linux-acpi@vger.kernel.org
 help / color / mirror / Atom feed
* [2.6.12-rc2][suspend] Suspending Thinkpad: drive bay light in S3 mode stays on
@ 2005-04-11 20:03 Shawn Starr
       [not found] ` <20050411200341.39635.qmail-4fJJQiRtypeB9c0Qi4KiSl5cfvJIxWXgQQ4Iyu8u01E@public.gmane.org>
  0 siblings, 1 reply; 5+ messages in thread
From: Shawn Starr @ 2005-04-11 20:03 UTC (permalink / raw)
  To: Pavel Machek
  Cc: acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA

I notice in Linux and in XP the drive bay light
remains on while the laptop is in suspend-to-RAM. I
know the ACPI  thinkpad extras added to the kernel
recently can turn this off. I wonder if we can/or need
to write hooks to turn the light off so to conserve
power when we're in S3

Thoughts?

Shawn.


-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [2.6.12-rc2][suspend] Suspending Thinkpad: drive bay light in S3 mode stays on
       [not found] ` <20050411200341.39635.qmail-4fJJQiRtypeB9c0Qi4KiSl5cfvJIxWXgQQ4Iyu8u01E@public.gmane.org>
@ 2005-04-11 20:08   ` Pavel Machek
  2005-04-11 20:14   ` Matthew Garrett
  1 sibling, 0 replies; 5+ messages in thread
From: Pavel Machek @ 2005-04-11 20:08 UTC (permalink / raw)
  To: Shawn Starr
  Cc: acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA

Hi!

> I notice in Linux and in XP the drive bay light
> remains on while the laptop is in suspend-to-RAM. I
> know the ACPI  thinkpad extras added to the kernel
> recently can turn this off. I wonder if we can/or need
> to write hooks to turn the light off so to conserve
> power when we're in S3
> 
> Thoughts?

You probably want to expand the extras driver to turn it off during
S3.

								Pavel
-- 
Boycott Kodak -- for their patent abuse against Java.


-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [2.6.12-rc2][suspend] Suspending Thinkpad: drive bay light in S3 mode stays on
       [not found] ` <20050411200341.39635.qmail-4fJJQiRtypeB9c0Qi4KiSl5cfvJIxWXgQQ4Iyu8u01E@public.gmane.org>
  2005-04-11 20:08   ` Pavel Machek
@ 2005-04-11 20:14   ` Matthew Garrett
  1 sibling, 0 replies; 5+ messages in thread
From: Matthew Garrett @ 2005-04-11 20:14 UTC (permalink / raw)
  To: Shawn Starr
  Cc: Pavel Machek, acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA

On Mon, 2005-04-11 at 16:03 -0400, Shawn Starr wrote:
> I notice in Linux and in XP the drive bay light
> remains on while the laptop is in suspend-to-RAM. I
> know the ACPI  thinkpad extras added to the kernel
> recently can turn this off. I wonder if we can/or need
> to write hooks to turn the light off so to conserve
> power when we're in S3

Just disable it in your suspend script. There's no reason to push that
sort of policy into the kernel.

-- 
Matthew Garrett | mjg59-1xO5oi07KQx4cg9Nei1l7Q@public.gmane.org



-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [2.6.12-rc2][suspend] Suspending Thinkpad: drive bay light in S3 mode stays on
@ 2005-04-11 20:17 Shawn Starr
       [not found] ` <20050411201741.73077.qmail-U/uKf82u23SB9c0Qi4KiSl5cfvJIxWXgQQ4Iyu8u01E@public.gmane.org>
  0 siblings, 1 reply; 5+ messages in thread
From: Shawn Starr @ 2005-04-11 20:17 UTC (permalink / raw)
  To: Matthew Garrett
  Cc: acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA

Sure, I suppose you can, but most suspend tools just
echo stuff to /sys (or still /proc/acpi/sleep) which
makes it harder to script it. Besides, when a laptop
goes into suspend to RAM there should be no extra
power   on except a Moon or some other icon.

That said, the ACPI thinkpad extras was designed to do
all of this so why shouldn't the driver do S3 suspend
if it hooks into it already?

Shawn.

--- Matthew Garrett <mjg59-1xO5oi07KQx4cg9Nei1l7Q@public.gmane.org> wrote:
> On Mon, 2005-04-11 at 16:03 -0400, Shawn Starr
> wrote:
> > I notice in Linux and in XP the drive bay light
> > remains on while the laptop is in suspend-to-RAM.
> I
> > know the ACPI  thinkpad extras added to the kernel
> > recently can turn this off. I wonder if we can/or
> need
> > to write hooks to turn the light off so to
> conserve
> > power when we're in S3
> 
> Just disable it in your suspend script. There's no
> reason to push that
> sort of policy into the kernel.
> 
> -- 
> Matthew Garrett | mjg59-1xO5oi07KQx4cg9Nei1l7Q@public.gmane.org
> 
> 


-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [2.6.12-rc2][suspend] Suspending Thinkpad: drive bay light in S3 mode stays on
       [not found] ` <20050411201741.73077.qmail-U/uKf82u23SB9c0Qi4KiSl5cfvJIxWXgQQ4Iyu8u01E@public.gmane.org>
@ 2005-04-11 20:32   ` Matthew Garrett
  0 siblings, 0 replies; 5+ messages in thread
From: Matthew Garrett @ 2005-04-11 20:32 UTC (permalink / raw)
  To: Shawn Starr
  Cc: acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA

On Mon, 2005-04-11 at 16:17 -0400, Shawn Starr wrote:
> Sure, I suppose you can, but most suspend tools just
> echo stuff to /sys (or still /proc/acpi/sleep) which
> makes it harder to script it. Besides, when a laptop
> goes into suspend to RAM there should be no extra
> power   on except a Moon or some other icon.

Most suspend tools are depressingly stupid. That's not a good reason to
push functionality into the kernel. The vast majority of hardware won't
work with that approach at the moment.

> That said, the ACPI thinkpad extras was designed to do
> all of this so why shouldn't the driver do S3 suspend
> if it hooks into it already?

Because, well, strictly it wasn't. The LED control functionality in the
IBM-acpi code exists because it exposes methods that are used by the
BIOS in normal usage. It gives some degree of extra flexibility -
there's no point in removing that for the sake of having one fewer line
of shell in a suspend script. I might want the LEDs to be in different
states depending on what triggered the suspend.

-- 
Matthew Garrett | mjg59-1xO5oi07KQx4cg9Nei1l7Q@public.gmane.org



-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2005-04-11 20:32 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-04-11 20:03 [2.6.12-rc2][suspend] Suspending Thinkpad: drive bay light in S3 mode stays on Shawn Starr
     [not found] ` <20050411200341.39635.qmail-4fJJQiRtypeB9c0Qi4KiSl5cfvJIxWXgQQ4Iyu8u01E@public.gmane.org>
2005-04-11 20:08   ` Pavel Machek
2005-04-11 20:14   ` Matthew Garrett
  -- strict thread matches above, loose matches on Subject: below --
2005-04-11 20:17 Shawn Starr
     [not found] ` <20050411201741.73077.qmail-U/uKf82u23SB9c0Qi4KiSl5cfvJIxWXgQQ4Iyu8u01E@public.gmane.org>
2005-04-11 20:32   ` Matthew Garrett

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox