[PATCH 02/16] xilinx_mbv: Remove empty config header

[Tom Rini]

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 <trini at konsulko.com>
> > ---
> > Cc: Michal Simek <michal.simek at amd.com>
> > ---
> >   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 <michal.simek at amd.com>
> > - */
> 
> Acked-by: Michal Simek <michal.simek at amd.com>
> 
> 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20240123/f534d8f2/attachment.sig>