[PATCH] apci: aspm: clean up messages
Colin King
colin.king at canonical.com
Fri Dec 7 18:33:20 UTC 2012
From: Colin Ian King <colin.king at canonical.com>
Fix a few messages:
aspm -> ASPM
PCIE -> PCIe
..and emit a message when lspci fails to produce any output.
Signed-off-by: Colin Ian King <colin.king at canonical.com>
---
src/pci/aspm/aspm.c | 22 ++++++++++++----------
1 file changed, 12 insertions(+), 10 deletions(-)
diff --git a/src/pci/aspm/aspm.c b/src/pci/aspm/aspm.c
index a2cda8c..7d1e53f 100644
--- a/src/pci/aspm/aspm.c
+++ b/src/pci/aspm/aspm.c
@@ -100,10 +100,10 @@ static int facp_get_aspm_control(fwts_framework *fw, int *aspm)
if ((fadt->iapc_boot_arch & FWTS_FACP_IAPC_BOOT_ARCH_PCIE_ASPM_CONTROLS) == 0) {
*aspm = 1;
- fwts_log_info(fw, "PCIE ASPM is controlled by Linux kernel.");
+ fwts_log_info(fw, "PCIe ASPM is controlled by Linux kernel.");
} else {
*aspm = 0;
- fwts_log_info(fw, "PCIE ASPM is not controlled by Linux kernel.");
+ fwts_log_info(fw, "PCIe ASPM is not controlled by Linux kernel.");
fwts_advice(fw,
"BIOS reports that Linux kernel should not modify ASPM "
"settings that BIOS configured. It can be intentional "
@@ -192,11 +192,11 @@ static int pcie_compare_rp_dev_aspm_registers(fwts_framework *fw,
rp_aspm_cntrl = rp_cap->link_contrl & FWTS_PCIE_ASPM_CONTROL_FIELD;
device_aspm_cntrl = device_cap->link_contrl & FWTS_PCIE_ASPM_CONTROL_FIELD;
if (rp_aspm_cntrl != device_aspm_cntrl) {
- fwts_failed(fw, LOG_LEVEL_MEDIUM, "PCIEASPM_UNMATCHED",
- "PCIE aspm setting was not matched.");
- fwts_log_error(fw, "RP %02Xh:%02Xh.%02Xh has aspm = %02Xh."
+ fwts_failed(fw, LOG_LEVEL_MEDIUM, "PCIEASPM_Unmatched",
+ "PCIe ASPM setting was not matched.");
+ fwts_log_error(fw, "RP %02Xh:%02Xh.%02Xh has ASPM = %02Xh."
, rp->bus, rp->dev, rp->func, rp_aspm_cntrl);
- fwts_log_error(fw, "Device %02Xh:%02Xh.%02Xh has aspm = %02Xh.",
+ fwts_log_error(fw, "Device %02Xh:%02Xh.%02Xh has ASPM = %02Xh.",
dev->bus, dev->dev, dev->func, device_aspm_cntrl);
fwts_advice(fw,
"ASPM control registers between root port and device "
@@ -205,7 +205,7 @@ static int pcie_compare_rp_dev_aspm_registers(fwts_framework *fw,
"ASPM correctly and the system is not saving power "
"at its full potential.");
} else {
- fwts_passed(fw, "PCIE aspm setting matched was matched.");
+ fwts_passed(fw, "PCIe ASPM setting matched was matched.");
}
return ret;
@@ -234,11 +234,13 @@ static int pcie_check_aspm_registers(fwts_framework *fw)
snprintf(command, sizeof(command), "%s", fw->lspci);
if (fwts_pipe_exec(command, &lspci_output, &status) != FWTS_OK) {
- fwts_log_warning(fw, "Could not execute %s", command);
+ fwts_log_warning(fw, "Could not execute %s.", command);
return FWTS_ERROR;
}
- if (lspci_output == NULL)
+ if (lspci_output == NULL) {
+ fwts_log_warning(fw, "Output from lspci was empty.");
return FWTS_ERROR;
+ }
/* Get the list of pci devices and their configuration */
fwts_list_foreach(item, lspci_output) {
@@ -330,7 +332,7 @@ static int aspm_check_configuration(fwts_framework *fw)
ret = facp_get_aspm_control(fw, &aspm_facp);
if (ret == FWTS_ERROR) {
- fwts_log_info(fw, "No valid FACP information present: cannot test aspm.");
+ fwts_log_info(fw, "No valid FACP information present: cannot test ASPM.");
return FWTS_ERROR;
}
--
1.8.0
More information about the fwts-devel
mailing list