diff mbox

acpi: method: fix incorrect failure messages for _TRT

Message ID 1410933812-21044-1-git-send-email-alex.hung@canonical.com
State Accepted
Headers show

Commit Message

Alex Hung Sept. 17, 2014, 6:03 a.m. UTC 
Signed-off-by: Alex Hung <alex.hung@canonical.com>
---
 src/acpi/method/method.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Colin Ian King Sept. 17, 2014, 7:46 a.m. UTC | #1 
On 17/09/14 07:03, Alex Hung wrote:
> Signed-off-by: Alex Hung <alex.hung@canonical.com>
> ---
>  src/acpi/method/method.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/acpi/method/method.c b/src/acpi/method/method.c
> index 51fefce..a21e4b1 100644
> --- a/src/acpi/method/method.c
> +++ b/src/acpi/method/method.c
> @@ -4650,7 +4650,7 @@ static void method_test_TRT_return(
>  		for (j = 0; j < 2; j++) {
>  			if (pkg->Package.Elements[j].Type != ACPI_TYPE_LOCAL_REFERENCE) {
>  				fwts_failed(fw, LOG_LEVEL_MEDIUM,
> -					"Method_TRTSubPackageElementCount",
> +					"Method_TRTBadSubPackageReturnType",
>  					"%s sub-package %" PRIu32
>  					" element %" PRIu32 " is not "
>  					"a reference.",
> @@ -4662,7 +4662,7 @@ static void method_test_TRT_return(
>  		for (j = 2; j < 8; j++) {
>  			if (pkg->Package.Elements[j].Type != ACPI_TYPE_INTEGER) {
>  				fwts_failed(fw, LOG_LEVEL_MEDIUM,
> -					"Method_TRTSubPackageElementCount",
> +					"Method_TRTBadSubPackageReturnType",
>  					"%s sub-package %" PRIu32
>  					" element %" PRIu32 " is not "
>  					"an integer.",
> 
Acked-by: Colin Ian King <colin.king@canonical.com>
Ivan Hu Sept. 17, 2014, 8:34 a.m. UTC | #2 
On 09/17/2014 02:03 PM, Alex Hung wrote:
> Signed-off-by: Alex Hung <alex.hung@canonical.com>
> ---
>   src/acpi/method/method.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/acpi/method/method.c b/src/acpi/method/method.c
> index 51fefce..a21e4b1 100644
> --- a/src/acpi/method/method.c
> +++ b/src/acpi/method/method.c
> @@ -4650,7 +4650,7 @@ static void method_test_TRT_return(
>   		for (j = 0; j < 2; j++) {
>   			if (pkg->Package.Elements[j].Type != ACPI_TYPE_LOCAL_REFERENCE) {
>   				fwts_failed(fw, LOG_LEVEL_MEDIUM,
> -					"Method_TRTSubPackageElementCount",
> +					"Method_TRTBadSubPackageReturnType",
>   					"%s sub-package %" PRIu32
>   					" element %" PRIu32 " is not "
>   					"a reference.",
> @@ -4662,7 +4662,7 @@ static void method_test_TRT_return(
>   		for (j = 2; j < 8; j++) {
>   			if (pkg->Package.Elements[j].Type != ACPI_TYPE_INTEGER) {
>   				fwts_failed(fw, LOG_LEVEL_MEDIUM,
> -					"Method_TRTSubPackageElementCount",
> +					"Method_TRTBadSubPackageReturnType",
>   					"%s sub-package %" PRIu32
>   					" element %" PRIu32 " is not "
>   					"an integer.",
>

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

Patch

diff --git a/src/acpi/method/method.c b/src/acpi/method/method.c
index 51fefce..a21e4b1 100644
--- a/src/acpi/method/method.c
+++ b/src/acpi/method/method.c
@@ -4650,7 +4650,7 @@  static void method_test_TRT_return(
 		for (j = 0; j < 2; j++) {
 			if (pkg->Package.Elements[j].Type != ACPI_TYPE_LOCAL_REFERENCE) {
 				fwts_failed(fw, LOG_LEVEL_MEDIUM,
-					"Method_TRTSubPackageElementCount",
+					"Method_TRTBadSubPackageReturnType",
 					"%s sub-package %" PRIu32
 					" element %" PRIu32 " is not "
 					"a reference.",
@@ -4662,7 +4662,7 @@  static void method_test_TRT_return(
 		for (j = 2; j < 8; j++) {
 			if (pkg->Package.Elements[j].Type != ACPI_TYPE_INTEGER) {
 				fwts_failed(fw, LOG_LEVEL_MEDIUM,
-					"Method_TRTSubPackageElementCount",
+					"Method_TRTBadSubPackageReturnType",
 					"%s sub-package %" PRIu32
 					" element %" PRIu32 " is not "
 					"an integer.",