All of lore.kernel.org
 help / color / mirror / Atom feed
From: Fabio Estevam <festevam@gmail.com>
To: sbabic@denx.de
Cc: peng.fan@nxp.com, u-boot@lists.denx.de, aford173@gmail.com,
	michael@amarulasolutions.com, marex@denx.de,
	Fabio Estevam <festevam@denx.de>
Subject: [PATCH 2/2] imx8mn/8mp: Select ENV_IS_NOWHERE
Date: Wed, 20 Apr 2022 18:07:57 -0300	[thread overview]
Message-ID: <20220420210757.1184891-2-festevam@gmail.com> (raw)
In-Reply-To: <20220420210757.1184891-1-festevam@gmail.com>

From: Fabio Estevam <festevam@denx.de>

Currently, on i.MX8MN/i.MX8MP (Bootrom version 2) it is not possible to
load U-Boot via serial download mode, unless CONFIG_ENV_IS_NOWHERE is
selected.

This was noticed before by Adam Ford and fixed on the imx8mn beacon
board in commit 2c7ebf7778cf ("imx: imx8mn_beacon: Fix USB booting").

Such commit log states:

"The i.MX8M Nano can boot over USB using the boot ROM instead of
adding extra code to SPL to support USB drivers, etc.  However,
when booting from USB, the environment doesnt' know where to load
and causes a hang.  Fix this hang by supporting CONFIG_ENV_IS_NOWHERE=y.
It only falls back to this condition when booting from USB, so it
does not impact MMC booting."

Select ENV_IS_NOWHERE globally when i.MX8MN or i.MX8MP are used,
so that serial download mode can be used by default on these SoCs.

Signed-off-by: Fabio Estevam <festevam@denx.de>
---
 arch/arm/mach-imx/imx8m/Kconfig | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/arm/mach-imx/imx8m/Kconfig b/arch/arm/mach-imx/imx8m/Kconfig
index 55db25062a9b..ba7328aec8a1 100644
--- a/arch/arm/mach-imx/imx8m/Kconfig
+++ b/arch/arm/mach-imx/imx8m/Kconfig
@@ -16,10 +16,12 @@ config IMX8MM
 config IMX8MN
 	bool
 	select IMX8M
+	select ENV_IS_NOWHERE
 
 config IMX8MP
 	bool
 	select IMX8M
+	select ENV_IS_NOWHERE
 
 config SYS_SOC
 	default "imx8m"
-- 
2.25.1


  reply	other threads:[~2022-04-20 21:08 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-04-20 21:07 [PATCH 1/2] imx8mn_ddr4_evk: Add USB Mass Storage support Fabio Estevam
2022-04-20 21:07 ` Fabio Estevam [this message]
2022-04-20 21:42   ` [PATCH 2/2] imx8mn/8mp: Select ENV_IS_NOWHERE Marek Vasut
2022-04-20 22:03     ` Fabio Estevam
2022-04-20 22:15       ` Marek Vasut
2022-04-21 14:17         ` Fabio Estevam
2022-04-21 14:34           ` Michael Nazzareno Trimarchi
2022-04-21 16:18             ` Fabio Estevam
2022-04-21 16:34               ` Michael Nazzareno Trimarchi
2022-04-21 16:47                 ` Fabio Estevam
2022-04-21 16:51                   ` Adam Ford
2022-04-21 17:01                     ` Michael Nazzareno Trimarchi
2022-04-21 17:03                       ` Fabio Estevam
2022-04-21 17:10                         ` Michael Nazzareno Trimarchi
2022-04-21 17:13                           ` Fabio Estevam
2022-04-21 14:38           ` Marek Vasut

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220420210757.1184891-2-festevam@gmail.com \
    --to=festevam@gmail.com \
    --cc=aford173@gmail.com \
    --cc=festevam@denx.de \
    --cc=marex@denx.de \
    --cc=michael@amarulasolutions.com \
    --cc=peng.fan@nxp.com \
    --cc=sbabic@denx.de \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.