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

[Michal Simek]

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.

Thanks,
Michal