[PATCH] octeontx: do not require cavium BDK node to be present

Stefan Roese sr at denx.de
Fri Jun 18 08:04:37 CEST 2021


On 18.06.21 01:31, Tim Harvey wrote:
> The cavium,bdk node is a non-standard dt node used by the BDK and
> therefore it is removed from the dt before booting Linux. Do not
> require this node to exist as it won't for standard dt's.
> 
> Signed-off-by: Tim Harvey <tharvey at gateworks.com>

Reviewed-by: Stefan Roese <sr at denx.de>

Thanks,
Stefan

> ---
>   board/Marvell/octeontx/board-fdt.c | 20 ++++++++------------
>   1 file changed, 8 insertions(+), 12 deletions(-)
> 
> diff --git a/board/Marvell/octeontx/board-fdt.c b/board/Marvell/octeontx/board-fdt.c
> index 0b05ef11e9..1db2a4a267 100644
> --- a/board/Marvell/octeontx/board-fdt.c
> +++ b/board/Marvell/octeontx/board-fdt.c
> @@ -281,20 +281,16 @@ int ft_board_setup(void *blob, struct bd_info *bd)
>   	}
>   
>   	if (blob) {
> +		/* delete cavium,bdk node if it exists */
>   		offset = fdt_path_offset(blob, "/cavium,bdk");
> -		if (offset < 0) {
> -			printf("ERROR: FDT BDK node not found\n");
> -			return offset;
> -		}
> -
> -		/* delete node */
> -		ret = fdt_del_node(blob, offset);
> -		if (ret < 0) {
> -			printf("WARNING : could not remove bdk node\n");
> -			return ret;
> +		if (offset >= 0) {
> +			ret = fdt_del_node(blob, offset);
> +			if (ret < 0) {
> +				printf("WARNING : could not remove bdk node\n");
> +				return ret;
> +			}
> +			debug("%s deleted bdk node\n", __func__);
>   		}
> -
> -		debug("%s deleted bdk node\n", __func__);
>   	}
>   
>   	return 0;
> 


Viele Grüße,
Stefan

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr at denx.de


More information about the U-Boot mailing list