public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* Re: [edk2-devel] [PATCH 0/5] Platforms cleanup v1
  2023-04-02 15:15 Chai, Evan
@ 2023-04-06  5:13 ` Sunil V L
  0 siblings, 0 replies; 5+ messages in thread
From: Sunil V L @ 2023-04-06  5:13 UTC (permalink / raw)
  To: devel, evan.chai

Hi Evan,

On Sun, Apr 02, 2023 at 11:15:37PM +0800, Chai, Evan wrote:
> *** BLURB HERE ***
>
This should be replaced with some valid cover letter for the series.
Also, please add [edk2-platforms] in the subject for the patch set.

Thanks,
Sunil

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH 0/5] Platforms cleanup v1
@ 2023-04-06  5:15 Chai, Evan
  2023-04-06  5:32 ` [edk2-devel] " Sunil V L
  0 siblings, 1 reply; 5+ messages in thread
From: Chai, Evan @ 2023-04-06  5:15 UTC (permalink / raw)
  To: devel

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


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [edk2-devel] [PATCH 0/5] Platforms cleanup v1
  2023-04-06  5:15 [PATCH 0/5] Platforms cleanup v1 Chai, Evan
@ 2023-04-06  5:32 ` Sunil V L
  2023-04-13  8:38   ` Chai, Evan
  0 siblings, 1 reply; 5+ messages in thread
From: Sunil V L @ 2023-04-06  5:32 UTC (permalink / raw)
  To: devel, evan.chai

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>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [edk2-devel] [PATCH 0/5] Platforms cleanup v1
  2023-04-06  5:32 ` [edk2-devel] " Sunil V L
@ 2023-04-13  8:38   ` Chai, Evan
  2023-04-17 10:35     ` Sunil V L
  0 siblings, 1 reply; 5+ messages in thread
From: Chai, Evan @ 2023-04-13  8:38 UTC (permalink / raw)
  To: devel@edk2.groups.io, sunilvl@ventanamicro.com

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>






^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [edk2-devel] [PATCH 0/5] Platforms cleanup v1
  2023-04-13  8:38   ` Chai, Evan
@ 2023-04-17 10:35     ` Sunil V L
  0 siblings, 0 replies; 5+ messages in thread
From: Sunil V L @ 2023-04-17 10:35 UTC (permalink / raw)
  To: devel, evan.chai

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

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2023-04-17 10:35 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-04-06  5:15 [PATCH 0/5] Platforms cleanup v1 Chai, Evan
2023-04-06  5:32 ` [edk2-devel] " Sunil V L
2023-04-13  8:38   ` Chai, Evan
2023-04-17 10:35     ` Sunil V L
  -- strict thread matches above, loose matches on Subject: below --
2023-04-02 15:15 Chai, Evan
2023-04-06  5:13 ` [edk2-devel] " Sunil V L

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox