All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] imx8mn_ddr4_evk: Allow booting the kernel by default
@ 2021-03-04 11:07 Fabio Estevam
  2021-03-04 12:59 ` Andrey Zhizhikin
                   ` (2 more replies)
  0 siblings, 3 replies; 9+ messages in thread
From: Fabio Estevam @ 2021-03-04 11:07 UTC (permalink / raw)
  To: u-boot

Currently it is not possible to boot Linux from the SD card
by default.

Make the necessary adjustments to allow it.

Reported-by: Peter Bergin <peter@berginkonsult.se>
Signed-off-by: Fabio Estevam <festevam@gmail.com>
---
 configs/imx8mn_ddr4_evk_defconfig | 1 +
 include/configs/imx8mn_evk.h      | 4 ++--
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/configs/imx8mn_ddr4_evk_defconfig b/configs/imx8mn_ddr4_evk_defconfig
index f7ea65886d51..2c12bdabe0a0 100644
--- a/configs/imx8mn_ddr4_evk_defconfig
+++ b/configs/imx8mn_ddr4_evk_defconfig
@@ -54,6 +54,7 @@ CONFIG_SPL_OF_CONTROL=y
 CONFIG_ENV_OVERWRITE=y
 CONFIG_ENV_IS_IN_MMC=y
 CONFIG_SYS_RELOC_GD_ENV_ADDR=y
+CONFIG_SYS_MMC_ENV_DEV=1
 CONFIG_ENV_VARS_UBOOT_RUNTIME_CONFIG=y
 CONFIG_SPL_DM=y
 CONFIG_SPL_CLK_COMPOSITE_CCF=y
diff --git a/include/configs/imx8mn_evk.h b/include/configs/imx8mn_evk.h
index 1e2825337e2d..596e255ca6de 100644
--- a/include/configs/imx8mn_evk.h
+++ b/include/configs/imx8mn_evk.h
@@ -34,10 +34,10 @@
 /* Initial environment variables */
 #define CONFIG_EXTRA_ENV_SETTINGS		\
 	"script=boot.scr\0" \
-	"image=Image.itb\0" \
+	"image=Image\0" \
 	"console=ttymxc1,115200\0" \
 	"fdt_addr=0x43000000\0"			\
-	"boot_fit=try\0" \
+	"boot_fit=no\0" \
 	"fdt_file=" CONFIG_DEFAULT_FDT_FILE "\0" \
 	"initrd_addr=0x43800000\0"		\
 	"bootm_size=0x10000000\0" \
-- 
2.25.1

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [PATCH] imx8mn_ddr4_evk: Allow booting the kernel by default
  2021-03-04 11:07 [PATCH] imx8mn_ddr4_evk: Allow booting the kernel by default Fabio Estevam
@ 2021-03-04 12:59 ` Andrey Zhizhikin
  2021-03-20 14:05   ` Fabio Estevam
  2021-03-22  1:40 ` Peng Fan
  2021-04-05 12:09 ` Tom Rini
  2 siblings, 1 reply; 9+ messages in thread
From: Andrey Zhizhikin @ 2021-03-04 12:59 UTC (permalink / raw)
  To: u-boot

Hello Fabio,

On Thu, Mar 4, 2021 at 12:07 PM Fabio Estevam <festevam@gmail.com> wrote:
>
> Currently it is not possible to boot Linux from the SD card
> by default.
>
> Make the necessary adjustments to allow it.
>
> Reported-by: Peter Bergin <peter@berginkonsult.se>
> Signed-off-by: Fabio Estevam <festevam@gmail.com>

Tested-by: Andrey Zhizhikin <andrey.z@gmail.com> # imx8mn_ddr4_evk

