Re: [edk2-devel] [PATCH] RISC-V/PlatformPkg: Fix compilation breakage in OpenSBI

["Abner Chang" <abner.chang@hpe.com>]

Hi Loic,
The current edk2 RISC-V is incorporated with opensbi v0.8, please check below link
https://github.com/riscv/riscv-uefi-edk2-docs

We have to update below library with the latest opensbi with your changes,
https://github.com/tianocore/edk2-platforms/tree/master/Silicon/RISC-V/ProcessorPkg/Library/RiscVOpensbiLib 

I believe your changes don't impact the build and the functionalities because we don't use those parameters in edk2. However, this change should be adopted with Edk2OpenSbiLib upgrade.
Daniel is currently working on edk2 RISC-V OVMF and also updated opensbi to the latest version. However there is a compatible issue when switching mode and we are fixing it now.
We can share more information with you if you would like to know.

Thanks
Abner
> -----Original Message-----
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
> Loic Devulder via groups.io
> Sent: Wednesday, March 24, 2021 9:59 PM
> To: devel@edk2.groups.io
> Subject: [edk2-devel] [PATCH] RISC-V/PlatformPkg: Fix compilation breakage
> in OpenSBI
> 
> platform_ops and platform structures have older entries that don't exist
> anymore on recent OpenSBI versions, so we can remove them.
> 
> Cc: Abner Chang <abner.chang@hpe.com>
> Cc: Daniel Schaefer <daniel.schaefer@hpe.com>
> Cc: Gilbert Chen <gilbert.chen@hpe.com>
> 
> Signed-off-by: Loic Devulder <ldevulder@suse.com>
> ---
>  .../PlatformPkg/Library/OpensbiPlatformLibNull/Platform.c     | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/Platform/RISC-
> V/PlatformPkg/Library/OpensbiPlatformLibNull/Platform.c b/Platform/RISC-
> V/PlatformPkg/Library/OpensbiPlatformLibNull/Platform.c
> index e78d811f4c..84b5f5dec8 100644
> --- a/Platform/RISC-
> V/PlatformPkg/Library/OpensbiPlatformLibNull/Platform.c
> +++ b/Platform/RISC-V/PlatformPkg/Library/OpensbiPlatformLibNull/Platfor
> +++ m.c
> @@ -28,8 +28,7 @@ const struct sbi_platform_operations platform_ops = {
>      .timer_event_stop   = NULL,
>      .timer_event_start  = NULL,
>      .timer_init         = NULL,
> -    .system_reboot      = NULL,
> -    .system_shutdown    = NULL
> +    .system_reset       = NULL
>  };
> 
>  const struct sbi_platform platform = {
> @@ -39,6 +38,5 @@ const struct sbi_platform platform = {
>      .features           = 0,
>      .hart_count         = 0,
>      .hart_stack_size    = 0,
> -    .disabled_hart_mask = 0,
>      .platform_ops_addr  = 0
>  };
> --
> 2.30.1
> 
> 
> 
> 
>