Patchwork acpi: method: Show the name of the _PSS control

login
register
mail settings
Submitter Colin King
Date Jan. 8, 2013, 11:29 a.m.
Message ID <1357644597-10185-1-git-send-email-colin.king@canonical.com>
Download mbox | patch
Permalink /patch/210345/
State Accepted
Headers show

Comments

Colin King - Jan. 8, 2013, 11:29 a.m.
From: Colin Ian King <colin.king@canonical.com>

Output the name of the _PSS control that is being dumped.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
 src/acpi/method/method.c | 1 +
 1 file changed, 1 insertion(+)
Keng-Yu Lin - Jan. 23, 2013, 7:03 a.m.
On Tue, Jan 8, 2013 at 7:29 PM, Colin King <colin.king@canonical.com> wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> Output the name of the _PSS control that is being dumped.
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> ---
>  src/acpi/method/method.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/src/acpi/method/method.c b/src/acpi/method/method.c
> index 231b5fb..2d6c0d8 100644
> --- a/src/acpi/method/method.c
> +++ b/src/acpi/method/method.c
> @@ -2481,6 +2481,7 @@ static void method_test_PSS_return(
>          *  If we have some valid data then dump it out, it is useful to see
>          */
>         if (dump_elements) {
> +               fwts_log_info_verbatum(fw, "%s values:", name);
>                 fwts_log_info_verbatum(fw, "P-State  Freq     Power  Latency   Bus Master");
>                 fwts_log_info_verbatum(fw, "         (MHz)    (mW)    (us)    Latency (us)");
>                 for (i = 0; i < obj->Package.Count; i++) {
> --
> 1.8.0
>
Acked-by: Keng-Yu Lin <kengyu@canonical.com>
Ivan Hu - Jan. 31, 2013, 1:51 a.m.
On 01/08/2013 07:29 PM, Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> Output the name of the _PSS control that is being dumped.
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> ---
>   src/acpi/method/method.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/src/acpi/method/method.c b/src/acpi/method/method.c
> index 231b5fb..2d6c0d8 100644
> --- a/src/acpi/method/method.c
> +++ b/src/acpi/method/method.c
> @@ -2481,6 +2481,7 @@ static void method_test_PSS_return(
>   	 *  If we have some valid data then dump it out, it is useful to see
>   	 */
>   	if (dump_elements) {
> +		fwts_log_info_verbatum(fw, "%s values:", name);
>   		fwts_log_info_verbatum(fw, "P-State  Freq     Power  Latency   Bus Master");
>   		fwts_log_info_verbatum(fw, "         (MHz)    (mW)    (us)    Latency (us)");
>   		for (i = 0; i < obj->Package.Count; i++) {
>

Acked-by: Ivan Hu <ivan.hu@canonical.com>

Patch

diff --git a/src/acpi/method/method.c b/src/acpi/method/method.c
index 231b5fb..2d6c0d8 100644
--- a/src/acpi/method/method.c
+++ b/src/acpi/method/method.c
@@ -2481,6 +2481,7 @@  static void method_test_PSS_return(
 	 *  If we have some valid data then dump it out, it is useful to see
 	 */
 	if (dump_elements) {
+		fwts_log_info_verbatum(fw, "%s values:", name);
 		fwts_log_info_verbatum(fw, "P-State  Freq     Power  Latency   Bus Master");
 		fwts_log_info_verbatum(fw, "         (MHz)    (mW)    (us)    Latency (us)");
 		for (i = 0; i < obj->Package.Count; i++) {