On Tue, Jan 23, 2024 at 08:53:21AM +0100, Michal Simek wrote: > > > On 1/22/24 23:39, Tom Rini wrote: > > Now that we support having CONFIG_SYS_CONFIG_NAME be unset to indicate a > > lack of board.h file, unset this on the xilinx_mbv platforms and remove > > the otherwise empty file. > > > > Signed-off-by: Tom Rini <[email protected]> > > --- > > Cc: Michal Simek <[email protected]> > > --- > > board/xilinx/mbv/Kconfig | 3 --- > > board/xilinx/mbv/MAINTAINERS | 1 - > > include/configs/xilinx_mbv.h | 6 ------ > > 3 files changed, 10 deletions(-) > > delete mode 100644 include/configs/xilinx_mbv.h > > > > diff --git a/board/xilinx/mbv/Kconfig b/board/xilinx/mbv/Kconfig > > index 4bc9f72c541b..d2dec397ed6f 100644 > > --- a/board/xilinx/mbv/Kconfig > > +++ b/board/xilinx/mbv/Kconfig > > @@ -9,9 +9,6 @@ config SYS_VENDOR > > config SYS_CPU > > default "generic" > > -config SYS_CONFIG_NAME > > - default "xilinx_mbv" > > - > > config TEXT_BASE > > default 0x80000000 if !RISCV_SMODE > > default 0x80400000 if RISCV_SMODE && ARCH_RV32I > > diff --git a/board/xilinx/mbv/MAINTAINERS b/board/xilinx/mbv/MAINTAINERS > > index 445654fe740e..db9f03388df9 100644 > > --- a/board/xilinx/mbv/MAINTAINERS > > +++ b/board/xilinx/mbv/MAINTAINERS > > @@ -4,4 +4,3 @@ S: Maintained > > F: arch/riscv/dts/xilinx-mbv* > > F: board/xilinx/mbv/ > > F: configs/xilinx_mbv* > > -F: include/configs/xilinx_mbv.h > > diff --git a/include/configs/xilinx_mbv.h b/include/configs/xilinx_mbv.h > > deleted file mode 100644 > > index dba398aeec49..000000000000 > > --- a/include/configs/xilinx_mbv.h > > +++ /dev/null > > @@ -1,6 +0,0 @@ > > -/* SPDX-License-Identifier: GPL-2.0 */ > > -/* > > - * (C) Copyright 2023, Advanced Micro Devices, Inc. > > - * > > - * Michal Simek <[email protected]> > > - */ > > Acked-by: Michal Simek <[email protected]> > > But maybe in future we need to revert this patch to add more options there. > Or do you plan to remove all of these configs? > I can imagine distro boot wiring but that can be done via vars file only but > it is not practical.
Do you mean that plain text environment feels impractical for standard boot (which should be used instead of the pure scripts of distro_boot) ? -- Tom
signature.asc
Description: PGP signature
