Re: [edk2-devel] [edk2-platforms PATCH v2 0/8] Silicon/Marvell/OdysseyPkg:

["Marcin Wojtas via groups.io" <mw=semihalf.com@groups.io>]

+marcin.s.wojtas@gmail.com

Hi Narinder,

czw., 21 gru 2023 o 01:54 Narinder Dhillon <ndhillon@marvell.com> napisał(a):
>
> From: Narinder Dhillon <ndhillon@marvell.com>
>
> New Marvell Odyssey SoC
>
> This patchset contains only the very basic elements needed to boot to
> EDK2 UiApp on Marvell Odyssey SoC
> - ARM BL31 firmware component copies EDK2 image into memory, so it is
>   always executing from memory
> - There is a SMC library to get system information from BL31
> - There are drivers to get board configuration details from a device
>   tree
> - There is no on-chip RTC, a dummy driver is used
> - Emulated variable storage is used for now
>
> v2:
> Split patch into 8 commits
>
> v1:
> Original patch in single commit
>
> Narinder Dhillon (8):
>   Silicon/Marvell: New Marvell Odyssey processor
>   Silicon/Marvell: Odyssey ArmPlatformLib
>   Silicon/Marvell: Odyssey SmcLib
>   Silicon/Marvell: Odyssey watchdog driver
>   Silicon/Marvell: RTC driver
>   Silicon/Marvell: Device tree driver
>   Silicon/Marvell: Driver to dump board configuration
>   Silicon/Marvell: Odyssey project description files
>

Thanks for splitting the change to multiple commits, I'm done with the
first round of review and looking forward to the next iteration.

Best regards,
Marcin

