ACK: [PATCH] acpi: method: fix a few incorrectly idented lines of code

Alex Hung alex.hung at canonical.com
Mon Mar 7 10:13:37 UTC 2016


On 03/03/2016 04:32 PM, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
>
> trivial indentation fix, no functional change.
>
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>   src/acpi/method/method.c | 10 +++++-----
>   1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/src/acpi/method/method.c b/src/acpi/method/method.c
> index edf4e78..19eca6a 100644
> --- a/src/acpi/method/method.c
> +++ b/src/acpi/method/method.c
> @@ -2138,7 +2138,7 @@ static void method_test_CRS_large_resource_items(
>   				"%s GPIO Connection Descriptor has an invalid "
>   				"Connection Type 0x%" PRIx8 ".",
>   				name, data[2]);
> -				*passed = false;
> +			*passed = false;
>   			fwts_advice(fw,
>   				"The GPIO pin connection type is "
>   				"not recognised. It should be either "
> @@ -2153,7 +2153,7 @@ static void method_test_CRS_large_resource_items(
>   				"%s GPIO Connection Descriptor has an invalid "
>   				"Pin Configuration Type 0x%" PRIx8 ".",
>   				name, data[9]);
> -				*passed = false;
> +			*passed = false;
>   			fwts_advice(fw,
>   				"The GPIO pin configuration type "
>   				"is not recognised. It should be one of:"
> @@ -4139,7 +4139,7 @@ static void method_test_LPI_return(
>   	for (i = 0; i < obj->Package.Count; i++) {
>   		if (i < 3) {
>   			if (obj->Package.Elements[i].Type != ACPI_TYPE_INTEGER) {
> -					fwts_failed(fw, LOG_LEVEL_HIGH,
> +				fwts_failed(fw, LOG_LEVEL_HIGH,
>   					"Method_LPIBadElementType",
>   					"%s element %" PRIu32 " is not an integer.", name, i);
>   				failed = true;
> @@ -4168,7 +4168,7 @@ static void method_test_LPI_return(
>   		} else {
>   			ACPI_OBJECT *pkg;
>   			if (obj->Package.Elements[i].Type != ACPI_TYPE_PACKAGE) {
> -					fwts_failed(fw, LOG_LEVEL_HIGH,
> +				fwts_failed(fw, LOG_LEVEL_HIGH,
>   					"Method_LPIBadElementType",
>   					"%s element %" PRIu32 " is not a package.", name, i);
>   				failed = true;
> @@ -4268,7 +4268,7 @@ static void method_test_RDI_return(
>   		pkg = &obj->Package.Elements[i];
>
>   		if (pkg->Type != ACPI_TYPE_PACKAGE) {
> -				fwts_failed(fw, LOG_LEVEL_HIGH,
> +			fwts_failed(fw, LOG_LEVEL_HIGH,
>   				"Method_RDIBadElementType",
>   				"%s element %" PRIu32 " is not a package.", name, i);
>   			failed = true;
>
Acked-by: Alex Hung <alex.hung at canonical.com>



More information about the fwts-devel mailing list