[edk2] [PATCH edk2-platforms] Silicon/AMD/Styx/AcpiTables: fix wrongly copy/pasted variable name

Ard Biesheuvel posted 1 patch 5 years, 11 months ago
Failed in applying to current master (apply log)
Silicon/AMD/Styx/AcpiTables/Pptt.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
[edk2] [PATCH edk2-platforms] Silicon/AMD/Styx/AcpiTables: fix wrongly copy/pasted variable name
Posted by Ard Biesheuvel 5 years, 11 months ago
Rename the variable holding the Styx PPTT table to mStyxPpttTable.

Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
---
 Silicon/AMD/Styx/AcpiTables/Pptt.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Silicon/AMD/Styx/AcpiTables/Pptt.c b/Silicon/AMD/Styx/AcpiTables/Pptt.c
index d9d7c494d86f..1746bbe8b596 100644
--- a/Silicon/AMD/Styx/AcpiTables/Pptt.c
+++ b/Silicon/AMD/Styx/AcpiTables/Pptt.c
@@ -156,7 +156,7 @@ typedef struct {
   }                                                                           \
 }
 
-STATIC STYX_PPTT_TABLE mSynQuacerPpttTable = {
+STATIC STYX_PPTT_TABLE mStyxPpttTable = {
   {
     AMD_ACPI_HEADER(EFI_ACPI_6_2_PROCESSOR_PROPERTIES_TOPOLOGY_TABLE_STRUCTURE_SIGNATURE,
                     STYX_PPTT_TABLE,
@@ -221,5 +221,5 @@ PpttHeader (
   VOID
   )
 {
-  return (EFI_ACPI_DESCRIPTION_HEADER *)&mSynQuacerPpttTable.Pptt.Header;
+  return (EFI_ACPI_DESCRIPTION_HEADER *)&mStyxPpttTable.Pptt.Header;
 }
-- 
2.17.0

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Re: [edk2] [PATCH edk2-platforms] Silicon/AMD/Styx/AcpiTables: fix wrongly copy/pasted variable name
Posted by Leif Lindholm 5 years, 11 months ago
On Tue, May 01, 2018 at 06:32:44PM +0200, Ard Biesheuvel wrote:
> Rename the variable holding the Styx PPTT table to mStyxPpttTable.
> 
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>

Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>

> ---
>  Silicon/AMD/Styx/AcpiTables/Pptt.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/Silicon/AMD/Styx/AcpiTables/Pptt.c b/Silicon/AMD/Styx/AcpiTables/Pptt.c
> index d9d7c494d86f..1746bbe8b596 100644
> --- a/Silicon/AMD/Styx/AcpiTables/Pptt.c
> +++ b/Silicon/AMD/Styx/AcpiTables/Pptt.c
> @@ -156,7 +156,7 @@ typedef struct {
>    }                                                                           \
>  }
>  
> -STATIC STYX_PPTT_TABLE mSynQuacerPpttTable = {
> +STATIC STYX_PPTT_TABLE mStyxPpttTable = {
>    {
>      AMD_ACPI_HEADER(EFI_ACPI_6_2_PROCESSOR_PROPERTIES_TOPOLOGY_TABLE_STRUCTURE_SIGNATURE,
>                      STYX_PPTT_TABLE,
> @@ -221,5 +221,5 @@ PpttHeader (
>    VOID
>    )
>  {
> -  return (EFI_ACPI_DESCRIPTION_HEADER *)&mSynQuacerPpttTable.Pptt.Header;
> +  return (EFI_ACPI_DESCRIPTION_HEADER *)&mStyxPpttTable.Pptt.Header;
>  }
> -- 
> 2.17.0
> 
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Re: [edk2] [PATCH edk2-platforms] Silicon/AMD/Styx/AcpiTables: fix wrongly copy/pasted variable name
Posted by Ard Biesheuvel 5 years, 11 months ago
On 1 May 2018 at 19:55, Leif Lindholm <leif.lindholm@linaro.org> wrote:
> On Tue, May 01, 2018 at 06:32:44PM +0200, Ard Biesheuvel wrote:
>> Rename the variable holding the Styx PPTT table to mStyxPpttTable.
>>
>> Contributed-under: TianoCore Contribution Agreement 1.1
>> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>
> Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
>

Thanks. Pushed as 0db6171fb5a63e8d58ee7cfdecf36f5c57ce9315

>> ---
>>  Silicon/AMD/Styx/AcpiTables/Pptt.c | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/Silicon/AMD/Styx/AcpiTables/Pptt.c b/Silicon/AMD/Styx/AcpiTables/Pptt.c
>> index d9d7c494d86f..1746bbe8b596 100644
>> --- a/Silicon/AMD/Styx/AcpiTables/Pptt.c
>> +++ b/Silicon/AMD/Styx/AcpiTables/Pptt.c
>> @@ -156,7 +156,7 @@ typedef struct {
>>    }                                                                           \
>>  }
>>
>> -STATIC STYX_PPTT_TABLE mSynQuacerPpttTable = {
>> +STATIC STYX_PPTT_TABLE mStyxPpttTable = {
>>    {
>>      AMD_ACPI_HEADER(EFI_ACPI_6_2_PROCESSOR_PROPERTIES_TOPOLOGY_TABLE_STRUCTURE_SIGNATURE,
>>                      STYX_PPTT_TABLE,
>> @@ -221,5 +221,5 @@ PpttHeader (
>>    VOID
>>    )
>>  {
>> -  return (EFI_ACPI_DESCRIPTION_HEADER *)&mSynQuacerPpttTable.Pptt.Header;
>> +  return (EFI_ACPI_DESCRIPTION_HEADER *)&mStyxPpttTable.Pptt.Header;
>>  }
>> --
>> 2.17.0
>>
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel