On Fri, Oct 14, 2022 at 01:51:54PM +0200, Jan Kiszka wrote: > On 21.10.21 01:34, Heinrich Schuchardt wrote: > > On 9/25/21 2:30 AM, Simon Glass wrote: > >> This code should never have been added as it builds a new feature on top > >> of legacy code. This has already been improved with the dependency on > >> BLK. > >> > >> Add a dependency on DM_ETH also, to avoid needing to deal with this old > >> code. > >> > >> Boards which want EFI_LOADER should migrate to driver model first. > >> > >> Note this patch is included to resolve the following build error: > >> > >> lib/efi_loader/efi_runtime.c:680:16: error: ‘CONFIG_SYS_TEXT_BASE’ > >> undeclared (first use in this function); did you mean > >> ‘CONFIG_SYS_SRAM_BASE’? > >> 680 | ulong base = CONFIG_SYS_TEXT_BASE; > >> | ^~~~~~~~~~~~~~~~~~~~ > >> | CONFIG_SYS_SRAM_BASE > >> > >> Signed-off-by: Simon Glass <s...@chromium.org> > > > > Reviewed-by: Heinrich Schuchardt <xypron.g...@gmx.de> > > > > How to deal with boards that need CONFIG_NET but do not actually > implement any driver (yet)? This now broke UEFI for the IOT2050 which > needs NET for network-related device tree setup (see also [1]) and > enforces a local hack for us.
Yes, looking at the patch, this should probably use NETDEVICES and not NET as the test? Would that fix your problem, Jan? -- Tom
signature.asc
Description: PGP signature
