[edk2-devel] [PATCH 0/5] Platforms cleanup v1

Chai, Evan posted 5 patches 1 year ago
Only 0 patches received!
Platform/RISC-V/PlatformPkg/Library/ResetSystemLib/ResetSystemLib.c                          |   4 +++-
Platform/RISC-V/PlatformPkg/Library/ResetSystemLib/ResetSystemLib.inf                        |   4 +++-
Platform/RISC-V/PlatformPkg/Universal/Sec/SecMain.inf                                        |   3 ++-
Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.dsc                                   |  19 ++++++++++---------
Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.fdf                                   |   5 +++--
Platform/SiFive/U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.dsc                         |  19 ++++++++++---------
Platform/SiFive/U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.fdf                         |   5 +++--
Platform/SiFive/U5SeriesPkg/Universal/Dxe/RamFvbServicesRuntimeDxe/FvbServicesRuntimeDxe.inf |   2 ++
Platform/SiFive/U5SeriesPkg/Universal/Dxe/TimerDxe/Timer.c                                   | 311 -----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
Platform/SiFive/U5SeriesPkg/Universal/Dxe/TimerDxe/Timer.h                                   | 174 ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
Platform/SiFive/U5SeriesPkg/Universal/Dxe/TimerDxe/Timer.uni                                 |  14 --------------
Platform/SiFive/U5SeriesPkg/Universal/Dxe/TimerDxe/TimerDxe.inf                              |  54 ------------------------------------------------------
Platform/SiFive/U5SeriesPkg/Universal/Dxe/TimerDxe/TimerExtra.uni                            |  12 ------------
Silicon/RISC-V/ProcessorPkg/Include/Library/RiscVCpuLib.h                                    |  20 +-------------------
Silicon/RISC-V/ProcessorPkg/Library/RiscVCpuLib/Cpu.S                                        |  41 +----------------------------------------
Silicon/RISC-V/ProcessorPkg/Library/RiscVTimerLib/BaseRiscVTimerLib.inf                      |  35 -----------------------------------
Silicon/RISC-V/ProcessorPkg/Library/RiscVTimerLib/RiscVTimerLib.c                            | 199 -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
Silicon/RISC-V/ProcessorPkg/RiscVProcessorPkg.dsc                                            |   6 +-----
Silicon/RISC-V/ProcessorPkg/Universal/CpuDxe/CpuDxe.c                                        | 310 ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
Silicon/RISC-V/ProcessorPkg/Universal/CpuDxe/CpuDxe.h                                        | 198 ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
Silicon/RISC-V/ProcessorPkg/Universal/CpuDxe/CpuDxe.inf                                      |  49 -------------------------------------------------
Silicon/RISC-V/ProcessorPkg/Universal/CpuDxe/CpuDxe.uni                                      |  13 -------------
Silicon/RISC-V/ProcessorPkg/Universal/CpuDxe/CpuDxeExtra.uni                                 |  14 --------------
Silicon/SiFive/U54/Library/PeiCoreInfoHobLib/PeiCoreInfoHobLib.inf                           |   3 ++-
24 files changed, 41 insertions(+), 1473 deletions(-)
delete mode 100644 Platform/SiFive/U5SeriesPkg/Universal/Dxe/TimerDxe/Timer.c
delete mode 100644 Platform/SiFive/U5SeriesPkg/Universal/Dxe/TimerDxe/Timer.h
delete mode 100644 Platform/SiFive/U5SeriesPkg/Universal/Dxe/TimerDxe/Timer.uni
delete mode 100644 Platform/SiFive/U5SeriesPkg/Universal/Dxe/TimerDxe/TimerDxe.inf
delete mode 100644 Platform/SiFive/U5SeriesPkg/Universal/Dxe/TimerDxe/TimerExtra.uni
delete mode 100644 Silicon/RISC-V/ProcessorPkg/Library/RiscVTimerLib/BaseRiscVTimerLib.inf
delete mode 100644 Silicon/RISC-V/ProcessorPkg/Library/RiscVTimerLib/RiscVTimerLib.c
delete mode 100644 Silicon/RISC-V/ProcessorPkg/Universal/CpuDxe/CpuDxe.c
delete mode 100644 Silicon/RISC-V/ProcessorPkg/Universal/CpuDxe/CpuDxe.h
delete mode 100644 Silicon/RISC-V/ProcessorPkg/Universal/CpuDxe/CpuDxe.inf
delete mode 100644 Silicon/RISC-V/ProcessorPkg/Universal/CpuDxe/CpuDxe.uni
delete mode 100644 Silicon/RISC-V/ProcessorPkg/Universal/CpuDxe/CpuDxeExtra.uni
[edk2-devel] [PATCH 0/5] Platforms cleanup v1
Posted by Chai, Evan 1 year ago
Booting to uefi shell is verfied in Qemu mode, 
The qemu command be refered to '5a> Boot on QEMU' in
https://github.com/riscv-admin/riscv-uefi-edk2-docs

