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

2020-09-24 Thread Stefan Roese 

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 
Reported-by: Andre Heider 
Signed-off-by: Baruch Siach 


Applied to u-boot-marvell/master

Thanks,
Stefan


---
  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(_tlv_data, "Clearfog Base"))

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

2020-09-09 Thread Andre Heider 

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 
Reported-by: Andre Heider 
Signed-off-by: Baruch Siach 


Reviewed-by: Andre Heider

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

2020-09-09 Thread Baruch Siach 
Only set the fdtfile environment variable when not already set.

Fixes: 867572f09ebe6 ("ARM: mvebu: clearfog: run-time selection of DT file")
Cc: Joel Johnson 
Reported-by: Andre Heider 
Signed-off-by: Baruch Siach 
---
 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(_tlv_data, "Clearfog Base"))
-- 
2.28.0