> ---
>  configs/imx8mn_ddr4_evk_defconfig | 1 +
>  include/configs/imx8mn_evk.h      | 4 ++--
>  2 files changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/configs/imx8mn_ddr4_evk_defconfig b/configs/imx8mn_ddr4_evk_defconfig
> index f7ea65886d51..2c12bdabe0a0 100644
> --- a/configs/imx8mn_ddr4_evk_defconfig
> +++ b/configs/imx8mn_ddr4_evk_defconfig
> @@ -54,6 +54,7 @@ CONFIG_SPL_OF_CONTROL=y
>  CONFIG_ENV_OVERWRITE=y
>  CONFIG_ENV_IS_IN_MMC=y
>  CONFIG_SYS_RELOC_GD_ENV_ADDR=y
> +CONFIG_SYS_MMC_ENV_DEV=1
>  CONFIG_ENV_VARS_UBOOT_RUNTIME_CONFIG=y
>  CONFIG_SPL_DM=y
>  CONFIG_SPL_CLK_COMPOSITE_CCF=y
> diff --git a/include/configs/imx8mn_evk.h b/include/configs/imx8mn_evk.h
> index 1e2825337e2d..596e255ca6de 100644
> --- a/include/configs/imx8mn_evk.h
> +++ b/include/configs/imx8mn_evk.h
> @@ -34,10 +34,10 @@
>  /* Initial environment variables */
>  #define CONFIG_EXTRA_ENV_SETTINGS              \
>         "script=boot.scr\0" \
> -       "image=Image.itb\0" \
> +       "image=Image\0" \
>         "console=ttymxc1,115200\0" \
>         "fdt_addr=0x43000000\0"                 \
> -       "boot_fit=try\0" \
> +       "boot_fit=no\0" \
>         "fdt_file=" CONFIG_DEFAULT_FDT_FILE "\0" \
>         "initrd_addr=0x43800000\0"              \
>         "bootm_size=0x10000000\0" \
> --
> 2.25.1
>


-- 
Regards,
Andrey.

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [PATCH] imx8mn_ddr4_evk: Allow booting the kernel by default
  2021-03-04 12:59 ` Andrey Zhizhikin
@ 2021-03-20 14:05   ` Fabio Estevam
  2021-03-20 18:18     ` Stefano Babic
  0 siblings, 1 reply; 9+ messages in thread
From: Fabio Estevam @ 2021-03-20 14:05 UTC (permalink / raw)
  To: u-boot

Hi Stefano and Tom,

On Thu, Mar 4, 2021 at 9:59 AM Andrey Zhizhikin <andrey.z@gmail.com> wrote:
>
> Hello Fabio,
>
> On Thu, Mar 4, 2021 at 12:07 PM Fabio Estevam <festevam@gmail.com> wrote:
> >
> > Currently it is not possible to boot Linux from the SD card
> > by default.
> >
> > Make the necessary adjustments to allow it.
> >
> > Reported-by: Peter Bergin <peter@berginkonsult.se>
> > Signed-off-by: Fabio Estevam <festevam@gmail.com>
>
> Tested-by: Andrey Zhizhikin <andrey.z@gmail.com> # imx8mn_ddr4_evk

Please consider applying this one for 2021.04 as it allows imx8mn-evk
booting Linux by default.

Thanks

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [PATCH] imx8mn_ddr4_evk: Allow booting the kernel by default
  2021-03-20 14:05   ` Fabio Estevam
@ 2021-03-20 18:18     ` Stefano Babic
  2021-04-05 11:54       ` Fabio Estevam
  0 siblings, 1 reply; 9+ messages in thread
From: Stefano Babic @ 2021-03-20 18:18 UTC (permalink / raw)
  To: u-boot

Hi Fabio,

On 20.03.21 15:05, Fabio Estevam wrote:
> Hi Stefano and Tom,
> 
> On Thu, Mar 4, 2021 at 9:59 AM Andrey Zhizhikin <andrey.z@gmail.com> wrote:
>>
>> Hello Fabio,
>>
>> On Thu, Mar 4, 2021 at 12:07 PM Fabio Estevam <festevam@gmail.com> wrote:
>>>
>>> Currently it is not possible to boot Linux from the SD card
>>> by default.
>>>
>>> Make the necessary adjustments to allow it.
>>>
>>> Reported-by: Peter Bergin <peter@berginkonsult.se>
>>> Signed-off-by: Fabio Estevam <festevam@gmail.com>
>>
>> Tested-by: Andrey Zhizhikin <andrey.z@gmail.com> # imx8mn_ddr4_evk
> 
> Please consider applying this one for 2021.04 as it allows imx8mn-evk
> booting Linux by default.
> 

