Re: [PATCH] PCI / ACPI / PM: Use correct power state strings in messages
On Friday, June 14, 2013 04:49:49 PM Bjorn Helgaas wrote: > On Sat, Jun 15, 2013 at 12:28:12AM +0200, Rafael J. Wysocki wrote: > > On Friday, June 14, 2013 11:08:44 AM Bjorn Helgaas wrote: > > > On Thu, Jun 13, 2013 at 4:29 PM, Rafael J. Wysocki wrote: > > > > From: Rafael J. Wysocki > > > > > > > > Make acpi_pci_set_power_state() print the name of the ACPI device > > > > power state the device has been actually put into instead of printing > > > > the name of the requested PCI device power state, which need not be > > > > the same. > > > > > > > > Signed-off-by: Rafael J. Wysocki > > > > --- > > > > > > > > For 3.11. > > > > > > > > Thanks, > > > > Rafael > > > > > > > > --- > > > > drivers/pci/pci-acpi.c |2 +- > > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > > > Index: linux-pm/drivers/pci/pci-acpi.c > > > > === > > > > --- linux-pm.orig/drivers/pci/pci-acpi.c > > > > +++ linux-pm/drivers/pci/pci-acpi.c > > > > @@ -211,7 +211,7 @@ static int acpi_pci_set_power_state(stru > > > > > > > > if (!error) > > > > dev_info(>dev, "power state changed by ACPI to > > > > %s\n", > > > > -pci_power_name(state)); > > > > +acpi_power_state_string(state_conv[state])); > > > > > > > > return error; > > > > } > > > > > > > > > > Just to double-check this, it *looks* like the effective change is > > > that for PCI_D3hot and PCI_D3cold, we'll print "(unknown)" instead of > > > "D3hot" and "D3cold" because state_conv[] folds both PCI_D3hot and > > > PCI_D3cold into ACPI_STATE_D3, and acpi_power_state_string() doesn't > > > have a case for ACPI_STATE_D3. > > > > No, it won't work like this, because ACPI_STATE_D3 == ACPI_STATE_D3_COLD. > > :-) > > > > So, actually, "D3cold" will be printed for both PCI_D3hot and PCI_D3cold > > (and I have tested this). > > Ah, right, I should have noticed that. > > > Well, I think it's better to actually replace ACPI_STATE_D3 everywhere in > > pci-acpi.c with ACPI_STATE_D3_COLD to avoid that confusion. Do you want me > > to prepare a patch? > > If the following is OK, I'll just put it in my pci/misc branch: Yes, it is, thanks! Rafael > commit fc6504b3a4dc9beae782a11e6f7c3c4a9f077fb8 > Author: Rafael J. Wysocki > Date: Fri Jun 14 00:29:50 2013 +0200 > > PCI / ACPI / PM: Use correct power state strings in messages > > Make acpi_pci_set_power_state() print the name of the ACPI device > power state the device has been actually put into instead of printing > the name of the requested PCI device power state, which need not be > the same. > > [bhelgaas: use ACPI_STATE_D3_COLD (ACPI_STATE_D3 == ACPI_STATE_D3_COLD)] > Signed-off-by: Rafael J. Wysocki > Signed-off-by: Bjorn Helgaas > > diff --git a/drivers/pci/pci-acpi.c b/drivers/pci/pci-acpi.c > index 6c15d6a..dbdc5f7 100644 > --- a/drivers/pci/pci-acpi.c > +++ b/drivers/pci/pci-acpi.c > @@ -186,8 +186,8 @@ static int acpi_pci_set_power_state(struct pci_dev *dev, > pci_power_t state) > [PCI_D0] = ACPI_STATE_D0, > [PCI_D1] = ACPI_STATE_D1, > [PCI_D2] = ACPI_STATE_D2, > - [PCI_D3hot] = ACPI_STATE_D3, > - [PCI_D3cold] = ACPI_STATE_D3 > + [PCI_D3hot] = ACPI_STATE_D3_COLD, > + [PCI_D3cold] = ACPI_STATE_D3_COLD, > }; > int error = -EINVAL; > > @@ -211,7 +211,7 @@ static int acpi_pci_set_power_state(struct pci_dev *dev, > pci_power_t state) > > if (!error) > dev_info(>dev, "power state changed by ACPI to %s\n", > - pci_power_name(state)); > + 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-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] PCI / ACPI / PM: Use correct power state strings in messages
On Sat, Jun 15, 2013 at 12:28:12AM +0200, Rafael J. Wysocki wrote: > On Friday, June 14, 2013 11:08:44 AM Bjorn Helgaas wrote: > > On Thu, Jun 13, 2013 at 4:29 PM, Rafael J. Wysocki wrote: > > > From: Rafael J. Wysocki > > > > > > Make acpi_pci_set_power_state() print the name of the ACPI device > > > power state the device has been actually put into instead of printing > > > the name of the requested PCI device power state, which need not be > > > the same. > > > > > > Signed-off-by: Rafael J. Wysocki > > > --- > > > > > > For 3.11. > > > > > > Thanks, > > > Rafael > > > > > > --- > > > drivers/pci/pci-acpi.c |2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > Index: linux-pm/drivers/pci/pci-acpi.c > > > === > > > --- linux-pm.orig/drivers/pci/pci-acpi.c > > > +++ linux-pm/drivers/pci/pci-acpi.c > > > @@ -211,7 +211,7 @@ static int acpi_pci_set_power_state(stru > > > > > > if (!error) > > > dev_info(>dev, "power state changed by ACPI to %s\n", > > > -pci_power_name(state)); > > > +acpi_power_state_string(state_conv[state])); > > > > > > return error; > > > } > > > > > > > Just to double-check this, it *looks* like the effective change is > > that for PCI_D3hot and PCI_D3cold, we'll print "(unknown)" instead of > > "D3hot" and "D3cold" because state_conv[] folds both PCI_D3hot and > > PCI_D3cold into ACPI_STATE_D3, and acpi_power_state_string() doesn't > > have a case for ACPI_STATE_D3. > > No, it won't work like this, because ACPI_STATE_D3 == ACPI_STATE_D3_COLD. :-) > > So, actually, "D3cold" will be printed for both PCI_D3hot and PCI_D3cold > (and I have tested this). Ah, right, I should have noticed that. > Well, I think it's better to actually replace ACPI_STATE_D3 everywhere in > pci-acpi.c with ACPI_STATE_D3_COLD to avoid that confusion. Do you want me > to prepare a patch? If the following is OK, I'll just put it in my pci/misc branch: commit fc6504b3a4dc9beae782a11e6f7c3c4a9f077fb8 Author: Rafael J. Wysocki Date: Fri Jun 14 00:29:50 2013 +0200 PCI / ACPI / PM: Use correct power state strings in messages Make acpi_pci_set_power_state() print the name of the ACPI device power state the device has been actually put into instead of printing the name of the requested PCI device power state, which need not be the same. [bhelgaas: use ACPI_STATE_D3_COLD (ACPI_STATE_D3 == ACPI_STATE_D3_COLD)] Signed-off-by: Rafael J. Wysocki Signed-off-by: Bjorn Helgaas diff --git a/drivers/pci/pci-acpi.c b/drivers/pci/pci-acpi.c index 6c15d6a..dbdc5f7 100644 --- a/drivers/pci/pci-acpi.c +++ b/drivers/pci/pci-acpi.c @@ -186,8 +186,8 @@ static int acpi_pci_set_power_state(struct pci_dev *dev, pci_power_t state) [PCI_D0] = ACPI_STATE_D0, [PCI_D1] = ACPI_STATE_D1, [PCI_D2] = ACPI_STATE_D2, - [PCI_D3hot] = ACPI_STATE_D3, - [PCI_D3cold] = ACPI_STATE_D3 + [PCI_D3hot] = ACPI_STATE_D3_COLD, + [PCI_D3cold] = ACPI_STATE_D3_COLD, }; int error = -EINVAL; @@ -211,7 +211,7 @@ static int acpi_pci_set_power_state(struct pci_dev *dev, pci_power_t state) if (!error) dev_info(>dev, "power state changed by ACPI to %s\n", -pci_power_name(state)); +acpi_power_state_string(state_conv[state])); return error; } -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] PCI / ACPI / PM: Use correct power state strings in messages
On Friday, June 14, 2013 11:08:44 AM Bjorn Helgaas wrote: > On Thu, Jun 13, 2013 at 4:29 PM, Rafael J. Wysocki wrote: > > From: Rafael J. Wysocki > > > > Make acpi_pci_set_power_state() print the name of the ACPI device > > power state the device has been actually put into instead of printing > > the name of the requested PCI device power state, which need not be > > the same. > > > > Signed-off-by: Rafael J. Wysocki > > --- > > > > For 3.11. > > > > Thanks, > > Rafael > > > > --- > > drivers/pci/pci-acpi.c |2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > Index: linux-pm/drivers/pci/pci-acpi.c > > === > > --- linux-pm.orig/drivers/pci/pci-acpi.c > > +++ linux-pm/drivers/pci/pci-acpi.c > > @@ -211,7 +211,7 @@ static int acpi_pci_set_power_state(stru > > > > if (!error) > > dev_info(>dev, "power state changed by ACPI to %s\n", > > -pci_power_name(state)); > > +acpi_power_state_string(state_conv[state])); > > > > return error; > > } > > > > Just to double-check this, it *looks* like the effective change is > that for PCI_D3hot and PCI_D3cold, we'll print "(unknown)" instead of > "D3hot" and "D3cold" because state_conv[] folds both PCI_D3hot and > PCI_D3cold into ACPI_STATE_D3, and acpi_power_state_string() doesn't > have a case for ACPI_STATE_D3. No, it won't work like this, because ACPI_STATE_D3 == ACPI_STATE_D3_COLD. :-) So, actually, "D3cold" will be printed for both PCI_D3hot and PCI_D3cold (and I have tested this). Well, I think it's better to actually replace ACPI_STATE_D3 everywhere in pci-acpi.c with ACPI_STATE_D3_COLD to avoid that confusion. Do you want me to prepare a patch? Rafael -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] PCI / ACPI / PM: Use correct power state strings in messages
On Thu, Jun 13, 2013 at 4:29 PM, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > Make acpi_pci_set_power_state() print the name of the ACPI device > power state the device has been actually put into instead of printing > the name of the requested PCI device power state, which need not be > the same. > > Signed-off-by: Rafael J. Wysocki > --- > > For 3.11. > > Thanks, > Rafael > > --- > drivers/pci/pci-acpi.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > Index: linux-pm/drivers/pci/pci-acpi.c > === > --- linux-pm.orig/drivers/pci/pci-acpi.c > +++ linux-pm/drivers/pci/pci-acpi.c > @@ -211,7 +211,7 @@ static int acpi_pci_set_power_state(stru > > if (!error) > dev_info(>dev, "power state changed by ACPI to %s\n", > -pci_power_name(state)); > +acpi_power_state_string(state_conv[state])); > > return error; > } > Just to double-check this, it *looks* like the effective change is that for PCI_D3hot and PCI_D3cold, we'll print "(unknown)" instead of "D3hot" and "D3cold" because state_conv[] folds both PCI_D3hot and PCI_D3cold into ACPI_STATE_D3, and acpi_power_state_string() doesn't have a case for ACPI_STATE_D3. Is that what you intended? Bjorn -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] PCI / ACPI / PM: Use correct power state strings in messages
On Thu, Jun 13, 2013 at 4:29 PM, Rafael J. Wysocki r...@sisk.pl wrote: From: Rafael J. Wysocki rafael.j.wyso...@intel.com Make acpi_pci_set_power_state() print the name of the ACPI device power state the device has been actually put into instead of printing the name of the requested PCI device power state, which need not be the same. Signed-off-by: Rafael J. Wysocki rafael.j.wyso...@intel.com --- For 3.11. Thanks, Rafael --- drivers/pci/pci-acpi.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Index: linux-pm/drivers/pci/pci-acpi.c === --- linux-pm.orig/drivers/pci/pci-acpi.c +++ linux-pm/drivers/pci/pci-acpi.c @@ -211,7 +211,7 @@ static int acpi_pci_set_power_state(stru if (!error) dev_info(dev-dev, power state changed by ACPI to %s\n, -pci_power_name(state)); +acpi_power_state_string(state_conv[state])); return error; } Just to double-check this, it *looks* like the effective change is that for PCI_D3hot and PCI_D3cold, we'll print (unknown) instead of D3hot and D3cold because state_conv[] folds both PCI_D3hot and PCI_D3cold into ACPI_STATE_D3, and acpi_power_state_string() doesn't have a case for ACPI_STATE_D3. Is that what you intended? Bjorn -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] PCI / ACPI / PM: Use correct power state strings in messages
On Friday, June 14, 2013 11:08:44 AM Bjorn Helgaas wrote: On Thu, Jun 13, 2013 at 4:29 PM, Rafael J. Wysocki r...@sisk.pl wrote: From: Rafael J. Wysocki rafael.j.wyso...@intel.com Make acpi_pci_set_power_state() print the name of the ACPI device power state the device has been actually put into instead of printing the name of the requested PCI device power state, which need not be the same. Signed-off-by: Rafael J. Wysocki rafael.j.wyso...@intel.com --- For 3.11. Thanks, Rafael --- drivers/pci/pci-acpi.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Index: linux-pm/drivers/pci/pci-acpi.c === --- linux-pm.orig/drivers/pci/pci-acpi.c +++ linux-pm/drivers/pci/pci-acpi.c @@ -211,7 +211,7 @@ static int acpi_pci_set_power_state(stru if (!error) dev_info(dev-dev, power state changed by ACPI to %s\n, -pci_power_name(state)); +acpi_power_state_string(state_conv[state])); return error; } Just to double-check this, it *looks* like the effective change is that for PCI_D3hot and PCI_D3cold, we'll print (unknown) instead of D3hot and D3cold because state_conv[] folds both PCI_D3hot and PCI_D3cold into ACPI_STATE_D3, and acpi_power_state_string() doesn't have a case for ACPI_STATE_D3. No, it won't work like this, because ACPI_STATE_D3 == ACPI_STATE_D3_COLD. :-) So, actually, D3cold will be printed for both PCI_D3hot and PCI_D3cold (and I have tested this). Well, I think it's better to actually replace ACPI_STATE_D3 everywhere in pci-acpi.c with ACPI_STATE_D3_COLD to avoid that confusion. Do you want me to prepare a patch? Rafael -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] PCI / ACPI / PM: Use correct power state strings in messages
On Sat, Jun 15, 2013 at 12:28:12AM +0200, Rafael J. Wysocki wrote: On Friday, June 14, 2013 11:08:44 AM Bjorn Helgaas wrote: On Thu, Jun 13, 2013 at 4:29 PM, Rafael J. Wysocki r...@sisk.pl wrote: From: Rafael J. Wysocki rafael.j.wyso...@intel.com Make acpi_pci_set_power_state() print the name of the ACPI device power state the device has been actually put into instead of printing the name of the requested PCI device power state, which need not be the same. Signed-off-by: Rafael J. Wysocki rafael.j.wyso...@intel.com --- For 3.11. Thanks, Rafael --- drivers/pci/pci-acpi.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Index: linux-pm/drivers/pci/pci-acpi.c === --- linux-pm.orig/drivers/pci/pci-acpi.c +++ linux-pm/drivers/pci/pci-acpi.c @@ -211,7 +211,7 @@ static int acpi_pci_set_power_state(stru if (!error) dev_info(dev-dev, power state changed by ACPI to %s\n, -pci_power_name(state)); +acpi_power_state_string(state_conv[state])); return error; } Just to double-check this, it *looks* like the effective change is that for PCI_D3hot and PCI_D3cold, we'll print (unknown) instead of D3hot and D3cold because state_conv[] folds both PCI_D3hot and PCI_D3cold into ACPI_STATE_D3, and acpi_power_state_string() doesn't have a case for ACPI_STATE_D3. No, it won't work like this, because ACPI_STATE_D3 == ACPI_STATE_D3_COLD. :-) So, actually, D3cold will be printed for both PCI_D3hot and PCI_D3cold (and I have tested this). Ah, right, I should have noticed that. Well, I think it's better to actually replace ACPI_STATE_D3 everywhere in pci-acpi.c with ACPI_STATE_D3_COLD to avoid that confusion. Do you want me to prepare a patch? If the following is OK, I'll just put it in my pci/misc branch: commit fc6504b3a4dc9beae782a11e6f7c3c4a9f077fb8 Author: Rafael J. Wysocki rafael.j.wyso...@intel.com Date: Fri Jun 14 00:29:50 2013 +0200 PCI / ACPI / PM: Use correct power state strings in messages Make acpi_pci_set_power_state() print the name of the ACPI device power state the device has been actually put into instead of printing the name of the requested PCI device power state, which need not be the same. [bhelgaas: use ACPI_STATE_D3_COLD (ACPI_STATE_D3 == ACPI_STATE_D3_COLD)] Signed-off-by: Rafael J. Wysocki rafael.j.wyso...@intel.com Signed-off-by: Bjorn Helgaas bhelg...@google.com diff --git a/drivers/pci/pci-acpi.c b/drivers/pci/pci-acpi.c index 6c15d6a..dbdc5f7 100644 --- a/drivers/pci/pci-acpi.c +++ b/drivers/pci/pci-acpi.c @@ -186,8 +186,8 @@ static int acpi_pci_set_power_state(struct pci_dev *dev, pci_power_t state) [PCI_D0] = ACPI_STATE_D0, [PCI_D1] = ACPI_STATE_D1, [PCI_D2] = ACPI_STATE_D2, - [PCI_D3hot] = ACPI_STATE_D3, - [PCI_D3cold] = ACPI_STATE_D3 + [PCI_D3hot] = ACPI_STATE_D3_COLD, + [PCI_D3cold] = ACPI_STATE_D3_COLD, }; int error = -EINVAL; @@ -211,7 +211,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, -pci_power_name(state)); +acpi_power_state_string(state_conv[state])); return error; } -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] PCI / ACPI / PM: Use correct power state strings in messages
On Friday, June 14, 2013 04:49:49 PM Bjorn Helgaas wrote: On Sat, Jun 15, 2013 at 12:28:12AM +0200, Rafael J. Wysocki wrote: On Friday, June 14, 2013 11:08:44 AM Bjorn Helgaas wrote: On Thu, Jun 13, 2013 at 4:29 PM, Rafael J. Wysocki r...@sisk.pl wrote: From: Rafael J. Wysocki rafael.j.wyso...@intel.com Make acpi_pci_set_power_state() print the name of the ACPI device power state the device has been actually put into instead of printing the name of the requested PCI device power state, which need not be the same. Signed-off-by: Rafael J. Wysocki rafael.j.wyso...@intel.com --- For 3.11. Thanks, Rafael --- drivers/pci/pci-acpi.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Index: linux-pm/drivers/pci/pci-acpi.c === --- linux-pm.orig/drivers/pci/pci-acpi.c +++ linux-pm/drivers/pci/pci-acpi.c @@ -211,7 +211,7 @@ static int acpi_pci_set_power_state(stru if (!error) dev_info(dev-dev, power state changed by ACPI to %s\n, -pci_power_name(state)); +acpi_power_state_string(state_conv[state])); return error; } Just to double-check this, it *looks* like the effective change is that for PCI_D3hot and PCI_D3cold, we'll print (unknown) instead of D3hot and D3cold because state_conv[] folds both PCI_D3hot and PCI_D3cold into ACPI_STATE_D3, and acpi_power_state_string() doesn't have a case for ACPI_STATE_D3. No, it won't work like this, because ACPI_STATE_D3 == ACPI_STATE_D3_COLD. :-) So, actually, D3cold will be printed for both PCI_D3hot and PCI_D3cold (and I have tested this). Ah, right, I should have noticed that. Well, I think it's better to actually replace ACPI_STATE_D3 everywhere in pci-acpi.c with ACPI_STATE_D3_COLD to avoid that confusion. Do you want me to prepare a patch? If the following is OK, I'll just put it in my pci/misc branch: Yes, it is, thanks! Rafael commit fc6504b3a4dc9beae782a11e6f7c3c4a9f077fb8 Author: Rafael J. Wysocki rafael.j.wyso...@intel.com Date: Fri Jun 14 00:29:50 2013 +0200 PCI / ACPI / PM: Use correct power state strings in messages Make acpi_pci_set_power_state() print the name of the ACPI device power state the device has been actually put into instead of printing the name of the requested PCI device power state, which need not be the same. [bhelgaas: use ACPI_STATE_D3_COLD (ACPI_STATE_D3 == ACPI_STATE_D3_COLD)] Signed-off-by: Rafael J. Wysocki rafael.j.wyso...@intel.com Signed-off-by: Bjorn Helgaas bhelg...@google.com diff --git a/drivers/pci/pci-acpi.c b/drivers/pci/pci-acpi.c index 6c15d6a..dbdc5f7 100644 --- a/drivers/pci/pci-acpi.c +++ b/drivers/pci/pci-acpi.c @@ -186,8 +186,8 @@ static int acpi_pci_set_power_state(struct pci_dev *dev, pci_power_t state) [PCI_D0] = ACPI_STATE_D0, [PCI_D1] = ACPI_STATE_D1, [PCI_D2] = ACPI_STATE_D2, - [PCI_D3hot] = ACPI_STATE_D3, - [PCI_D3cold] = ACPI_STATE_D3 + [PCI_D3hot] = ACPI_STATE_D3_COLD, + [PCI_D3cold] = ACPI_STATE_D3_COLD, }; int error = -EINVAL; @@ -211,7 +211,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, - pci_power_name(state)); + 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-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH] PCI / ACPI / PM: Use correct power state strings in messages
From: Rafael J. Wysocki Make acpi_pci_set_power_state() print the name of the ACPI device power state the device has been actually put into instead of printing the name of the requested PCI device power state, which need not be the same. Signed-off-by: Rafael J. Wysocki --- For 3.11. Thanks, Rafael --- drivers/pci/pci-acpi.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Index: linux-pm/drivers/pci/pci-acpi.c === --- linux-pm.orig/drivers/pci/pci-acpi.c +++ linux-pm/drivers/pci/pci-acpi.c @@ -211,7 +211,7 @@ static int acpi_pci_set_power_state(stru if (!error) dev_info(>dev, "power state changed by ACPI to %s\n", -pci_power_name(state)); +acpi_power_state_string(state_conv[state])); return error; } -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH] PCI / ACPI / PM: Use correct power state strings in messages
From: Rafael J. Wysocki rafael.j.wyso...@intel.com Make acpi_pci_set_power_state() print the name of the ACPI device power state the device has been actually put into instead of printing the name of the requested PCI device power state, which need not be the same. Signed-off-by: Rafael J. Wysocki rafael.j.wyso...@intel.com --- For 3.11. Thanks, Rafael --- drivers/pci/pci-acpi.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Index: linux-pm/drivers/pci/pci-acpi.c === --- linux-pm.orig/drivers/pci/pci-acpi.c +++ linux-pm/drivers/pci/pci-acpi.c @@ -211,7 +211,7 @@ static int acpi_pci_set_power_state(stru if (!error) dev_info(dev-dev, power state changed by ACPI to %s\n, -pci_power_name(state)); +acpi_power_state_string(state_conv[state])); return error; } -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/