Re: [PATCH v4 2/7] MdePkg/Register: RISC-V: Add satp mode bits shift definition

["Michael D Kinney" <michael.d.kinney@intel.com>]

Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>

> -----Original Message-----
> From: Tuan Phan <tphan@ventanamicro.com>
> Sent: Friday, June 23, 2023 11:39 AM
> To: devel@edk2.groups.io
> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming
> <gaoliming@byosoft.com.cn>; Liu, Zhiguang <zhiguang.liu@intel.com>;
> sunilvl@ventanamicro.com; git@danielschaefer.me; Warkentin, Andrei
> <andrei.warkentin@intel.com>; ardb+tianocore@kernel.org; Tuan Phan
> <tphan@ventanamicro.com>
> Subject: [PATCH v4 2/7] MdePkg/Register: RISC-V: Add satp mode bits shift
> definition
> 
> The satp mode bits shift is used cross modules. It should be defined
> in one place.
> 
> Signed-off-by: Tuan Phan <tphan@ventanamicro.com>
> Reviewed-by: Andrei Warkentin <andrei.warkentin@intel.com>
> Reviewed-by: Sunil V L <sunilvl@ventanamicro.com>
> ---
>  MdePkg/Include/Register/RiscV64/RiscVEncoding.h | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/MdePkg/Include/Register/RiscV64/RiscVEncoding.h
> b/MdePkg/Include/Register/RiscV64/RiscVEncoding.h
> index 5c2989b797bf..2bde8db478ff 100644
> --- a/MdePkg/Include/Register/RiscV64/RiscVEncoding.h
> +++ b/MdePkg/Include/Register/RiscV64/RiscVEncoding.h
> @@ -58,9 +58,10 @@
>  #define PRV_S  1UL
> 
>  #define PRV_M  3UL
> 
> 
> 
> -#define SATP64_MODE  0xF000000000000000ULL
> 
> -#define SATP64_ASID  0x0FFFF00000000000ULL
> 
> -#define SATP64_PPN   0x00000FFFFFFFFFFFULL
> 
> +#define SATP64_MODE        0xF000000000000000ULL
> 
> +#define SATP64_MODE_SHIFT  60
> 
> +#define SATP64_ASID        0x0FFFF00000000000ULL
> 
> +#define SATP64_PPN         0x00000FFFFFFFFFFFULL
> 
> 
> 
>  #define SATP_MODE_OFF   0UL
> 
>  #define SATP_MODE_SV32  1UL
> 
> --
> 2.25.1