Evan Chai (5):
  Silicon/RISC-V/ProcessorPkg: remove redundant CpuDxe driver
  Platform/Sifive: remove redundant TimerDxe from Platform
  Silicon/RISC-V: remove redundant RiscVTimerLib
  Silicon/RISC-V: remove redundant function code from RiscVCpuLib
  Platform/ Siliocn/: Fix building failure caused by wrong lib.

 Platform/RISC-V/PlatformPkg/Library/ResetSystemLib/ResetSystemLib.c                          |   4 +++-
 Platform/RISC-V/PlatformPkg/Library/ResetSystemLib/ResetSystemLib.inf                        |   4 +++-
 Platform/RISC-V/PlatformPkg/Universal/Sec/SecMain.inf                                        |   3 ++-
 Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.dsc                                   |  19 ++++++++++---------
 Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.fdf                                   |   5 +++--
 Platform/SiFive/U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.dsc                         |  19 ++++++++++---------
 Platform/SiFive/U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.fdf                         |   5 +++--
 Platform/SiFive/U5SeriesPkg/Universal/Dxe/RamFvbServicesRuntimeDxe/FvbServicesRuntimeDxe.inf |   2 ++
 Platform/SiFive/U5SeriesPkg/Universal/Dxe/TimerDxe/Timer.c                                   | 311 -----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 Platform/SiFive/U5SeriesPkg/Universal/Dxe/TimerDxe/Timer.h                                   | 174 ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 Platform/SiFive/U5SeriesPkg/Universal/Dxe/TimerDxe/Timer.uni                                 |  14 --------------
 Platform/SiFive/U5SeriesPkg/Universal/Dxe/TimerDxe/TimerDxe.inf                              |  54 ------------------------------------------------------
 Platform/SiFive/U5SeriesPkg/Universal/Dxe/TimerDxe/TimerExtra.uni                            |  12 ------------
 Silicon/RISC-V/ProcessorPkg/Include/Library/RiscVCpuLib.h                                    |  20 +-------------------
 Silicon/RISC-V/ProcessorPkg/Library/RiscVCpuLib/Cpu.S                                        |  41 +----------------------------------------
 Silicon/RISC-V/ProcessorPkg/Library/RiscVTimerLib/BaseRiscVTimerLib.inf                      |  35 -----------------------------------
 Silicon/RISC-V/ProcessorPkg/Library/RiscVTimerLib/RiscVTimerLib.c                            | 199 -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 Silicon/RISC-V/ProcessorPkg/RiscVProcessorPkg.dsc                                            |   6 +-----
 Silicon/RISC-V/ProcessorPkg/Universal/CpuDxe/CpuDxe.c                                        | 310 ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 Silicon/RISC-V/ProcessorPkg/Universal/CpuDxe/CpuDxe.h                                        | 198 ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 Silicon/RISC-V/ProcessorPkg/Universal/CpuDxe/CpuDxe.inf                                      |  49 -------------------------------------------------
 Silicon/RISC-V/ProcessorPkg/Universal/CpuDxe/CpuDxe.uni                                      |  13 -------------
 Silicon/RISC-V/ProcessorPkg/Universal/CpuDxe/CpuDxeExtra.uni                                 |  14 --------------
 Silicon/SiFive/U54/Library/PeiCoreInfoHobLib/PeiCoreInfoHobLib.inf                           |   3 ++-
 24 files changed, 41 insertions(+), 1473 deletions(-)
 delete mode 100644 Platform/SiFive/U5SeriesPkg/Universal/Dxe/TimerDxe/Timer.c
 delete mode 100644 Platform/SiFive/U5SeriesPkg/Universal/Dxe/TimerDxe/Timer.h
 delete mode 100644 Platform/SiFive/U5SeriesPkg/Universal/Dxe/TimerDxe/Timer.uni
 delete mode 100644 Platform/SiFive/U5SeriesPkg/Universal/Dxe/TimerDxe/TimerDxe.inf
 delete mode 100644 Platform/SiFive/U5SeriesPkg/Universal/Dxe/TimerDxe/TimerExtra.uni
 delete mode 100644 Silicon/RISC-V/ProcessorPkg/Library/RiscVTimerLib/BaseRiscVTimerLib.inf
 delete mode 100644 Silicon/RISC-V/ProcessorPkg/Library/RiscVTimerLib/RiscVTimerLib.c
 delete mode 100644 Silicon/RISC-V/ProcessorPkg/Universal/CpuDxe/CpuDxe.c
 delete mode 100644 Silicon/RISC-V/ProcessorPkg/Universal/CpuDxe/CpuDxe.h
 delete mode 100644 Silicon/RISC-V/ProcessorPkg/Universal/CpuDxe/CpuDxe.inf
 delete mode 100644 Silicon/RISC-V/ProcessorPkg/Universal/CpuDxe/CpuDxe.uni
 delete mode 100644 Silicon/RISC-V/ProcessorPkg/Universal/CpuDxe/CpuDxeExtra.uni

