Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754508Ab3G2DGk (ORCPT ); Sun, 28 Jul 2013 23:06:40 -0400 Received: from mail-oa0-f46.google.com ([209.85.219.46]:61254 "EHLO mail-oa0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752591Ab3G2DGh (ORCPT ); Sun, 28 Jul 2013 23:06:37 -0400 MIME-Version: 1.0 In-Reply-To: <1895563.95T8SxMegu@vostro.rjw.lan> References: <10433383.dueoNg39qi@vostro.rjw.lan> <1895563.95T8SxMegu@vostro.rjw.lan> Date: Mon, 29 Jul 2013 11:06:36 +0800 Message-ID: Subject: Re: [PATCH 2/3] ACPI / PM: Make messages in acpi_device_set_power() print device names From: Lan Tianyu To: "Rafael J. Wysocki" Cc: ACPI Devel Maling List , LKML , Linux PM list , Yinghai Lu , Bjorn Helgaas , Aaron Lu , Tejun Heo , linux-ide@vger.kernel.org Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3741 Lines: 90 2013/7/27 Rafael J. Wysocki : > From: Rafael J. Wysocki > > Modify acpi_device_set_power() so that diagnostic messages printed by > it to the kernel log always contain the name of the device concerned > to make it possible to identify the device that triggered the message > if need be. > > Also replace printk(KERN_WARNING ) with dev_warn() everywhere in that > function. > > Signed-off-by: Rafael J. Wysocki > --- > drivers/acpi/device_pm.c | 22 ++++++++++------------ > 1 file changed, 10 insertions(+), 12 deletions(-) > > Index: linux-pm/drivers/acpi/device_pm.c > =================================================================== > --- linux-pm.orig/drivers/acpi/device_pm.c > +++ linux-pm/drivers/acpi/device_pm.c > @@ -166,20 +166,20 @@ int acpi_device_set_power(struct acpi_de > /* Make sure this is a valid target state */ > > if (state == device->power.state) { > - ACPI_DEBUG_PRINT((ACPI_DB_INFO, "Device is already at %s\n", > + ACPI_DEBUG_PRINT((ACPI_DB_INFO, "Device [%s] already in %s\n", ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^ Missing "is" or it should be omitted? > + device->pnp.bus_id, > acpi_power_state_string(state))); > return 0; > } > > if (!device->power.states[state].flags.valid) { > - printk(KERN_WARNING PREFIX "Device does not support %s\n", > - acpi_power_state_string(state)); > + dev_warn(&device->dev, "Power state %s not supported\n", > + acpi_power_state_string(state)); > return -ENODEV; > } > if (device->parent && (state < device->parent->power.state)) { > - printk(KERN_WARNING PREFIX > - "Cannot set device to a higher-powered" > - " state than parent\n"); > + dev_warn(&device->dev, "Cannot transition to a higher-powered " > + "state than parent\n"); > return -ENODEV; > } > > @@ -192,8 +192,8 @@ int acpi_device_set_power(struct acpi_de > > if (state < device->power.state && state != ACPI_STATE_D0 > && device->power.state >= ACPI_STATE_D3_HOT) { > - printk(KERN_WARNING PREFIX > - "Cannot transition to non-D0 state from D3\n"); > + dev_warn(&device->dev, > + "Cannot transition to non-D0 state from D3\n"); > return -ENODEV; > } > > @@ -220,10 +220,8 @@ int acpi_device_set_power(struct acpi_de > > end: > if (result) { > - printk(KERN_WARNING PREFIX > - "Device [%s] failed to transition to %s\n", > - device->pnp.bus_id, > - acpi_power_state_string(state)); > + dev_warn(&device->dev, "Failed to change power state to %s\n", > + acpi_power_state_string(state)); > } else { > device->power.state = state; > ACPI_DEBUG_PRINT((ACPI_DB_INFO, > > -- > To unsubscribe from this list: send the line "unsubscribe linux-acpi" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Best regards Tianyu Lan -- 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/