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 <[email protected]> wrote:
> >
> > > From: Simon Glass <[email protected]>
> > > Sent: Monday, April 03, 2023 4:28 AM
> > > To: U-Boot Mailing List <[email protected]>
> > > Cc: Sean Anderson <[email protected]>; Bin Meng <[email protected]>; 
> > > Rick Jian-Zhi Chen(陳建志) <[email protected]>; Leo Yu-Chi Liang(梁育齊) 
> > > <[email protected]>; Simon Glass <[email protected]>; Andre Przywara 
> > > <[email protected]>; Marc Kleine-Budde <[email protected]>; 
> > > SESA644425 <[email protected]>; Samuel Holland 
> > > <[email protected]>; Steven Lawrance <[email protected]>
> > > 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 <[email protected]>
> > > ---
> > >
> > >  boot/image.c    | 3 ++-
> > >  include/image.h | 3 ++-
> > >  2 files changed, 4 insertions(+), 2 deletions(-)
> >
> > Reviewed-by: Rick Chen <[email protected]>
> 
> 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

Reply via email to