Re: [PATCH] PCI/ACPI: Change dev_info() to dev_dbg() in the acpi_pci_set_power_state()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jul 30, 2013 at 7:59 AM, Rafael J. Wysocki <rjw@xxxxxxx> wrote:
> On Tuesday, July 30, 2013 10:32:30 AM tianyu.lan@xxxxxxxxx wrote:
>> From: Lan Tianyu <tianyu.lan@xxxxxxxxx>
>>
>> acpi_pci_set_power_state() produces log of device power state changing.
>> Now the log is produced by dev_info(). This seems make no sense when user
>> doesn't open debug option. So change it to dev_dbg() and prevent kernel
>> log from being.
>>
>> Reference: https://bugzilla.kernel.org/show_bug.cgi?id=60636
>> Suggested-by: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx>
>> Signed-off-by: Lan Tianyu <tianyu.lan@xxxxxxxxx>
>
> Bjorn, what do you think about this?

It's fine with me.  For information that I want in the dmesg log all
the time, with no special action from the user, I personally avoid
dev_dbg() because I can never figure out whether it's enabled or not.
But I don't use the information from this printk, so it's completely
up to you.  Just let me know what you want.  If you have other pending
changes to this file, you can take it, or I can put it in my tree.

Bjorn

>> ---
>>  drivers/pci/pci-acpi.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/pci/pci-acpi.c b/drivers/pci/pci-acpi.c
>> index c78cc43..fb3522957 100644
>> --- a/drivers/pci/pci-acpi.c
>> +++ b/drivers/pci/pci-acpi.c
>> @@ -210,7 +210,7 @@ static int acpi_pci_set_power_state(struct pci_dev *dev, pci_power_t state)
>>       }
>>
>>       if (!error)
>> -             dev_info(&dev->dev, "power state changed by ACPI to %s\n",
>> +             dev_dbg(&dev->dev, "power state changed by ACPI to %s\n",
>>                        acpi_power_state_string(state_conv[state]));
>>
>>       return error;
>>
> --
> I speak only for myself.
> Rafael J. Wysocki, Intel Open Source Technology Center.
--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux