Hi Bin, On Thu, Apr 13, 2023 at 06:06:29PM +0800, Bin Meng wrote: > On Mon, Apr 10, 2023 at 3:25 PM Rick Chen <rickche...@gmail.com> wrote: > > > > > From: Simon Glass <s...@chromium.org> > > > Sent: Monday, April 03, 2023 4:28 AM > > > To: U-Boot Mailing List <u-boot@lists.denx.de> > > > Cc: Sean Anderson <sean...@gmail.com>; Bin Meng <bmeng...@gmail.com>; > > > Rick Jian-Zhi Chen(陳建志) <r...@andestech.com>; Leo Yu-Chi Liang(梁育齊) > > > <ycli...@andestech.com>; Simon Glass <s...@chromium.org>; Andre Przywara > > > <andre.przyw...@arm.com>; Marc Kleine-Budde <m...@pengutronix.de>; > > > SESA644425 <giojaherm...@gmail.com>; Samuel Holland > > > <sam...@sholland.org>; Steven Lawrance <steven.lawra...@softathome.com> > > > Subject: [PATCH] riscv: Add a 64-bit image type > > > > > > At present it is not possible to know whether an image can be booted by a > > > 32- or 64-bit bootloader. This means that U-Boot may attempt to boot the > > > wrong image. This may cause a crash which might be hard to debug. > > > > > > Add a new property to make this explicit. > > > > > > The existing 'RISC-V' is now taken to mean 32-bit. > > > > > > Signed-off-by: Simon Glass <s...@chromium.org> > > > --- > > > > > > boot/image.c | 3 ++- > > > include/image.h | 3 ++- > > > 2 files changed, 4 insertions(+), 2 deletions(-) > > > > Reviewed-by: Rick Chen <r...@andestech.com> > > This might create compatibility issues if the shipped host tool > (mkimage) does not match U-Boot version. Any idea how to avoid that? >
I am not sure if I understand you correctly. Do you mean that there could be compatibility issue if one uses 2023.04 release host tool (mkimage) to build images that are to be booted by 2023.07 release u-boot (if this patch is merged in 2023.07 u-boot)? If that is the case, I guess there is no easy way to avoid that. We could probably have RISCV represent 64 bit riscv architecture and RISCV32 32 bit because most of the boards are 64 bit. If that is not the case, could you elaborate more ? And Rick has sent out a patch to fix booting issue based on Simon's patch as well. Best regards, Leo > Regards, > Bin