Thanks for fixing it. I pick it up for release.

Stefano

> Thanks
> 

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [PATCH] imx8mn_ddr4_evk: Allow booting the kernel by default
  2021-03-04 11:07 [PATCH] imx8mn_ddr4_evk: Allow booting the kernel by default Fabio Estevam
  2021-03-04 12:59 ` Andrey Zhizhikin
@ 2021-03-22  1:40 ` Peng Fan
  2021-04-05 12:09 ` Tom Rini
  2 siblings, 0 replies; 9+ messages in thread
From: Peng Fan @ 2021-03-22  1:40 UTC (permalink / raw)
  To: u-boot

> Subject: [PATCH] imx8mn_ddr4_evk: Allow booting the kernel by default
> 
> Currently it is not possible to boot Linux from the SD card by default.
> 
> Make the necessary adjustments to allow it.
> 
> Reported-by: Peter Bergin <peter@berginkonsult.se>
> Signed-off-by: Fabio Estevam <festevam@gmail.com>
> ---
>  configs/imx8mn_ddr4_evk_defconfig | 1 +
>  include/configs/imx8mn_evk.h      | 4 ++--
>  2 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/configs/imx8mn_ddr4_evk_defconfig
> b/configs/imx8mn_ddr4_evk_defconfig
> index f7ea65886d51..2c12bdabe0a0 100644
> --- a/configs/imx8mn_ddr4_evk_defconfig
> +++ b/configs/imx8mn_ddr4_evk_defconfig
> @@ -54,6 +54,7 @@ CONFIG_SPL_OF_CONTROL=y
> CONFIG_ENV_OVERWRITE=y  CONFIG_ENV_IS_IN_MMC=y
> CONFIG_SYS_RELOC_GD_ENV_ADDR=y
> +CONFIG_SYS_MMC_ENV_DEV=1
>  CONFIG_ENV_VARS_UBOOT_RUNTIME_CONFIG=y
>  CONFIG_SPL_DM=y
>  CONFIG_SPL_CLK_COMPOSITE_CCF=y
> diff --git a/include/configs/imx8mn_evk.h b/include/configs/imx8mn_evk.h
> index 1e2825337e2d..596e255ca6de 100644
> --- a/include/configs/imx8mn_evk.h
> +++ b/include/configs/imx8mn_evk.h
> @@ -34,10 +34,10 @@
>  /* Initial environment variables */
>  #define CONFIG_EXTRA_ENV_SETTINGS		\
>  	"script=boot.scr\0" \
> -	"image=Image.itb\0" \
> +	"image=Image\0" \
>  	"console=ttymxc1,115200\0" \
>  	"fdt_addr=0x43000000\0"			\
> -	"boot_fit=try\0" \
> +	"boot_fit=no\0" \
>  	"fdt_file=" CONFIG_DEFAULT_FDT_FILE "\0" \
>  	"initrd_addr=0x43800000\0"		\
>  	"bootm_size=0x10000000\0" \
> --

Reviewed-by: Peng Fan <peng.fan@nxp.com>

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [PATCH] imx8mn_ddr4_evk: Allow booting the kernel by default
  2021-03-20 18:18     ` Stefano Babic
@ 2021-04-05 11:54       ` Fabio Estevam
  2021-04-05 12:27         ` Stefano Babic
  0 siblings, 1 reply; 9+ messages in thread
From: Fabio Estevam @ 2021-04-05 11:54 UTC (permalink / raw)
  To: u-boot

Hi Stefano,

On Sat, Mar 20, 2021 at 3:18 PM Stefano Babic <sbabic@denx.de> wrote:

> > Please consider applying this one for 2021.04 as it allows imx8mn-evk
> > booting Linux by default.
> >
>
> Thanks for fixing it. I pick it up for release.

It seems this one is still missing. Could it be applied for 2021.04?

Thanks

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [PATCH] imx8mn_ddr4_evk: Allow booting the kernel by default
  2021-03-04 11:07 [PATCH] imx8mn_ddr4_evk: Allow booting the kernel by default Fabio Estevam
  2021-03-04 12:59 ` Andrey Zhizhikin
  2021-03-22  1:40 ` Peng Fan
