[PATCH][FWTS-TEST] syntaxcheck-0001: bug LP: #1200568 fixes the ASL errors being skipped over

[IvanHu]

On 07/12/2013 07:15 PM, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
>
> This fixes the test now that bug LP: #1200568 is fixed.
>
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>   syntaxcheck-0001/syntaxcheck-0001.log | 70 +++++++++++++++++++++++++++++++++--
>   1 file changed, 67 insertions(+), 3 deletions(-)
>
> diff --git a/syntaxcheck-0001/syntaxcheck-0001.log b/syntaxcheck-0001/syntaxcheck-0001.log
> index 679967a..80a7b80 100644
> --- a/syntaxcheck-0001/syntaxcheck-0001.log
> +++ b/syntaxcheck-0001/syntaxcheck-0001.log
> @@ -5,8 +5,72 @@ syntaxcheck     Test 1 of 2: Disassemble and reassemble DSDT
>   syntaxcheck
>   syntaxcheck     Checking ACPI table DSDT (#0)
>   syntaxcheck
> -syntaxcheck     PASSED: Test 1, DSDT (0) reassembly, Found 0 errors, 0
> -syntaxcheck     warnings.
> +syntaxcheck     FAILED [HIGH] AMLAsmASL_MSG_UPPER_CASE: Test 1, Assembler
> +syntaxcheck     error in line 709
> +syntaxcheck     Line | AML source
> +syntaxcheck     ----------------------------------------------------------
> +syntaxcheck     00706|     {
> +syntaxcheck     00707|         Device (AMW0)
> +syntaxcheck     00708|         {
> +syntaxcheck     00709|             Name (_HID, "pnp0c14")  // _HID: Hardware ID
> +syntaxcheck          |                                ^
> +syntaxcheck          | Error 6136: Non-hex letters must be upper case    (pnp0c14)
> +syntaxcheck     00710|             Name (_UID, Zero)  // _UID: Unique ID
> +syntaxcheck     00711|             Name (_WDG, Buffer (0x3C)
> +syntaxcheck     00712|             {
> +syntaxcheck     ==========================================================
> +syntaxcheck
> +syntaxcheck     ADVICE: (for Error #6136, ASL_MSG_UPPER_CASE): Characters
> +syntaxcheck     in literal string that are not hexadecimal letters must be
> +syntaxcheck     upper case.
> +syntaxcheck
> +syntaxcheck     FAILED [HIGH] AMLAsmASL_MSG_INVALID_ADDR_FLAGS: Test 1,
> +syntaxcheck     Assembler error in line 1069
> +syntaxcheck     Line | AML source
> +syntaxcheck     ----------------------------------------------------------
> +syntaxcheck     01066|                     0x00000000,         // Range Minimum
> +syntaxcheck     01067|                     0xDFFFFFFF,         // Range Maximum
> +syntaxcheck     01068|                     0x00000000,         // Translation Offset
> +syntaxcheck     01069|                     0x00000000,         // Length
> +syntaxcheck          |                             ^
> +syntaxcheck          | Error 6043: Invalid combination of Length and Min/Max fixed flags
> +syntaxcheck     01070|                     ,, _Y0D, AddressRangeMemory, TypeStatic)
> +syntaxcheck     01071|                 DWordMemory (ResourceProducer, PosDecode, MinFixed, MaxFixed, Cacheable, ReadWrite,
> +syntaxcheck     01072|                     0x00000000,         // Granularity
> +syntaxcheck     ==========================================================
> +syntaxcheck
> +syntaxcheck     ADVICE: (for Error #6043, ASL_MSG_INVALID_ADDR_FLAGS):
> +syntaxcheck     This occurs if the length is zero and just one of the
> +syntaxcheck     resource MIF/MAF flags are set, or the length is non-zero
> +syntaxcheck     and resource MIF/MAF flags are both set. These are illegal
> +syntaxcheck     combinations and need to be fixed. See section 6.4.3.5
> +syntaxcheck     Address Space Resource Descriptors of the ACPI
> +syntaxcheck     specification for more details.
> +syntaxcheck
> +syntaxcheck     FAILED [HIGH] AMLAsmASL_MSG_INVALID_ADDR_FLAGS: Test 1,
> +syntaxcheck     Assembler error in line 1083
> +syntaxcheck     Line | AML source
> +syntaxcheck     ----------------------------------------------------------
> +syntaxcheck     01080|                     0xFED40000,         // Range Minimum
> +syntaxcheck     01081|                     0xFED44FFF,         // Range Maximum
> +syntaxcheck     01082|                     0x00000000,         // Translation Offset
> +syntaxcheck     01083|                     0x00000000,         // Length
> +syntaxcheck          |                             ^
> +syntaxcheck          | Error 6043: Invalid combination of Length and Min/Max fixed flags
> +syntaxcheck     01084|                     ,, , AddressRangeMemory, TypeStatic)
> +syntaxcheck     01085|             })
> +syntaxcheck     01086|             Method (_CRS, 0, Serialized)  // _CRS: Current Resource Settings
> +syntaxcheck     ==========================================================
> +syntaxcheck
> +syntaxcheck     ADVICE: (for Error #6043, ASL_MSG_INVALID_ADDR_FLAGS):
> +syntaxcheck     This occurs if the length is zero and just one of the
> +syntaxcheck     resource MIF/MAF flags are set, or the length is non-zero
> +syntaxcheck     and resource MIF/MAF flags are both set. These are illegal
> +syntaxcheck     combinations and need to be fixed. See section 6.4.3.5
> +syntaxcheck     Address Space Resource Descriptors of the ACPI
> +syntaxcheck     specification for more details.
> +syntaxcheck
> +syntaxcheck     Table DSDT (0) reassembly: Found 3 errors, 0 warnings.
>   syntaxcheck
>   syntaxcheck
>   syntaxcheck     Test 2 of 2: Disassemble and reassemble SSDT
> @@ -42,6 +106,6 @@ syntaxcheck     warnings.
>   syntaxcheck
>   syntaxcheck
>   syntaxcheck     ==========================================================
> -syntaxcheck     6 passed, 0 failed, 0 warnings, 0 aborted, 0 skipped, 0
> +syntaxcheck     5 passed, 3 failed, 0 warnings, 0 aborted, 0 skipped, 0
>   syntaxcheck     info only.
>   syntaxcheck     ==========================================================
>

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