-- 
2.34.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#102591): https://edk2.groups.io/g/devel/message/102591
Mute This Topic: https://groups.io/mt/98015394/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH 0/5] Platforms cleanup v1
Posted by Sunil V L 1 year ago
On Thu, Apr 06, 2023 at 01:15:21PM +0800, Chai, Evan wrote:
> Booting to uefi shell is verfied in Qemu mode, 
> The qemu command be refered to '5a> Boot on QEMU' in
> https://github.com/riscv-admin/riscv-uefi-edk2-docs
> 
Thanks! Evan for fixing this. The series LGTM.

Reviewed-by: Sunil V L <sunilvl@ventanamicro.com>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#102598): https://edk2.groups.io/g/devel/message/102598
Mute This Topic: https://groups.io/mt/98015394/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH 0/5] Platforms cleanup v1
Posted by Chai, Evan 1 year ago
Hi Sunil, 
   Sorry to remind, do we need one more reviewer for it? Or you can help to merge it now? 

BR,
Evan

-----Original Message-----
From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Sunil V L
Sent: Thursday, April 6, 2023 1:32 PM
To: devel@edk2.groups.io; Chai, Evan <evan.chai@intel.com>
Subject: Re: [edk2-devel] [PATCH 0/5] Platforms cleanup v1

On Thu, Apr 06, 2023 at 01:15:21PM +0800, Chai, Evan wrote:
> Booting to uefi shell is verfied in Qemu mode, The qemu command be 
> refered to '5a> Boot on QEMU' in 
> https://github.com/riscv-admin/riscv-uefi-edk2-docs
> 
Thanks! Evan for fixing this. The series LGTM.

Reviewed-by: Sunil V L <sunilvl@ventanamicro.com>







-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#102931): https://edk2.groups.io/g/devel/message/102931
Mute This Topic: https://groups.io/mt/98015394/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH 0/5] Platforms cleanup v1
Posted by Sunil V L 1 year ago
On Thu, Apr 13, 2023 at 08:38:24AM +0000, Chai, Evan wrote:
> Hi Sunil, 
>    Sorry to remind, do we need one more reviewer for it? Or you can help to merge it now? 
> 
Merged.

Thanks,
Sunil


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#103082): https://edk2.groups.io/g/devel/message/103082
Mute This Topic: https://groups.io/mt/98015394/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-