>  Platform/Marvell/OdysseyPkg/OdysseyPkg.dsc    | 216 ++++++++++
>  Platform/Marvell/OdysseyPkg/OdysseyPkg.fdf    | 304 +++++++++++++
>  .../Drivers/Fdt/FdtClientDxe/FdtClientDxe.c   | 382 ++++++++++++++++
>  .../Drivers/Fdt/FdtClientDxe/FdtClientDxe.inf |  43 ++
>  .../Drivers/Fdt/FdtPlatformDxe/FdtPlatform.c  | 361 ++++++++++++++++
>  .../Drivers/Fdt/FdtPlatformDxe/FdtPlatform.h  | 102 +++++
>  .../Fdt/FdtPlatformDxe/FdtPlatformDxe.inf     |  60 +++
>  .../Fdt/FdtPlatformDxe/FdtPlatformDxe.uni     | 106 +++++
>  .../Drivers/Fdt/FdtPlatformDxe/README.txt     |  69 +++
>  .../Drivers/Fdt/FdtPlatformDxe/ShellDumpFdt.c | 283 ++++++++++++
>  .../Marvell/Drivers/Null/RtcNull/RtcNullDxe.c | 280 ++++++++++++
>  .../Marvell/Drivers/Null/RtcNull/RtcNullDxe.h |  37 ++
>  .../Drivers/Null/RtcNull/RtcNullDxe.inf       |  46 ++
>  .../Drivers/Wdt/GtiWatchdogDxe/GtiWatchdog.c  | 408 ++++++++++++++++++
>  .../Wdt/GtiWatchdogDxe/GtiWatchdogDxe.inf     |  45 ++
>  .../AArch64/ArmPlatformHelper.S               |  86 ++++
>  .../Library/ArmPlatformLib/ArmPlatformLib.c   |  79 ++++
>  .../Library/ArmPlatformLib/ArmPlatformLib.inf |  55 +++
>  .../ArmPlatformLib/ArmPlatformLibMem.c        | 131 ++++++
>  Silicon/Marvell/Library/SmcLib/SmcLib.c       |  24 ++
>  Silicon/Marvell/Library/SmcLib/SmcLib.inf     |  29 ++
>  .../Include/Library/SmcLib.h                  |  28 ++
>  .../Include/Protocol/FdtClient.h              | 180 ++++++++
>  .../MarvellSiliconPkg/MarvellSiliconPkg.dec   |  19 +
>  Silicon/Marvell/OdysseyPkg/OdysseyPkg.dsc.inc | 394 +++++++++++++++++
>  .../PrePi/AArch64/ModuleEntryPoint.S          | 136 ++++++
>  .../ArmPlatformPkg/PrePi/PeiMPCore.inf        | 110 +++++
>  .../Override/ArmPlatformPkg/PrePi/PrePi.c     | 238 ++++++++++
>  28 files changed, 4251 insertions(+)
>  create mode 100644 Platform/Marvell/OdysseyPkg/OdysseyPkg.dsc
>  create mode 100644 Platform/Marvell/OdysseyPkg/OdysseyPkg.fdf
>  create mode 100644 Silicon/Marvell/Drivers/Fdt/FdtClientDxe/FdtClientDxe.c
>  create mode 100644 Silicon/Marvell/Drivers/Fdt/FdtClientDxe/FdtClientDxe.inf
>  create mode 100644 Silicon/Marvell/Drivers/Fdt/FdtPlatformDxe/FdtPlatform.c
>  create mode 100644 Silicon/Marvell/Drivers/Fdt/FdtPlatformDxe/FdtPlatform.h
>  create mode 100644 Silicon/Marvell/Drivers/Fdt/FdtPlatformDxe/FdtPlatformDxe.inf
>  create mode 100644 Silicon/Marvell/Drivers/Fdt/FdtPlatformDxe/FdtPlatformDxe.uni
>  create mode 100644 Silicon/Marvell/Drivers/Fdt/FdtPlatformDxe/README.txt
>  create mode 100644 Silicon/Marvell/Drivers/Fdt/FdtPlatformDxe/ShellDumpFdt.c
>  create mode 100644 Silicon/Marvell/Drivers/Null/RtcNull/RtcNullDxe.c
>  create mode 100644 Silicon/Marvell/Drivers/Null/RtcNull/RtcNullDxe.h
>  create mode 100644 Silicon/Marvell/Drivers/Null/RtcNull/RtcNullDxe.inf
>  create mode 100644 Silicon/Marvell/Drivers/Wdt/GtiWatchdogDxe/GtiWatchdog.c
>  create mode 100644 Silicon/Marvell/Drivers/Wdt/GtiWatchdogDxe/GtiWatchdogDxe.inf
>  create mode 100644 Silicon/Marvell/Library/ArmPlatformLib/AArch64/ArmPlatformHelper.S
>  create mode 100644 Silicon/Marvell/Library/ArmPlatformLib/ArmPlatformLib.c
>  create mode 100644 Silicon/Marvell/Library/ArmPlatformLib/ArmPlatformLib.inf
>  create mode 100644 Silicon/Marvell/Library/ArmPlatformLib/ArmPlatformLibMem.c
>  create mode 100644 Silicon/Marvell/Library/SmcLib/SmcLib.c
>  create mode 100644 Silicon/Marvell/Library/SmcLib/SmcLib.inf
>  create mode 100644 Silicon/Marvell/MarvellSiliconPkg/Include/Library/SmcLib.h
>  create mode 100644 Silicon/Marvell/MarvellSiliconPkg/Include/Protocol/FdtClient.h
>  create mode 100644 Silicon/Marvell/OdysseyPkg/OdysseyPkg.dsc.inc
>  create mode 100644 Silicon/Marvell/Override/ArmPlatformPkg/PrePi/AArch64/ModuleEntryPoint.S
>  create mode 100644 Silicon/Marvell/Override/ArmPlatformPkg/PrePi/PeiMPCore.inf
>  create mode 100644 Silicon/Marvell/Override/ArmPlatformPkg/PrePi/PrePi.c
>
>
> base-commit: 59ef582c1bada1e25d0f1490e2af2d68b067fad2
> --
> 2.34.1
>
>
>
> 
>
>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#113739): https://edk2.groups.io/g/devel/message/113739
Mute This Topic: https://groups.io/mt/103292508/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-