Re: [U-Boot] Pull request: u-boot-riscv/master

From: Tom Rini [mailto:trini@konsulko.com] Sent: Thursday, July 02, 2020 9:53 PM To: Open Source Project uboot Cc: u-boot@lists.denx.de; Rick Jian-Zhi Chen(陳建志) Subject: Re: [U-Boot] Pull request: u-boot-riscv/master
On Thu, Jul 02, 2020 at 10:51:48AM +0800, uboot@andestech.com wrote:
Hi Tom,
Please pull some riscv updates:
- sbi: Add newline to error message
- fu540: dts: Correct reg size of otp and dmc nodes
- Enhance reserved memory fixup about PMP information passed from
OpenSbi
- sifive: fu540: Add gpio-restart support
- qemu: Add syscon reboot and poweroff support
- qemu-riscv: Update QEMU run command
- Assorted fixes related to reserved memory
- fu540: enable all cache ways from U-Boot proper
- use log functions in fdt_fixup
This changes a few MIPS and ARM platforms. We're a few days away from release, so I don't feel comfortable taking this right now as I assume it hasn't been tested on the other platforms as well. Sorry.
Hi Tom,
Thanks for your suggestions.
Hi Bin,
Can you separate this series about sysreset into two series ? [PATCH 1/5] sysreset: syscon: Don't assume default value for offset and mask property [PATCH 2/5] sysreset: syscon: Support value property [PATCH 3/5] riscv: Do not build reset.c if SYSRESET is on [PATCH 4/5] riscv: qemu: Add syscon reboot and poweroff support [PATCH 5/5] riscv: sifive: fu540: Add gpio-restart support
It looks like that patch 3/5, 4/5, 5/5 doesn't depend on patch 1/5 and 2/5.
Thanks, Rick
--

Hi Rick,
On Fri, Jul 3, 2020 at 2:01 PM Rick Chen rickchen36@gmail.com wrote:
From: Tom Rini [mailto:trini@konsulko.com] Sent: Thursday, July 02, 2020 9:53 PM To: Open Source Project uboot Cc: u-boot@lists.denx.de; Rick Jian-Zhi Chen(陳建志) Subject: Re: [U-Boot] Pull request: u-boot-riscv/master
On Thu, Jul 02, 2020 at 10:51:48AM +0800, uboot@andestech.com wrote:
Hi Tom,
Please pull some riscv updates:
- sbi: Add newline to error message
- fu540: dts: Correct reg size of otp and dmc nodes
- Enhance reserved memory fixup about PMP information passed from
OpenSbi
- sifive: fu540: Add gpio-restart support
- qemu: Add syscon reboot and poweroff support
- qemu-riscv: Update QEMU run command
- Assorted fixes related to reserved memory
- fu540: enable all cache ways from U-Boot proper
- use log functions in fdt_fixup
This changes a few MIPS and ARM platforms. We're a few days away from release, so I don't feel comfortable taking this right now as I assume it hasn't been tested on the other platforms as well. Sorry.
Hi Tom,
Thanks for your suggestions.
Hi Bin,
Can you separate this series about sysreset into two series ? [PATCH 1/5] sysreset: syscon: Don't assume default value for offset and mask property [PATCH 2/5] sysreset: syscon: Support value property [PATCH 3/5] riscv: Do not build reset.c if SYSRESET is on [PATCH 4/5] riscv: qemu: Add syscon reboot and poweroff support [PATCH 5/5] riscv: sifive: fu540: Add gpio-restart support
It looks like that patch 3/5, 4/5, 5/5 doesn't depend on patch 1/5 and 2/5.
Patch 3 and 5 can be targeted for this release, and let's do patch 1,2,4 for the next release.
So you can drop patch 1,2,4 in your queue and resend the PR. Thanks!
Regards, Bin
participants (2)
-
Bin Meng
-
Rick Chen