RE: [PATCH v3] board: apalis-tk1: launch toradex easy installer in usb recovery

2021-08-11 Thread Tom Warren
> The rest of that series which was i.MX specific was already applied by 
> Stefano. So if you could take this one as a singleton that would be great. 
> Thanks!
Will do, but it won't happen until at least next Monday as I'm OOTO til then.

Tom
--
nvpublic

-Original Message-
From: Marcel Ziswiler  
Sent: Wednesday, August 11, 2021 12:56 PM
To: u-boot@lists.denx.de; Tom Warren ; sba...@denx.de
Cc: s...@chromium.org; Oleksandr Suvorov 
Subject: Re: [PATCH v3] board: apalis-tk1: launch toradex easy installer in usb 
recovery

External email: Use caution opening links or attachments


On Wed, 2021-08-11 at 16:45 +, Tom Warren wrote:
> I'm fine with someone else taking this tegra patch in if it's part of 
> a larger series. If not, let me know and I'll take it as a singleton.

The rest of that series which was i.MX specific was already applied by Stefano. 
So if you could take this one as a singleton that would be great. Thanks!

> Tom

Cheers

Marcel

> --
> nvpublic
>
> -Original Message-
> From: Stefano Babic 
> Sent: Wednesday, August 11, 2021 6:24 AM
> To: Marcel Ziswiler ; u-boot@lists.denx.de
> Cc: Stefano Babic ; Simon Glass ; 
> Marcel Ziswiler ; Oleksandr Suvorov 
> ; Tom Warren 
> Subject: Re: [PATCH v3] board: apalis-tk1: launch toradex easy 
> installer in usb recovery
>
> External email: Use caution opening links or attachments
>
>
> Hallo Marcel,
>
> On 11.08.21 15:12, Marcel Ziswiler wrote:
> > From: Marcel Ziswiler 
> >
> > The USB recovery mode is used by Toradex to load the Toradex Easy 
> > Installer image which supports further system images installation.
> > Prepare for loading and launching the Toradex Easy Installer if the 
> > USB Recovery mode is activated.
> >
> > Signed-off-by: Marcel Ziswiler 
> > Signed-off-by: Oleksandr Suvorov 
> >
> > ---
> >
>
> Added Tom in CC as he's in charge for Tegra (I have just reviewed V2 because 
> it was part of i.MX series).
>
> Best regards,
> Stefano
>
> > Changes in v3:
> > - Switch very long lines to multiline to make checkpatch happy as
> >suggested by Stefano.
> >
> >   board/toradex/apalis-tk1/apalis-tk1.c | 20 ++--
> >   1 file changed, 18 insertions(+), 2 deletions(-)
> >
> > diff --git a/board/toradex/apalis-tk1/apalis-tk1.c
> > b/board/toradex/apalis-tk1/apalis-tk1.c
> > index b97617cfca3..2769b546010 100644
> > --- a/board/toradex/apalis-tk1/apalis-tk1.c
> > +++ b/board/toradex/apalis-tk1/apalis-tk1.c
> > @@ -38,8 +38,24 @@
> >   int arch_misc_init(void)
> >   {
> >   if (readl(NV_PA_BASE_SRAM + NVBOOTINFOTABLE_BOOTTYPE) ==
> > - NVBOOTTYPE_RECOVERY)
> > - printf("USB recovery mode\n");
> > + NVBOOTTYPE_RECOVERY) {
> > + printf("USB recovery mode, attempting to boot Toradex Easy "
> > +"Installer\n");
> > + env_set("bootdelay", "-2");
> > + env_set("defargs", "pcie_aspm=off user_debug=30");
> > + env_set("fdt_high", "");
> > + env_set("initrd_high", "");
> > +
> > + env_set("setup", "env set setupargs igb_mac=${ethaddr} "
> > + "consoleblank=0 no_console_suspend=1 "
> > + "console=${console},${baudrate}n8 ${memargs}");
> > + env_set("teziargs", "rootfstype=squashfs root=/dev/ram quiet "
> > + "autoinstall");
> > + env_set("vidargs", "video=HDMI-A-1:640x480-16@60D");
> > + env_set("bootcmd", "run setup; env set bootargs ${defargs} "
> > + "${setupargs} ${vidargs} ${teziargs}; bootm 
> > 0x80208000"
> > + "#config@${soc}-${fdt_module}-${fdt_board}.dtb");
> > + }
> >
> >   /* PCB Version Indication: V1.2 and later have GPIO_PV0 wired to GND 
> > */
> >   gpio_request(TEGRA_GPIO(V, 0), "PCB Version Indication");


Re: [PATCH v3] board: apalis-tk1: launch toradex easy installer in usb recovery

2021-08-11 Thread Marcel Ziswiler
On Wed, 2021-08-11 at 16:45 +, Tom Warren wrote:
> I'm fine with someone else taking this tegra patch in if it's part of a 
> larger series. If not, let me know
> and I'll take it as a singleton.

