[RESEND PATCH v2] configs: imx8mn_bsh_smm_s2: remove console from bootargs

[sbabic]

> The Linux kernel device tree already specifies the device to be used for
> boot console output with a stdout-path property under /chosen.
> Commit 36b661dc919da ("Merge branch 'next'") re-added the console
> setting that commit bede82f750752 ("configs: imx8mn_bsh_smm_s2: remove
> console from bootargs") had previously removed.
> Fixes: 36b661dc919da ("Merge branch 'next'")
> Signed-off-by: Dario Binacchi 
> Reviewed-by: Fabio Estevam 
Applied to u-boot-imx, master, thanks !

Best regards,
Stefano Babic

-- 
=
DENX Software Engineering GmbH,Managing Director: Erika Unter  
HRB 165235 Munich,   Office: Kirchenstr.5, 82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=

Re: [RESEND PATCH v2] configs: imx8mn_bsh_smm_s2: remove console from bootargs

[Stefano Babic]

On 13.02.23 14:28, Michael Nazzareno Trimarchi wrote:

Hi Stefano

On Wed, Feb 8, 2023 at 9:15 AM Dario Binacchi
 wrote:


The Linux kernel device tree already specifies the device to be used for
boot console output with a stdout-path property under /chosen.

Commit 36b661dc919da ("Merge branch 'next'") re-added the console
setting that commit bede82f750752 ("configs: imx8mn_bsh_smm_s2: remove
console from bootargs") had previously removed.

Fixes: 36b661dc919da ("Merge branch 'next'")
Signed-off-by: Dario Binacchi 
Reviewed-by: Fabio Estevam 

---

Changes in v2:
- Add the 'Reviewed-by' tag.
- Improve commit message.

  include/configs/imx8mn_bsh_smm_s2.h | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/configs/imx8mn_bsh_smm_s2.h 
b/include/configs/imx8mn_bsh_smm_s2.h
index e97b8e871d22..deeed9c2f582 100644
--- a/include/configs/imx8mn_bsh_smm_s2.h
+++ b/include/configs/imx8mn_bsh_smm_s2.h
@@ -14,7 +14,7 @@
  #include 

  #define NANDARGS \
-   "nandargs=setenv bootargs console=${console} " \
+   "nandargs=setenv bootargs " \
 "${optargs} " \
 "mtdparts=${mtdparts} " \
 "root=${nandroot} " \
--


This patch is a regression fix, can you pick it?


I'll do it, thanks !

Stefano



Michael


2.32.0



--
=
DENX Software Engineering GmbH,Managing Director: Erika Unter
HRB 165235 Munich,   Office: Kirchenstr.5, 82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=

Re: [RESEND PATCH v2] configs: imx8mn_bsh_smm_s2: remove console from bootargs

[Michael Nazzareno Trimarchi]

Hi Stefano

On Wed, Feb 8, 2023 at 9:15 AM Dario Binacchi
 wrote:
>
> The Linux kernel device tree already specifies the device to be used for
> boot console output with a stdout-path property under /chosen.
>
> Commit 36b661dc919da ("Merge branch 'next'") re-added the console
> setting that commit bede82f750752 ("configs: imx8mn_bsh_smm_s2: remove
> console from bootargs") had previously removed.
>
> Fixes: 36b661dc919da ("Merge branch 'next'")
> Signed-off-by: Dario Binacchi 
> Reviewed-by: Fabio Estevam 
>
> ---
>
> Changes in v2:
> - Add the 'Reviewed-by' tag.
> - Improve commit message.
>
>  include/configs/imx8mn_bsh_smm_s2.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/configs/imx8mn_bsh_smm_s2.h 
> b/include/configs/imx8mn_bsh_smm_s2.h
> index e97b8e871d22..deeed9c2f582 100644
> --- a/include/configs/imx8mn_bsh_smm_s2.h
> +++ b/include/configs/imx8mn_bsh_smm_s2.h
> @@ -14,7 +14,7 @@
>  #include 
>
>  #define NANDARGS \
> -   "nandargs=setenv bootargs console=${console} " \
> +   "nandargs=setenv bootargs " \
> "${optargs} " \
> "mtdparts=${mtdparts} " \
> "root=${nandroot} " \
> --

This patch is a regression fix, can you pick it?

Michael

> 2.32.0
>

[RESEND PATCH v2] configs: imx8mn_bsh_smm_s2: remove console from bootargs

[Dario Binacchi]

The Linux kernel device tree already specifies the device to be used for
boot console output with a stdout-path property under /chosen.

Commit 36b661dc919da ("Merge branch 'next'") re-added the console
setting that commit bede82f750752 ("configs: imx8mn_bsh_smm_s2: remove
console from bootargs") had previously removed.

Fixes: 36b661dc919da ("Merge branch 'next'")
Signed-off-by: Dario Binacchi 
Reviewed-by: Fabio Estevam 

---

Changes in v2:
- Add the 'Reviewed-by' tag.
- Improve commit message.

 include/configs/imx8mn_bsh_smm_s2.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/configs/imx8mn_bsh_smm_s2.h 
b/include/configs/imx8mn_bsh_smm_s2.h
index e97b8e871d22..deeed9c2f582 100644
--- a/include/configs/imx8mn_bsh_smm_s2.h
+++ b/include/configs/imx8mn_bsh_smm_s2.h
@@ -14,7 +14,7 @@
 #include 
 
 #define NANDARGS \
-   "nandargs=setenv bootargs console=${console} " \
+   "nandargs=setenv bootargs " \
"${optargs} " \
"mtdparts=${mtdparts} " \
"root=${nandroot} " \
-- 
2.32.0