Patchwork arg-show-tests-full-0001.log: add aspm test and fix spelling error

login
register
mail settings
Submitter Keng-Yu Lin
Date Feb. 1, 2012, 7:32 a.m.
Message ID <1328081559-17849-1-git-send-email-kengyu@canonical.com>
Download mbox | patch
Permalink /patch/138915/
State Accepted
Headers show

Comments

Keng-Yu Lin - Feb. 1, 2012, 7:32 a.m.
Signed-off-by: Keng-Yu Lin <kengyu@canonical.com>
---
 .../arg-show-tests-full-0001.log                   |    6 ++++--
 1 files changed, 4 insertions(+), 2 deletions(-)
Colin King - Feb. 1, 2012, 2:48 p.m.
On 01/02/12 07:32, Keng-Yu Lin wrote:
> Signed-off-by: Keng-Yu Lin<kengyu@canonical.com>
> ---
>   .../arg-show-tests-full-0001.log                   |    6 ++++--
>   1 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/arg-show-tests-full-0001/arg-show-tests-full-0001.log b/arg-show-tests-full-0001/arg-show-tests-full-0001.log
> index 33c2b01..d9599f1 100644
> --- a/arg-show-tests-full-0001/arg-show-tests-full-0001.log
> +++ b/arg-show-tests-full-0001/arg-show-tests-full-0001.log
> @@ -7,6 +7,8 @@ Batch tests:
>     Legacy and PCI Interrupt Edge/Level trigger checks.
>    apicinstance  (1 test):
>     Check single instance of APIC/MADT table.
> + aspm          (1 test):
> +  PCIe ASPM configuration test.
>    bios32        (1 test):
>     Check BIOS32 Service Directory.
>    bios_info     (1 test):
> @@ -16,7 +18,7 @@ Batch tests:
>    crs           (1 test):
>     Check PCI host bridge configuration using _CRS.
>    csm           (1 test):
> -  Check for UEFI Compatability Support Module.
> +  Check for UEFI Compatibility Support Module.
>    cstates       (1 test):
>     Check all CPUs C-states.
>    dmesg_common  (1 test):
> @@ -245,4 +247,4 @@ Utilities:
>    uefidump      (1 test):
>     Dump UEFI Variables.
>
> -Total of 181 tests
> +Total of 182 tests

Thanks for taking care of this Keng-Yu!

Acked-by: Colin King <colin.king@canonical.com>
Alex Hung - Feb. 9, 2012, 3:16 a.m.
On 02/01/2012 03:32 PM, Keng-Yu Lin wrote:
> Signed-off-by: Keng-Yu Lin<kengyu@canonical.com>
> ---
>   .../arg-show-tests-full-0001.log                   |    6 ++++--
>   1 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/arg-show-tests-full-0001/arg-show-tests-full-0001.log b/arg-show-tests-full-0001/arg-show-tests-full-0001.log
> index 33c2b01..d9599f1 100644
> --- a/arg-show-tests-full-0001/arg-show-tests-full-0001.log
> +++ b/arg-show-tests-full-0001/arg-show-tests-full-0001.log
> @@ -7,6 +7,8 @@ Batch tests:
>     Legacy and PCI Interrupt Edge/Level trigger checks.
>    apicinstance  (1 test):
>     Check single instance of APIC/MADT table.
> + aspm          (1 test):
> +  PCIe ASPM configuration test.
>    bios32        (1 test):
>     Check BIOS32 Service Directory.
>    bios_info     (1 test):
> @@ -16,7 +18,7 @@ Batch tests:
>    crs           (1 test):
>     Check PCI host bridge configuration using _CRS.
>    csm           (1 test):
> -  Check for UEFI Compatability Support Module.
> +  Check for UEFI Compatibility Support Module.
>    cstates       (1 test):
>     Check all CPUs C-states.
>    dmesg_common  (1 test):
> @@ -245,4 +247,4 @@ Utilities:
>    uefidump      (1 test):
>     Dump UEFI Variables.
>
> -Total of 181 tests
> +Total of 182 tests
Acked by: Alex Hung <alex.hung@canonical.com>
Colin King - Feb. 9, 2012, 9:34 a.m.
On 01/02/12 07:32, Keng-Yu Lin wrote:
> Signed-off-by: Keng-Yu Lin<kengyu@canonical.com>
> ---
>   .../arg-show-tests-full-0001.log                   |    6 ++++--
>   1 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/arg-show-tests-full-0001/arg-show-tests-full-0001.log b/arg-show-tests-full-0001/arg-show-tests-full-0001.log
> index 33c2b01..d9599f1 100644
> --- a/arg-show-tests-full-0001/arg-show-tests-full-0001.log
> +++ b/arg-show-tests-full-0001/arg-show-tests-full-0001.log
> @@ -7,6 +7,8 @@ Batch tests:
>     Legacy and PCI Interrupt Edge/Level trigger checks.
>    apicinstance  (1 test):
>     Check single instance of APIC/MADT table.
> + aspm          (1 test):
> +  PCIe ASPM configuration test.
>    bios32        (1 test):
>     Check BIOS32 Service Directory.
>    bios_info     (1 test):
> @@ -16,7 +18,7 @@ Batch tests:
>    crs           (1 test):
>     Check PCI host bridge configuration using _CRS.
>    csm           (1 test):
> -  Check for UEFI Compatability Support Module.
> +  Check for UEFI Compatibility Support Module.
>    cstates       (1 test):
>     Check all CPUs C-states.
>    dmesg_common  (1 test):
> @@ -245,4 +247,4 @@ Utilities:
>    uefidump      (1 test):
>     Dump UEFI Variables.
>
> -Total of 181 tests
> +Total of 182 tests

Hmm, this is not showing up as ack'd by me in checkpatch, ack'ing again.

Acked-by: Colin King <colin.king@canonical.com>
Keng-Yu Lin - Feb. 9, 2012, 9:46 a.m.
On Thu, Feb 9, 2012 at 5:34 PM, Colin Ian King <colin.king@canonical.com> wrote:
> On 01/02/12 07:32, Keng-Yu Lin wrote:
>>
>> Signed-off-by: Keng-Yu Lin<kengyu@canonical.com>
>> ---
>>  .../arg-show-tests-full-0001.log                   |    6 ++++--
>>  1 files changed, 4 insertions(+), 2 deletions(-)
>>
>> diff --git a/arg-show-tests-full-0001/arg-show-tests-full-0001.log
>> b/arg-show-tests-full-0001/arg-show-tests-full-0001.log
>> index 33c2b01..d9599f1 100644
>> --- a/arg-show-tests-full-0001/arg-show-tests-full-0001.log
>> +++ b/arg-show-tests-full-0001/arg-show-tests-full-0001.log
>> @@ -7,6 +7,8 @@ Batch tests:
>>    Legacy and PCI Interrupt Edge/Level trigger checks.
>>   apicinstance  (1 test):
>>    Check single instance of APIC/MADT table.
>> + aspm          (1 test):
>> +  PCIe ASPM configuration test.
>>   bios32        (1 test):
>>    Check BIOS32 Service Directory.
>>   bios_info     (1 test):
>> @@ -16,7 +18,7 @@ Batch tests:
>>   crs           (1 test):
>>    Check PCI host bridge configuration using _CRS.
>>   csm           (1 test):
>> -  Check for UEFI Compatability Support Module.
>> +  Check for UEFI Compatibility Support Module.
>>   cstates       (1 test):
>>    Check all CPUs C-states.
>>   dmesg_common  (1 test):
>> @@ -245,4 +247,4 @@ Utilities:
>>   uefidump      (1 test):
>>    Dump UEFI Variables.
>>
>> -Total of 181 tests
>> +Total of 182 tests
>
>
> Hmm, this is not showing up as ack'd by me in checkpatch, ack'ing again.
>
> Acked-by: Colin King <colin.king@canonical.com>


Hi Colin,
 Just wonder if you'd like to ack another patch for
arg-show-tests-0001.log? this one is for arg-show-tests-full-0001.log.

Patch

diff --git a/arg-show-tests-full-0001/arg-show-tests-full-0001.log b/arg-show-tests-full-0001/arg-show-tests-full-0001.log
index 33c2b01..d9599f1 100644
--- a/arg-show-tests-full-0001/arg-show-tests-full-0001.log
+++ b/arg-show-tests-full-0001/arg-show-tests-full-0001.log
@@ -7,6 +7,8 @@  Batch tests:
   Legacy and PCI Interrupt Edge/Level trigger checks.
  apicinstance  (1 test):
   Check single instance of APIC/MADT table.
+ aspm          (1 test):
+  PCIe ASPM configuration test.
  bios32        (1 test):
   Check BIOS32 Service Directory.
  bios_info     (1 test):
@@ -16,7 +18,7 @@  Batch tests:
  crs           (1 test):
   Check PCI host bridge configuration using _CRS.
  csm           (1 test):
-  Check for UEFI Compatability Support Module.
+  Check for UEFI Compatibility Support Module.
  cstates       (1 test):
   Check all CPUs C-states.
  dmesg_common  (1 test):
@@ -245,4 +247,4 @@  Utilities:
  uefidump      (1 test):
   Dump UEFI Variables.
 
-Total of 181 tests
+Total of 182 tests