Enable CONFIG_SYSREST config to do reset.

Signed-off-by: Jaehoon Chung jh80.chung@samsung.com --- configs/starfive_visionfive2_defconfig | 1 + 1 file changed, 1 insertion(+)

diff --git a/configs/starfive_visionfive2_defconfig b/configs/starfive_visionfive2_defconfig index b21754feafce..afa74e17ebd5 100644 --- a/configs/starfive_visionfive2_defconfig +++ b/configs/starfive_visionfive2_defconfig @@ -122,6 +122,7 @@ CONFIG_PINCTRL_STARFIVE=y # CONFIG_RAM_SIFIVE is not set CONFIG_SYS_NS16550=y CONFIG_CADENCE_QSPI=y +CONFIG_SYSRESET=y CONFIG_TIMER_EARLY=y CONFIG_USB=y CONFIG_USB_XHCI_HCD=y

On Tue, Oct 31, 2023 at 05:24:38PM +0900, Jaehoon Chung wrote:

Add gpio-restart node to do reset.

Before applied this patch, System Reset Extension doesn't appear with sbi command.

OpenSBI 1.3 Machine: Vendor ID 489 Architecture ID 8000000000000007 Implementation ID 4210427 Extensions: sbi_set_timer sbi_console_putchar ...[snip]... IPI Extension RFENCE Extension Hart State Management Extension Performance Monitoring Unit Extension

After applied this patch, System Reset Extension is supported from SBI.

OpenSBI 1.3 Machine: Vendor ID 489 Architecture ID 8000000000000007 Implementation ID 4210427 Extensions: sbi_set_timer sbi_console_putchar ...[snip]... IPI Extension RFENCE Extension Hart State Management Extension System Reset Extension Performance Monitoring Unit Extension

Signed-off-by: Jaehoon Chung jh80.chung@samsung.com

arch/riscv/dts/jh7110-starfive-visionfive-2.dtsi | 5 +++++ 1 file changed, 5 insertions(+)

diff --git a/arch/riscv/dts/jh7110-starfive-visionfive-2.dtsi b/arch/riscv/dts/jh7110-starfive-visionfive-2.dtsi index e40f57a15080..e94f9fe826a8 100644 --- a/arch/riscv/dts/jh7110-starfive-visionfive-2.dtsi +++ b/arch/riscv/dts/jh7110-starfive-visionfive-2.dtsi @@ -34,6 +34,11 @@ device_type = "memory"; reg = <0x0 0x40000000 0x2 0x0>; };

• gpio-restart {

• compatible = "gpio-restart";
  

• gpios = <&sysgpio 35 GPIO_ACTIVE_HIGH>;
  

• };

};

&osc {

My only concern with this change (and a few other similar things I've seen of late) is where is this change with respect to the upstream kernel? The goal is the dts files should be able to be dropped in from the kernel (except for -u-boot.dtsi), so we don't want to have changed get dropped when they're re-synced.