On Fri, Sep 05, 2014 at 08:29:09AM +0200, Francis Moreau wrote: > On 09/04/2014 11:21 PM, Brian Norris wrote: [...] > > Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx> > > Cc: <stable@xxxxxxxxxxxxxxx> # 3.10+: 27ddcc6596e5: PM / sleep: Add state field to pm_states[] entries > > Cc: <stable@xxxxxxxxxxxxxxx> # 3.10+ > > --- > > This is a backport request for these two commits upstream: > > > > 27ddcc6596e5 PM / sleep: Add state field to pm_states[] entries > > 43e8317b0bba PM / sleep: Use valid_state() for platform-dependent sleep states only > > > > Wouldn't it be cleaner to have 2 separate backports then ? The first is purely a dependency for the second. It has no value on its own. So I thought the above form made sense and followed the process mentioned in Documentation/stable_kernel_rules.txt. Admittedly, it's a little asymmetric. But I really don't know what the "best" option is, since I'd prefer not having to send around any patch text at all, unless the backport is not trivial (these apply cleanly). Related: I don't feel like Documentation/stable_kernel_rules.txt is very clear under the "Procedure" section. It lists a series of non-sequential steps, some of which are mutually exclusive. Any tips on making my post-merge -stable submissions better are appreciated. And recommendations on improving the text (or just my interpretation) of Documentation/stable_kernel_rules.txt are welcome too. Brian -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html