The rest of that series which was i.MX specific was already applied by Stefano. 
So if you could take this one
as a singleton that would be great. Thanks!

> Tom

Cheers

Marcel

> --
> nvpublic
> 
> -Original Message-
> From: Stefano Babic  
> Sent: Wednesday, August 11, 2021 6:24 AM
> To: Marcel Ziswiler ; u-boot@lists.denx.de
> Cc: Stefano Babic ; Simon Glass ; Marcel 
> Ziswiler
> ; Oleksandr Suvorov 
> ; Tom Warren
> 
> Subject: Re: [PATCH v3] board: apalis-tk1: launch toradex easy installer in 
> usb recovery
> 
> External email: Use caution opening links or attachments
> 
> 
> Hallo Marcel,
> 
> On 11.08.21 15:12, Marcel Ziswiler wrote:
> > From: Marcel Ziswiler 
> > 
> > The USB recovery mode is used by Toradex to load the Toradex Easy 
> > Installer image which supports further system images installation.
> > Prepare for loading and launching the Toradex Easy Installer if the 
> > USB Recovery mode is activated.
> > 
> > Signed-off-by: Marcel Ziswiler 
> > Signed-off-by: Oleksandr Suvorov 
> > 
> > ---
> > 
> 
> Added Tom in CC as he's in charge for Tegra (I have just reviewed V2 because 
> it was part of i.MX series).
> 
> Best regards,
> Stefano
> 
> > Changes in v3:
> > - Switch very long lines to multiline to make checkpatch happy as
> >    suggested by Stefano.
> > 
> >   board/toradex/apalis-tk1/apalis-tk1.c | 20 ++--
> >   1 file changed, 18 insertions(+), 2 deletions(-)
> > 
> > diff --git a/board/toradex/apalis-tk1/apalis-tk1.c 
> > b/board/toradex/apalis-tk1/apalis-tk1.c
> > index b97617cfca3..2769b546010 100644
> > --- a/board/toradex/apalis-tk1/apalis-tk1.c
> > +++ b/board/toradex/apalis-tk1/apalis-tk1.c
> > @@ -38,8 +38,24 @@
> >   int arch_misc_init(void)
> >   {
> >   if (readl(NV_PA_BASE_SRAM + NVBOOTINFOTABLE_BOOTTYPE) ==
> > - NVBOOTTYPE_RECOVERY)
> > - printf("USB recovery mode\n");
> > + NVBOOTTYPE_RECOVERY) {
> > + printf("USB recovery mode, attempting to boot Toradex Easy "
> > +    "Installer\n");
> > + env_set("bootdelay", "-2");
> > + env_set("defargs", "pcie_aspm=off user_debug=30");
> > + env_set("fdt_high", "");
> > + env_set("initrd_high", "");
> > +
> > + env_set("setup", "env set setupargs igb_mac=${ethaddr} "
> > + "consoleblank=0 no_console_suspend=1 "
> > + "console=${console},${baudrate}n8 ${memargs}");
> > + env_set("teziargs", "rootfstype=squashfs root=/dev/ram quiet "
> > + "autoinstall");
> > + env_set("vidargs", "video=HDMI-A-1:640x480-16@60D");
> > + env_set("bootcmd", "run setup; env set bootargs ${defargs} "
> > + "${setupargs} ${vidargs} ${teziargs}; bootm 
> > 0x80208000"
> > + "#config@${soc}-${fdt_module}-${fdt_board}.dtb");
> > + }
> > 
> >   /* PCB Version Indication: V1.2 and later have GPIO_PV0 wired to GND 
> > */
> >   gpio_request(TEGRA_GPIO(V, 0), "PCB Version Indication");


RE: [PATCH v3] board: apalis-tk1: launch toradex easy installer in usb recovery

2021-08-11 Thread Tom Warren
I'm fine with someone else taking this tegra patch in if it's part of a larger 
series. If not, let me know and I'll take it as a singleton.

Tom

--
nvpublic

-Original Message-
From: Stefano Babic  
Sent: Wednesday, August 11, 2021 6:24 AM
To: Marcel Ziswiler ; u-boot@lists.denx.de
Cc: Stefano Babic ; Simon Glass ; Marcel 
Ziswiler ; Oleksandr Suvorov 
; Tom Warren 
Subject: Re: [PATCH v3] board: apalis-tk1: launch toradex easy installer in usb 
recovery

External email: Use caution opening links or attachments


Hallo Marcel,

On 11.08.21 15:12, Marcel Ziswiler wrote:
> From: Marcel Ziswiler 
>
> The USB recovery mode is used by Toradex to load the Toradex Easy 
> Installer image which supports further system images installation.
> Prepare for loading and launching the Toradex Easy Installer if the 
> USB Recovery mode is activated.
>
> Signed-off-by: Marcel Ziswiler 
> Signed-off-by: Oleksandr Suvorov 
>
> ---
>

Added Tom in CC as he's in charge for Tegra (I have just reviewed V2 because it 
was part of i.MX series).

