Re: [PATCH 10/10] xilinx: zynqmp: Support vendor specific board_init

[Michal Simek <michal.simek@amd.com>]

On 6/8/22 18:20, Stefan Herbrechtsmeier wrote:
> [CAUTION: External Email]
> 
> From: Stefan Herbrechtsmeier <stefan.herbrechtsmeier@weidmueller.com>
> 
> Add a board_init_xilinx function to allow foreign vendors to reuse the
> xilinx zynqmp board code and add addition code to the board_init
> function.

Do you plan to add support for that board who will be using it?

> 
> Signed-off-by: Stefan Herbrechtsmeier <stefan.herbrechtsmeier@weidmueller.com>
> 
> ---
> 
>   board/xilinx/common/board.c  | 5 +++++
>   board/xilinx/common/board.h  | 2 ++
>   board/xilinx/zynqmp/zynqmp.c | 2 +-
>   3 files changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/board/xilinx/common/board.c b/board/xilinx/common/board.c
> index 5f2afb9def..643959bee7 100644
> --- a/board/xilinx/common/board.c
> +++ b/board/xilinx/common/board.c
> @@ -401,6 +401,11 @@ void *board_fdt_blob_setup(int *err)
>   }
>   #endif
> 
> +int board_init_xilinx(void)
> +{
> +       return 0;
> +}

This file is used by zynq/zynqmp/versal platforms. I expect there is going to be 
any message that this function is unused on zynq and versal.

M