From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S967402Ab2ERUva (ORCPT ); Fri, 18 May 2012 16:51:30 -0400 Received: from ogre.sisk.pl ([193.178.161.156]:53333 "EHLO ogre.sisk.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965500Ab2ERUv0 (ORCPT ); Fri, 18 May 2012 16:51:26 -0400 From: "Rafael J. Wysocki" To: ACPI Devel Mailing List Subject: Re: [PATCH] ACPI / PM: Fix error message in __acpi_bus_set_power() Date: Fri, 18 May 2012 22:56:23 +0200 User-Agent: KMail/1.13.6 (Linux/3.4.0-rc7+; KDE/4.6.0; x86_64; ; ) Cc: LKML , Len Brown , Linux PM list References: <201205142253.57023.rjw@sisk.pl> In-Reply-To: <201205142253.57023.rjw@sisk.pl> MIME-Version: 1.0 Content-Type: Text/Plain; charset="utf-8" Content-Transfer-Encoding: 7bit Message-Id: <201205182256.23670.rjw@sisk.pl> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Monday, May 14, 2012, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > After recent changes of the ACPI device low-power states definitions > an error message in __acpi_bus_set_power() needs to be updated so > that it prints the correct name of the state that has been requested > (currently it will print "D3" for D3hot and "D4" for D3cold). Please disregard this patch, I'll send a better one in a little while. Thanks, Rafael > Signed-off-by: Rafael J. Wysocki > --- > drivers/acpi/bus.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > Index: linux/drivers/acpi/bus.c > =================================================================== > --- linux.orig/drivers/acpi/bus.c > +++ linux/drivers/acpi/bus.c > @@ -240,7 +240,9 @@ static int __acpi_bus_set_power(struct a > } > > if (!device->power.states[state].flags.valid) { > - printk(KERN_WARNING PREFIX "Device does not support D%d\n", state); > + printk(KERN_WARNING PREFIX "Device does not support D%d%s\n", > + state == ACPI_STATE_D3_COLD ? 3 : state, > + state == ACPI_STATE_D3_HOT ? "hot" : ""); > return -ENODEV; > } > if (device->parent && (state < device->parent->power.state)) { > -- > 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/ > >