[edk2-devel] [PATCH v2 03/16] DynamicTablesPkg: Remove struct CM_ARM_CPU_INFO

Sami Mujawar posted 16 patches 5 years, 10 months ago
There is a newer version of this series
[edk2-devel] [PATCH v2 03/16] DynamicTablesPkg: Remove struct CM_ARM_CPU_INFO
Posted by Sami Mujawar 5 years, 10 months ago
The VS2017 compiler reports 'error C2016: C requires that
a struct or union has at least one member' for the struct
CM_ARM_CPU_INFO.

Remove struct CM_ARM_CPU_INFO as this is not in use.

Signed-off-by: Sami Mujawar <sami.mujawar@arm.com>
Reviewed-by: Alexei Fedorov <Alexei.Fedorov@arm.com>
---

Notes:
    v2:
      Splitting patch series and re-submitting DynamicTablesPkg     [SAMI]
      patches from https://edk2.groups.io/g/devel/message/46261

 DynamicTablesPkg/Include/ArmNameSpaceObjects.h | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/DynamicTablesPkg/Include/ArmNameSpaceObjects.h b/DynamicTablesPkg/Include/ArmNameSpaceObjects.h
index da70cba2037592f02c72c026dc32f90b67bec8db..2874938b663c33edf5bdaac9f41495f75abfc945 100644
--- a/DynamicTablesPkg/Include/ArmNameSpaceObjects.h
+++ b/DynamicTablesPkg/Include/ArmNameSpaceObjects.h
@@ -71,10 +71,6 @@ typedef struct CmArmBootArchInfo {
   UINT32  BootArchFlags;
 } CM_ARM_BOOT_ARCH_INFO;
 
-typedef struct CmArmCpuInfo {
-  // Reserved for use when SMBIOS tables are implemented
-} CM_ARM_CPU_INFO;
-
 /** A structure that describes the
     Power Management Profile Information for the Platform.
 
-- 
'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)'


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#56564): https://edk2.groups.io/g/devel/message/56564
Mute This Topic: https://groups.io/mt/72631537/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH v2 03/16] DynamicTablesPkg: Remove struct CM_ARM_CPU_INFO
Posted by Ard Biesheuvel 5 years, 10 months ago
On Sun, 29 Mar 2020 at 17:14, Sami Mujawar <sami.mujawar@arm.com> wrote:
>
> The VS2017 compiler reports 'error C2016: C requires that
> a struct or union has at least one member' for the struct
> CM_ARM_CPU_INFO.
>
> Remove struct CM_ARM_CPU_INFO as this is not in use.
>
> Signed-off-by: Sami Mujawar <sami.mujawar@arm.com>
> Reviewed-by: Alexei Fedorov <Alexei.Fedorov@arm.com>

Reviewed-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>

> ---
>
> Notes:
>     v2:
>       Splitting patch series and re-submitting DynamicTablesPkg     [SAMI]
>       patches from https://edk2.groups.io/g/devel/message/46261
>
>  DynamicTablesPkg/Include/ArmNameSpaceObjects.h | 4 ----
>  1 file changed, 4 deletions(-)
>
> diff --git a/DynamicTablesPkg/Include/ArmNameSpaceObjects.h b/DynamicTablesPkg/Include/ArmNameSpaceObjects.h
> index da70cba2037592f02c72c026dc32f90b67bec8db..2874938b663c33edf5bdaac9f41495f75abfc945 100644
> --- a/DynamicTablesPkg/Include/ArmNameSpaceObjects.h
> +++ b/DynamicTablesPkg/Include/ArmNameSpaceObjects.h
> @@ -71,10 +71,6 @@ typedef struct CmArmBootArchInfo {
>    UINT32  BootArchFlags;
>  } CM_ARM_BOOT_ARCH_INFO;
>
> -typedef struct CmArmCpuInfo {
> -  // Reserved for use when SMBIOS tables are implemented
> -} CM_ARM_CPU_INFO;
> -
>  /** A structure that describes the
>      Power Management Profile Information for the Platform.
>
> --
> 'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)'
>
>
> 
>

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#56609): https://edk2.groups.io/g/devel/message/56609
Mute This Topic: https://groups.io/mt/72631537/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-