Hi,
> -----Original Message-----
> From: Tom Rini <tr...@konsulko.com>
> Sent: Tuesday, October 31, 2023 10:01 PM
> To: Jaehoon Chung <jh80.ch...@samsung.com>
> Cc: u-boot@lists.denx.de; ycli...@andestech.com;
> yanhong.w...@starfivetech.com;
> minda.c...@starfivetech.com; xingyu...@starfivetech.com
> Subject: Re: [PATCH 1/2] riscv: dts: jh7110: Add a gpio-restart node
>
> 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.ch...@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.
I understood what you said. Its node was applied with first dts files.
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/arch/riscv/boot/dts/starfive/jh7110-starfive-vision
five-2.dtsi?id=54baba33392d428a8be4942441a92a9b05cf537e
Then I will wait for re-syncing dt files about visionfive2.
Best Regards,
Jaehoon Chung
>
> --
> Tom