@ 2021-04-05 12:09 ` Tom Rini
  2 siblings, 0 replies; 9+ messages in thread
From: Tom Rini @ 2021-04-05 12:09 UTC (permalink / raw)
  To: u-boot

On Thu, Mar 04, 2021 at 08:07:16AM -0300, Fabio Estevam wrote:

> Currently it is not possible to boot Linux from the SD card
> by default.
> 
> Make the necessary adjustments to allow it.
> 
> Reported-by: Peter Bergin <peter@berginkonsult.se>
> Signed-off-by: Fabio Estevam <festevam@gmail.com>
> Tested-by: Andrey Zhizhikin <andrey.z@gmail.com> # imx8mn_ddr4_evk
> Reviewed-by: Peng Fan <peng.fan@nxp.com>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20210405/5e31f357/attachment.sig>

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [PATCH] imx8mn_ddr4_evk: Allow booting the kernel by default
  2021-04-05 11:54       ` Fabio Estevam
@ 2021-04-05 12:27         ` Stefano Babic
  2021-04-05 12:35           ` Tom Rini
  0 siblings, 1 reply; 9+ messages in thread
From: Stefano Babic @ 2021-04-05 12:27 UTC (permalink / raw)
  To: u-boot

Hi Fabio, Tom,

On 05.04.21 13:54, Fabio Estevam wrote:
> Hi Stefano,
> 
> On Sat, Mar 20, 2021 at 3:18 PM Stefano Babic <sbabic@denx.de> wrote:
> 
>>> Please consider applying this one for 2021.04 as it allows imx8mn-evk
>>> booting Linux by default.
>>>
>>
>> Thanks for fixing it. I pick it up for release.
> 
> It seems this one is still missing. Could it be applied for 2021.04?
> 

I see in patchwork that patch is already set to "accepted" and was 
delegated to Tom, but I could not find it as well in Tom's official 
tree. I merged into u-boot-imx, but is it maybe in another Tom's branch 
ready for release ?

Regards,
Stefano


-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [PATCH] imx8mn_ddr4_evk: Allow booting the kernel by default
  2021-04-05 12:27         ` Stefano Babic
@ 2021-04-05 12:35           ` Tom Rini
  0 siblings, 0 replies; 9+ messages in thread
From: Tom Rini @ 2021-04-05 12:35 UTC (permalink / raw)
  To: u-boot

On Mon, Apr 05, 2021 at 02:27:35PM +0200, Stefano Babic wrote:
> Hi Fabio, Tom,
> 
> On 05.04.21 13:54, Fabio Estevam wrote:
> > Hi Stefano,
> > 
> > On Sat, Mar 20, 2021 at 3:18 PM Stefano Babic <sbabic@denx.de> wrote:
> > 
> > > > Please consider applying this one for 2021.04 as it allows imx8mn-evk
> > > > booting Linux by default.
> > > > 
> > > 
> > > Thanks for fixing it. I pick it up for release.
> > 
> > It seems this one is still missing. Could it be applied for 2021.04?
> > 
> 
> I see in patchwork that patch is already set to "accepted" and was delegated
> to Tom, but I could not find it as well in Tom's official tree. I merged
> into u-boot-imx, but is it maybe in another Tom's branch ready for release ?

I've applied it locally to master, but I haven't pushed since I'm
otherwise getting ready to do the release and I don't want to get CI too
bogged down just yet.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20210405/c167845c/attachment.sig>

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2021-04-05 12:35 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-04 11:07 [PATCH] imx8mn_ddr4_evk: Allow booting the kernel by default Fabio Estevam
2021-03-04 12:59 ` Andrey Zhizhikin
2021-03-20 14:05   ` Fabio Estevam
2021-03-20 18:18     ` Stefano Babic
2021-04-05 11:54       ` Fabio Estevam
2021-04-05 12:27         ` Stefano Babic
2021-04-05 12:35           ` Tom Rini
2021-03-22  1:40 ` Peng Fan
2021-04-05 12:09 ` Tom Rini

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.