ACK: [PATCH] acpi: fwts-tests: Remove redundant tailing space and update fwts-tests

Colin Ian King colin.king at canonical.com
Sat May 30 07:55:35 UTC 2015


On 30/05/15 07:31, Alex Hung wrote:
> Signed-off-by: Alex Hung <alex.hung at canonical.com>
> ---
>  .../arg-show-tests-full-0001.log                         | 16 ++++++++--------
>  fwts-test/method-0001/method-0001.log                    |  4 ++--
>  src/acpi/method/method.c                                 |  4 ++--
>  src/hpet/hpet_check/hpet_check.c                         |  4 ++--
>  4 files changed, 14 insertions(+), 14 deletions(-)
> 
> diff --git a/fwts-test/arg-show-tests-full-0001/arg-show-tests-full-0001.log b/fwts-test/arg-show-tests-full-0001/arg-show-tests-full-0001.log
> index b32cf2e..bf0fb97 100644
> --- a/fwts-test/arg-show-tests-full-0001/arg-show-tests-full-0001.log
> +++ b/fwts-test/arg-show-tests-full-0001/arg-show-tests-full-0001.log
> @@ -19,8 +19,8 @@ ACPI tests:
>    Test FADT reset register.
>   hpet_check      (4 tests):
>    Test HPET base in kernel log.
> -  Test HPET base in HPET table. 
> -  Test HPET base in DSDT and/or SSDT. 
> +  Test HPET base in HPET table.
> +  Test HPET base in DSDT and/or SSDT.
>    Test HPET configuration.
>   mcfg            (2 tests):
>    Validate MCFG table.
> @@ -121,8 +121,8 @@ ACPI tests:
>    Test _ALC (Ambient Light Colour Chromaticity).
>    Test _ALI (Ambient Light Illuminance).
>    Test _ALT (Ambient Light Temperature).
> -  Test _ALP (Ambient Light Polling). 
> -  Test _ALR (Ambient Light Response). 
> +  Test _ALP (Ambient Light Polling).
> +  Test _ALR (Ambient Light Response).
>    Test _LID (Lid Status).
>    Test _GTF (Get Task File).
>    Test _GTM (Get Timing Mode).
> @@ -262,8 +262,8 @@ Batch tests:
>    HDA Audio Pin Configuration test.
>   hpet_check      (4 tests):
>    Test HPET base in kernel log.
> -  Test HPET base in HPET table. 
> -  Test HPET base in DSDT and/or SSDT. 
> +  Test HPET base in HPET table.
> +  Test HPET base in DSDT and/or SSDT.
>    Test HPET configuration.
>   klog            (1 test):
>    Kernel log error check.
> @@ -370,8 +370,8 @@ Batch tests:
>    Test _ALC (Ambient Light Colour Chromaticity).
>    Test _ALI (Ambient Light Illuminance).
>    Test _ALT (Ambient Light Temperature).
> -  Test _ALP (Ambient Light Polling). 
> -  Test _ALR (Ambient Light Response). 
> +  Test _ALP (Ambient Light Polling).
> +  Test _ALR (Ambient Light Response).
>    Test _LID (Lid Status).
>    Test _GTF (Get Task File).
>    Test _GTM (Get Timing Mode).
> diff --git a/fwts-test/method-0001/method-0001.log b/fwts-test/method-0001/method-0001.log
> index 4963154..2c9ffc7 100644
> --- a/fwts-test/method-0001/method-0001.log
> +++ b/fwts-test/method-0001/method-0001.log
> @@ -641,11 +641,11 @@ method          Test 95 of 180: Test _ALT (Ambient Light Temperature).
>  method          SKIPPED: Test 95, Skipping test for non-existant object
>  method          _ALT.
>  method          
> -method          Test 96 of 180: Test _ALP (Ambient Light Polling). 
> +method          Test 96 of 180: Test _ALP (Ambient Light Polling).
>  method          SKIPPED: Test 96, Skipping test for non-existant object
>  method          _ALP.
>  method          
> -method          Test 97 of 180: Test _ALR (Ambient Light Response). 
> +method          Test 97 of 180: Test _ALR (Ambient Light Response).
>  method          SKIPPED: Test 97, Skipping test for non-existant object
>  method          _ALR.
>  method          
> diff --git a/src/acpi/method/method.c b/src/acpi/method/method.c
> index 828e7aa..c9bf6f9 100644
> --- a/src/acpi/method/method.c
> +++ b/src/acpi/method/method.c
> @@ -6212,8 +6212,8 @@ static fwts_framework_minor_test method_tests[] = {
>  	{ method_test_ALC, "Test _ALC (Ambient Light Colour Chromaticity)." },
>  	{ method_test_ALI, "Test _ALI (Ambient Light Illuminance)." },
>  	{ method_test_ALT, "Test _ALT (Ambient Light Temperature)." },
> -	{ method_test_ALP, "Test _ALP (Ambient Light Polling). "},
> -	{ method_test_ALR, "Test _ALR (Ambient Light Response). "},
> +	{ method_test_ALP, "Test _ALP (Ambient Light Polling)."},
> +	{ method_test_ALR, "Test _ALR (Ambient Light Response)."},
>  
>  	/* Section 9.3 Battery Device */
>  
> diff --git a/src/hpet/hpet_check/hpet_check.c b/src/hpet/hpet_check/hpet_check.c
> index 7699bb2..6ccd3b0 100644
> --- a/src/hpet/hpet_check/hpet_check.c
> +++ b/src/hpet/hpet_check/hpet_check.c
> @@ -424,8 +424,8 @@ static int hpet_check_test4(fwts_framework *fw)
>  
>  static fwts_framework_minor_test hpet_check_tests[] = {
>  	{ hpet_check_test1, "Test HPET base in kernel log." },
> -	{ hpet_check_test2, "Test HPET base in HPET table. "},
> -	{ hpet_check_test3, "Test HPET base in DSDT and/or SSDT. "},
> +	{ hpet_check_test2, "Test HPET base in HPET table."},
> +	{ hpet_check_test3, "Test HPET base in DSDT and/or SSDT."},
>  	{ hpet_check_test4, "Test HPET configuration." },
>  	{ NULL, NULL }
>  };
> 
Thanks for spotting these.

Acked-by: Colin Ian King <colin.king at canonical.com>



More information about the fwts-devel mailing list