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

Loic Devulder via groups.io posted 1 patch 3 years ago
Failed in applying to current master (apply log)
.../PlatformPkg/Library/OpensbiPlatformLibNull/Platform.c     | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
[edk2-devel] [PATCH] RISC-V/PlatformPkg: Fix compilation breakage in OpenSBI
Posted by Loic Devulder via groups.io 3 years ago
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/Platform.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



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


Re: [edk2-devel] [PATCH] RISC-V/PlatformPkg: Fix compilation breakage in OpenSBI
Posted by Abner Chang 3 years ago
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
> 
> 
> 
> 
> 



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


Re: [edk2-devel] [PATCH] RISC-V/PlatformPkg: Fix compilation breakage in OpenSBI
Posted by Loic Devulder via groups.io 3 years ago
On Thu, Mar 25, 2021 at 01:46 AM, Abner Chang wrote:

> 
> 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

Yes, I think this is the version I used. In fact I followed https://github.com/tianocore/edk2-platforms#submodules to add the needed submodule.

> 
> 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.

Sounds like yes. I tested in qemu without any issue, but that was not a full regression test ;-)

> 
> 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.

So if this fix will be included in Daniel's changes that fine. I just wanted to share the fix if anyone else had issue and also check if the issue was not on my side, I'm a newbie on edk2 and on RISC-V :D

> 
> 
> Thanks
> Abner


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