* [PATCH] PCI/ACPI: Change dev_info() to dev_dbg() in the acpi_pci_set_power_state() @ 2013-07-30 2:32 tianyu.lan 2013-07-30 13:59 ` Rafael J. Wysocki 0 siblings, 1 reply; 5+ messages in thread From: tianyu.lan @ 2013-07-30 2:32 UTC (permalink / raw) To: rjw, lenb, bhelgaas; +Cc: Lan Tianyu, linux-acpi, linux-pci, stern From: Lan Tianyu <tianyu.lan@intel.com> 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@rowland.harvard.edu> Signed-off-by: Lan Tianyu <tianyu.lan@intel.com> --- 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; -- 1.8.3.2 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] PCI/ACPI: Change dev_info() to dev_dbg() in the acpi_pci_set_power_state() 2013-07-30 2:32 [PATCH] PCI/ACPI: Change dev_info() to dev_dbg() in the acpi_pci_set_power_state() tianyu.lan @ 2013-07-30 13:59 ` Rafael J. Wysocki 2013-07-30 15:40 ` Bjorn Helgaas 0 siblings, 1 reply; 5+ messages in thread From: Rafael J. Wysocki @ 2013-07-30 13:59 UTC (permalink / raw) To: tianyu.lan, bhelgaas; +Cc: lenb, linux-acpi, linux-pci, stern On Tuesday, July 30, 2013 10:32:30 AM tianyu.lan@intel.com wrote: > From: Lan Tianyu <tianyu.lan@intel.com> > > 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@rowland.harvard.edu> > Signed-off-by: Lan Tianyu <tianyu.lan@intel.com> Bjorn, what do you think about this? Rafael > --- > 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. ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] PCI/ACPI: Change dev_info() to dev_dbg() in the acpi_pci_set_power_state() 2013-07-30 13:59 ` Rafael J. Wysocki @ 2013-07-30 15:40 ` Bjorn Helgaas 2013-07-30 23:31 ` Rafael J. Wysocki 0 siblings, 1 reply; 5+ messages in thread From: Bjorn Helgaas @ 2013-07-30 15:40 UTC (permalink / raw) To: Rafael J. Wysocki Cc: lan,Tianyu, Len Brown, linux-acpi@vger.kernel.org, linux-pci@vger.kernel.org, Alan Stern On Tue, Jul 30, 2013 at 7:59 AM, Rafael J. Wysocki <rjw@sisk.pl> wrote: > On Tuesday, July 30, 2013 10:32:30 AM tianyu.lan@intel.com wrote: >> From: Lan Tianyu <tianyu.lan@intel.com> >> >> 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@rowland.harvard.edu> >> Signed-off-by: Lan Tianyu <tianyu.lan@intel.com> > > 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. ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] PCI/ACPI: Change dev_info() to dev_dbg() in the acpi_pci_set_power_state() 2013-07-30 15:40 ` Bjorn Helgaas @ 2013-07-30 23:31 ` Rafael J. Wysocki 2013-07-31 17:26 ` Bjorn Helgaas 0 siblings, 1 reply; 5+ messages in thread From: Rafael J. Wysocki @ 2013-07-30 23:31 UTC (permalink / raw) To: Bjorn Helgaas Cc: lan,Tianyu, Len Brown, linux-acpi@vger.kernel.org, linux-pci@vger.kernel.org, Alan Stern On Tuesday, July 30, 2013 09:40:18 AM Bjorn Helgaas wrote: > On Tue, Jul 30, 2013 at 7:59 AM, Rafael J. Wysocki <rjw@sisk.pl> wrote: > > On Tuesday, July 30, 2013 10:32:30 AM tianyu.lan@intel.com wrote: > >> From: Lan Tianyu <tianyu.lan@intel.com> > >> > >> 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@rowland.harvard.edu> > >> Signed-off-by: Lan Tianyu <tianyu.lan@intel.com> > > > > 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. Yes, I have one change to this file in my queue for 3.12, so I can take this patch. Thanks, Rafael > >> --- > >> 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. -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center. ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] PCI/ACPI: Change dev_info() to dev_dbg() in the acpi_pci_set_power_state() 2013-07-30 23:31 ` Rafael J. Wysocki @ 2013-07-31 17:26 ` Bjorn Helgaas 0 siblings, 0 replies; 5+ messages in thread From: Bjorn Helgaas @ 2013-07-31 17:26 UTC (permalink / raw) To: Rafael J. Wysocki Cc: lan,Tianyu, Len Brown, linux-acpi@vger.kernel.org, linux-pci@vger.kernel.org, Alan Stern On Tue, Jul 30, 2013 at 5:31 PM, Rafael J. Wysocki <rjw@sisk.pl> wrote: > On Tuesday, July 30, 2013 09:40:18 AM Bjorn Helgaas wrote: >> On Tue, Jul 30, 2013 at 7:59 AM, Rafael J. Wysocki <rjw@sisk.pl> wrote: >> > On Tuesday, July 30, 2013 10:32:30 AM tianyu.lan@intel.com wrote: >> >> From: Lan Tianyu <tianyu.lan@intel.com> >> >> >> >> 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@rowland.harvard.edu> >> >> Signed-off-by: Lan Tianyu <tianyu.lan@intel.com> >> > >> > 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. > > Yes, I have one change to this file in my queue for 3.12, so I can take this > patch. OK. Acked-by: Bjorn Helgaas <bhelgaas@google.com> >> >> --- >> >> 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. > -- > I speak only for myself. > Rafael J. Wysocki, Intel Open Source Technology Center. ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2013-07-31 17:26 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-07-30 2:32 [PATCH] PCI/ACPI: Change dev_info() to dev_dbg() in the acpi_pci_set_power_state() tianyu.lan 2013-07-30 13:59 ` Rafael J. Wysocki 2013-07-30 15:40 ` Bjorn Helgaas 2013-07-30 23:31 ` Rafael J. Wysocki 2013-07-31 17:26 ` Bjorn Helgaas
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).