iASL: Use ACPI_COMPARE_NAMESEG instead of strcmp

1193569 diff mbox series

From: Robert Moore <Robert.Moore@intel.com>

BugLink: https://bugs.launchpad.net/bugs/1819968

For _HID, _CID processing. Reported by Benjamin Berg
(benjamin@sipsolutions.net).

(backport from ACPICA upstream commit 3dcd13aa62e7)
Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
 src/acpica/source/compiler/aslmethod.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

On 2019-11-12 7:29 a.m., Colin King wrote:
> From: Robert Moore <Robert.Moore@intel.com>
> 
> BugLink: https://bugs.launchpad.net/bugs/1819968

The buglink is incorrect and it should be
https://bugs.launchpad.net/fwts/+bug/1851219 after confirming with
Colin. I can fix this up when merging this patch once it receives two acks.

> 
> For _HID, _CID processing. Reported by Benjamin Berg
> (benjamin@sipsolutions.net).
> 
> (backport from ACPICA upstream commit 3dcd13aa62e7)
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> ---
>  src/acpica/source/compiler/aslmethod.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/acpica/source/compiler/aslmethod.c b/src/acpica/source/compiler/aslmethod.c
> index 667eaaaf..4b29fc9e 100644
> --- a/src/acpica/source/compiler/aslmethod.c
> +++ b/src/acpica/source/compiler/aslmethod.c
> @@ -586,7 +586,7 @@ MtMethodAnalysisWalkBegin (
>  
>          /* Special typechecking for _HID */
>  
> -        if (!strcmp (METHOD_NAME__HID, Op->Asl.NameSeg))
> +        if (ACPI_COMPARE_NAMESEG (METHOD_NAME__HID, Op->Asl.NameSeg))
>          {
>              Next = Op->Asl.Child->Asl.Next;
>              AnCheckId (Next, ASL_TYPE_HID);
> @@ -594,7 +594,7 @@ MtMethodAnalysisWalkBegin (
>  
>          /* Special typechecking for _CID */
>  
> -        else if (!strcmp (METHOD_NAME__CID, Op->Asl.NameSeg))
> +        else if (ACPI_COMPARE_NAMESEG (METHOD_NAME__CID, Op->Asl.NameSeg))
>          {
>              Next = Op->Asl.Child->Asl.Next;
>  
> 

Acked-by: Alex Hung <alex.hung@canonical.com>

On 11/12/19 10:29 PM, Colin King wrote:
> From: Robert Moore <Robert.Moore@intel.com>
>
> BugLink: https://bugs.launchpad.net/bugs/1819968
>
> For _HID, _CID processing. Reported by Benjamin Berg
> (benjamin@sipsolutions.net).
>
> (backport from ACPICA upstream commit 3dcd13aa62e7)
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> ---
>  src/acpica/source/compiler/aslmethod.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/acpica/source/compiler/aslmethod.c b/src/acpica/source/compiler/aslmethod.c
> index 667eaaaf..4b29fc9e 100644
> --- a/src/acpica/source/compiler/aslmethod.c
> +++ b/src/acpica/source/compiler/aslmethod.c
> @@ -586,7 +586,7 @@ MtMethodAnalysisWalkBegin (
>  
>          /* Special typechecking for _HID */
>  
> -        if (!strcmp (METHOD_NAME__HID, Op->Asl.NameSeg))
> +        if (ACPI_COMPARE_NAMESEG (METHOD_NAME__HID, Op->Asl.NameSeg))
>          {
>              Next = Op->Asl.Child->Asl.Next;
>              AnCheckId (Next, ASL_TYPE_HID);
> @@ -594,7 +594,7 @@ MtMethodAnalysisWalkBegin (
>  
>          /* Special typechecking for _CID */
>  
> -        else if (!strcmp (METHOD_NAME__CID, Op->Asl.NameSeg))
> +        else if (ACPI_COMPARE_NAMESEG (METHOD_NAME__CID, Op->Asl.NameSeg))
>          {
>              Next = Op->Asl.Child->Asl.Next;
>  

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