Re: [PATCH] cmd: Make bootvx independent of bootelf

2024-10-11 Thread Daniel Palmer
Hi Tom, Simon, On Wed, 9 Oct 2024 at 11:10, Tom Rini wrote: > > > Applied to u-boot/master, thanks! > > > > It looks like my review here was missed? > > I guess you sent it in between when I b4'd it and then pushed it. > > -- > Tom I'll make the changes requested by Simon and send a follow up pa

Re: [PATCH] cmd: Make bootvx independent of bootelf

2024-10-08 Thread Tom Rini
On Tue, Oct 08, 2024 at 07:57:25PM -0600, Simon Glass wrote: > Hi, > > On Tue, 8 Oct 2024 at 07:45, Tom Rini wrote: > > > > On Sun, 29 Sep 2024 18:27:39 +0900, Daniel Palmer wrote: > > > > > There are lots of usecases for running baremetal ELF > > > binaries via bootelf but if you enable bootelf

Re: [PATCH] cmd: Make bootvx independent of bootelf

2024-10-08 Thread Simon Glass
Hi, On Tue, 8 Oct 2024 at 07:45, Tom Rini wrote: > > On Sun, 29 Sep 2024 18:27:39 +0900, Daniel Palmer wrote: > > > There are lots of usecases for running baremetal ELF > > binaries via bootelf but if you enable bootelf you > > get bootvx as well and you probably don't want or need > > it. > > >

Re: [PATCH] cmd: Make bootvx independent of bootelf

2024-10-08 Thread Tom Rini
On Sun, 29 Sep 2024 18:27:39 +0900, Daniel Palmer wrote: > There are lots of usecases for running baremetal ELF > binaries via bootelf but if you enable bootelf you > get bootvx as well and you probably don't want or need > it. > > Hide bootvx behind it's own configuration option. > > [...] App

Re: [PATCH] cmd: Make bootvx independent of bootelf

2024-09-30 Thread Simon Glass
Hi Daniel, On Sun, 29 Sept 2024 at 03:27, Daniel Palmer wrote: > > There are lots of usecases for running baremetal ELF > binaries via bootelf but if you enable bootelf you > get bootvx as well and you probably don't want or need > it. > > Hide bootvx behind it's own configuration option. > > Sig

[PATCH] cmd: Make bootvx independent of bootelf

2024-09-29 Thread Daniel Palmer
There are lots of usecases for running baremetal ELF binaries via bootelf but if you enable bootelf you get bootvx as well and you probably don't want or need it. Hide bootvx behind it's own configuration option. Signed-off-by: Daniel Palmer --- cmd/Kconfig | 11 +-- cmd/elf.c | 6 ++