Best regards,
Stefano

> Changes in v3:
> - Switch very long lines to multiline to make checkpatch happy as
>suggested by Stefano.
>
>   board/toradex/apalis-tk1/apalis-tk1.c | 20 ++--
>   1 file changed, 18 insertions(+), 2 deletions(-)
>
> diff --git a/board/toradex/apalis-tk1/apalis-tk1.c 
> b/board/toradex/apalis-tk1/apalis-tk1.c
> index b97617cfca3..2769b546010 100644
> --- a/board/toradex/apalis-tk1/apalis-tk1.c
> +++ b/board/toradex/apalis-tk1/apalis-tk1.c
> @@ -38,8 +38,24 @@
>   int arch_misc_init(void)
>   {
>   if (readl(NV_PA_BASE_SRAM + NVBOOTINFOTABLE_BOOTTYPE) ==
> - NVBOOTTYPE_RECOVERY)
> - printf("USB recovery mode\n");
> + NVBOOTTYPE_RECOVERY) {
> + printf("USB recovery mode, attempting to boot Toradex Easy "
> +"Installer\n");
> + env_set("bootdelay", "-2");
> + env_set("defargs", "pcie_aspm=off user_debug=30");
> + env_set("fdt_high", "");
> + env_set("initrd_high", "");
> +
> + env_set("setup", "env set setupargs igb_mac=${ethaddr} "
> + "consoleblank=0 no_console_suspend=1 "
> + "console=${console},${baudrate}n8 ${memargs}");
> + env_set("teziargs", "rootfstype=squashfs root=/dev/ram quiet "
> + "autoinstall");
> + env_set("vidargs", "video=HDMI-A-1:640x480-16@60D");
> + env_set("bootcmd", "run setup; env set bootargs ${defargs} "
> + "${setupargs} ${vidargs} ${teziargs}; bootm 0x80208000"
> + "#config@${soc}-${fdt_module}-${fdt_board}.dtb");
> + }
>
>   /* PCB Version Indication: V1.2 and later have GPIO_PV0 wired to GND */
>   gpio_request(TEGRA_GPIO(V, 0), "PCB Version Indication");
>


--
=
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: sba...@denx.de 
=


Re: [PATCH v3] board: apalis-tk1: launch toradex easy installer in usb recovery

2021-08-11 Thread Stefano Babic

Hallo Marcel,

On 11.08.21 15:12, Marcel Ziswiler wrote:

From: Marcel Ziswiler 

The USB recovery mode is used by Toradex to load the Toradex Easy
Installer image which supports further system images installation.
Prepare for loading and launching the Toradex Easy Installer if the
USB Recovery mode is activated.

Signed-off-by: Marcel Ziswiler 
Signed-off-by: Oleksandr Suvorov 

---



Added Tom in CC as he's in charge for Tegra (I have just reviewed V2 
because it was part of i.MX series).


Best regards,
Stefano


Changes in v3:
- Switch very long lines to multiline to make checkpatch happy as
   suggested by Stefano.

  board/toradex/apalis-tk1/apalis-tk1.c | 20 ++--
  1 file changed, 18 insertions(+), 2 deletions(-)

diff --git a/board/toradex/apalis-tk1/apalis-tk1.c 
b/board/toradex/apalis-tk1/apalis-tk1.c
index b97617cfca3..2769b546010 100644
--- a/board/toradex/apalis-tk1/apalis-tk1.c
+++ b/board/toradex/apalis-tk1/apalis-tk1.c
@@ -38,8 +38,24 @@
  int arch_misc_init(void)
  {
if (readl(NV_PA_BASE_SRAM + NVBOOTINFOTABLE_BOOTTYPE) ==
-   NVBOOTTYPE_RECOVERY)
-   printf("USB recovery mode\n");
+   NVBOOTTYPE_RECOVERY) {
+   printf("USB recovery mode, attempting to boot Toradex Easy "
+  "Installer\n");
+   env_set("bootdelay", "-2");
+   env_set("defargs", "pcie_aspm=off user_debug=30");
+   env_set("fdt_high", "");
+   env_set("initrd_high", "");
+
+   env_set("setup", "env set setupargs igb_mac=${ethaddr} "
+   "consoleblank=0 no_console_suspend=1 "
+   "console=${console},${baudrate}n8 ${memargs}");
+   env_set("teziargs", "rootfstype=squashfs root=/dev/ram quiet "
+   "autoinstall");
+   env_set("vidargs", "video=HDMI-A-1:640x480-16@60D");
+   env_set("bootcmd", "run setup; env set bootargs ${defargs} "
+   "${setupargs} ${vidargs} ${teziargs}; bootm 0x80208000"
+   "#config@${soc}-${fdt_module}-${fdt_board}.dtb");
+   }
  
  	/* PCB Version Indication: V1.2 and later have GPIO_PV0 wired to GND */

gpio_request(TEGRA_GPIO(V, 0), "PCB Version Indication");




--
=
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: sba...@denx.de
=