Re: [PATCH 1/1] cmd/riscv/sbi: support System Reset Extension

[Bin Meng]

On Mon, Jan 18, 2021 at 5:27 AM Heinrich Schuchardt  wrote:
>
> Let the sbi command detect the 'System Reset Extension'
> (EID #0x53525354 "SRST").
>
> Cf. https://github.com/riscv/riscv-sbi-doc
>
> Signed-off-by: Heinrich Schuchardt 
> ---
>  cmd/riscv/sbi.c | 1 +
>  1 file changed, 1 insertion(+)
>

Reviewed-by: Bin Meng 

We should also update the reset command to call into SBI when this
extension is available.

Regards,
Bin

Re: [PATCH 1/1] cmd/riscv/sbi: support System Reset Extension

[Leo Liang]

On Sun, Jan 17, 2021 at 09:27:13PM +, Heinrich Schuchardt wrote:
> Let the sbi command detect the 'System Reset Extension'
> (EID #0x53525354 "SRST").
> 
> Cf. https://github.com/riscv/riscv-sbi-doc
> 
> Signed-off-by: Heinrich Schuchardt 
> ---
>  cmd/riscv/sbi.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/cmd/riscv/sbi.c b/cmd/riscv/sbi.c
> index e66fc8e41d..9897483eb6 100644
> --- a/cmd/riscv/sbi.c
> +++ b/cmd/riscv/sbi.c
> @@ -29,6 +29,7 @@ static struct sbi_ext extensions[] = {
>   { 0x00735049, "IPI Extension" },
>   { 0x52464E43, "RFENCE Extension" },
>   { 0x0048534D, "Hart State Management Extension" },
> + { 0x53525354, "System Reset Extension" },
>  };
> 
>  static int do_sbi(struct cmd_tbl *cmdtp, int flag, int argc,
> --
> 2.28.0
>

Reviewed-by: Leo Liang 

RE: [PATCH 1/1] cmd/riscv/sbi: support System Reset Extension

[Pragnesh Patel]

>-Original Message-
>From: Heinrich Schuchardt 
>Sent: 18 January 2021 02:57
>To: Rick Chen 
>Cc: Atish Patra ; Bin Meng ;
>Pragnesh Patel ; u-boot@lists.denx.de; Heinrich
>Schuchardt 
>Subject: [PATCH 1/1] cmd/riscv/sbi: support System Reset Extension
>
>[External Email] Do not click links or attachments unless you recognize the
>sender and know the content is safe
>
>Let the sbi command detect the 'System Reset Extension'
>(EID #0x53525354 "SRST").
>
>Cf. https://github.com/riscv/riscv-sbi-doc
>
>Signed-off-by: Heinrich Schuchardt 
>---
> cmd/riscv/sbi.c | 1 +
> 1 file changed, 1 insertion(+)

Reviewed-by: Pragnesh Patel