Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752718Ab1FZDBY (ORCPT ); Sat, 25 Jun 2011 23:01:24 -0400 Received: from netrider.rowland.org ([192.131.102.5]:39610 "HELO netrider.rowland.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1751187Ab1FZDBV (ORCPT ); Sat, 25 Jun 2011 23:01:21 -0400 Date: Sat, 25 Jun 2011 23:01:21 -0400 (EDT) From: Alan Stern X-X-Sender: stern@netrider.rowland.org To: Jesper Juhl cc: "Rafael J. Wysocki" , Linux PM mailing list , Tejun Heo , Greg KH , LKML , Magnus Damm , Kevin Hilman , Subject: Re: [PATCH 1/3] PM / Runtime: Update documentation of interactions with system sleep In-Reply-To: Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1795 Lines: 47 On Sun, 26 Jun 2011, Jesper Juhl wrote: > On Sun, 26 Jun 2011, Rafael J. Wysocki wrote: > > > From: Rafael J. Wysocki > > > > A few tiny nit-picks.. And a few more... > > The documents describing the interactions between runtime PM and > > system sleep generally refer to the model in which the system sleep > > state is entered through a global firmware or hardware operation. > > As a result, some recommendations given in there are not entirely > > suitable for systems in which this is not the case. Update the > > documentation take the existence of those systems into accout. > > > > I believe this should read "... documentation to take the existence of > those systems ..." Also add the missing 'n' in "account". > > +known to it. If that is the case and none of the situations listed above takes > > +place (in particular, if the system is not waking up from hibernation), it may > > +be more efficient to leave the devices that had been suspended before the system > > +suspend began in the suspended state. > > + > > You are refering to device*s*, so I believe this last bit should be "... > in the suspended states". No, the text is correct as it stands. There are many devices, but there are only two power states: active and suspended. For example, it's perfectly correct to say "All five devices are in the suspended state" -- and it would be wrong to say "All five are in the suspended states" (that makes it sound as if the states are suspended rather than the devices being suspended). Alan Stern -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/