* [PATCH] ARM: mvebu: clearfog: don't override set fdtfile env
@ 2020-09-09 12:14 Baruch Siach
2020-09-10 4:54 ` Andre Heider
2020-09-24 10:37 ` Stefan Roese
0 siblings, 2 replies; 3+ messages in thread
From: Baruch Siach @ 2020-09-09 12:14 UTC (permalink / raw)
To: u-boot
Only set the fdtfile environment variable when not already set.
Fixes: 867572f09ebe6 ("ARM: mvebu: clearfog: run-time selection of DT file")
Cc: Joel Johnson <mrjoel@lixil.net>
Reported-by: Andre Heider <a.heider@gmail.com>
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
^ permalink raw reply related [flat|nested] 3+ messages in thread* [PATCH] ARM: mvebu: clearfog: don't override set fdtfile env
2020-09-09 12:14 [PATCH] ARM: mvebu: clearfog: don't override set fdtfile env Baruch Siach
@ 2020-09-10 4:54 ` Andre Heider
2020-09-24 10:37 ` Stefan Roese
1 sibling, 0 replies; 3+ messages in thread
From: Andre Heider @ 2020-09-10 4:54 UTC (permalink / raw)
To: u-boot
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 <mrjoel@lixil.net>
> Reported-by: Andre Heider <a.heider@gmail.com>
> Signed-off-by: Baruch Siach <baruch@tkos.co.il>
Reviewed-by: Andre Heider <a.heider@gmail.com>
^ permalink raw reply [flat|nested] 3+ messages in thread* [PATCH] ARM: mvebu: clearfog: don't override set fdtfile env
2020-09-09 12:14 [PATCH] ARM: mvebu: clearfog: don't override set fdtfile env Baruch Siach
2020-09-10 4:54 ` Andre Heider
@ 2020-09-24 10:37 ` Stefan Roese
1 sibling, 0 replies; 3+ messages in thread
From: Stefan Roese @ 2020-09-24 10:37 UTC (permalink / raw)
To: u-boot
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 <mrjoel@lixil.net>
> Reported-by: Andre Heider <a.heider@gmail.com>
> Signed-off-by: Baruch Siach <baruch@tkos.co.il>
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(&cf_tlv_data, "Clearfog Base"))
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-09-24 10:37 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-09-09 12:14 [PATCH] ARM: mvebu: clearfog: don't override set fdtfile env Baruch Siach
2020-09-10 4:54 ` Andre Heider
2020-09-24 10:37 ` Stefan Roese
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox