[PATCH] ARM: mvebu: clearfog: don't override set fdtfile env

Only set the fdtfile environment variable when not already set.

Fixes: 867572f09ebe6 ("ARM: mvebu: clearfog: run-time selection of DT file")
Cc: Joel Johnson <redacted>
Reported-by: Andre Heider <redacted>
Signed-off-by: Baruch Siach <baruch@tkos.co.il>
---
 board/solidrun/clearfog/clearfog.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/board/solidrun/clearfog/clearfog.c b/board/solidrun/clearfog/clearfog.c
index d5535ebebecd..92443d5177d9 100644
--- a/board/solidrun/clearfog/clearfog.c
+++ b/board/solidrun/clearfog/clearfog.c

@@ -240,6 +240,9 @@ int board_eth_init(struct bd_info *bis)
 
 int board_late_init(void)
 {
+	if (env_get("fdtfile"))
+		return 0;
+
 	cf_read_tlv_data();
 
 	if (sr_product_is(&cf_tlv_data, "Clearfog Base"))

-- 
2.28.0

On 09/09/2020 14:14, Baruch Siach wrote:

Only set the fdtfile environment variable when not already set.

Fixes: 867572f09ebe6 ("ARM: mvebu: clearfog: run-time selection of DT file")
Cc: Joel Johnson <redacted>
Reported-by: Andre Heider <redacted>
Signed-off-by: Baruch Siach <baruch@tkos.co.il>

Reviewed-by: Andre Heider <redacted>

On 09.09.20 14:14, Baruch Siach wrote:

Only set the fdtfile environment variable when not already set.

Fixes: 867572f09ebe6 ("ARM: mvebu: clearfog: run-time selection of DT file")
Cc: Joel Johnson <redacted>
Reported-by: Andre Heider <redacted>
Signed-off-by: Baruch Siach <baruch@tkos.co.il>

Applied to u-boot-marvell/master

Thanks,
Stefan