diff mbox series

[2/2] board: phytec: common: k3: Expose product infos to Linux

Message ID 20241108181914.3358114-2-d.schultz@phytec.de
State Changes Requested
Delegated to: Tom Rini
Headers show
Series [1/2] board: phytec: common: Add product information to FTD | expand

Commit Message

Daniel Schultz Nov. 8, 2024, 6:19 p.m. UTC
Call 'phytec_ft_board_fixup' in the common K3 board code
to expose the product name and part number to Linux.

Signed-off-by: Daniel Schultz <d.schultz@phytec.de>
---
 board/phytec/common/k3/board.c | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

Comments

Wadim Egorov Nov. 11, 2024, 8:20 a.m. UTC | #1
Am 08.11.24 um 19:19 schrieb Daniel Schultz:
> Call 'phytec_ft_board_fixup' in the common K3 board code
> to expose the product name and part number to Linux.
> 
> Signed-off-by: Daniel Schultz <d.schultz@phytec.de>
> ---
>   board/phytec/common/k3/board.c | 14 ++++++++++++++
>   1 file changed, 14 insertions(+)
> 
> diff --git a/board/phytec/common/k3/board.c b/board/phytec/common/k3/board.c
> index 3d7e090ccaa..14a10f34e53 100644
> --- a/board/phytec/common/k3/board.c
> +++ b/board/phytec/common/k3/board.c
> @@ -99,8 +99,22 @@ int board_late_init(void)
>   #if IS_ENABLED(CONFIG_OF_LIBFDT) && IS_ENABLED(CONFIG_OF_BOARD_SETUP)
>   int ft_board_setup(void *blob, struct bd_info *bd)
>   {
> +	struct phytec_eeprom_data data;
> +	int ret;
> +
>   	fdt_copy_fixed_partitions(blob);
>   
> +	ret = phytec_eeprom_data_setup(&data, 0, EEPROM_ADDR);
> +	if (ret || !data.valid)
> +		return 0;
> +
> +	ret = phytec_ft_board_fixup(&data, blob, bd);
> +	if (ret) {
> +		pr_err("%s: Failed to add PHYTEC information to fdt.\n",
> +		       __func__);
> +		return 0;

This return is not needed. Other than that, the patch looks good to me.

Reviewed-by: Wadim Egorov <w.egorov@phytec.de>

> +	}
> +
>   	return 0;
>   }
>   #endif
diff mbox series

Patch

diff --git a/board/phytec/common/k3/board.c b/board/phytec/common/k3/board.c
index 3d7e090ccaa..14a10f34e53 100644
--- a/board/phytec/common/k3/board.c
+++ b/board/phytec/common/k3/board.c
@@ -99,8 +99,22 @@  int board_late_init(void)
 #if IS_ENABLED(CONFIG_OF_LIBFDT) && IS_ENABLED(CONFIG_OF_BOARD_SETUP)
 int ft_board_setup(void *blob, struct bd_info *bd)
 {
+	struct phytec_eeprom_data data;
+	int ret;
+
 	fdt_copy_fixed_partitions(blob);
 
+	ret = phytec_eeprom_data_setup(&data, 0, EEPROM_ADDR);
+	if (ret || !data.valid)
+		return 0;
+
+	ret = phytec_ft_board_fixup(&data, blob, bd);
+	if (ret) {
+		pr_err("%s: Failed to add PHYTEC information to fdt.\n",
+		       __func__);
+		return 0;
+	}
+
 	return 0;
 }
 #endif