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

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@gateworks.com

Applied to u-boot-marvell/master

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