diff mbox series

[v2,6/6] board: apalis-tk1: launch toradex easy installer in usb recovery

Message ID 20210723063950.11326-7-oleksandr.suvorov@toradex.com
State Superseded
Delegated to: Tom Warren
Headers show
Series Toradex boards' specific patches | expand

Commit Message

Oleksandr Suvorov July 23, 2021, 6:39 a.m. UTC
From: Marcel Ziswiler <marcel.ziswiler@toradex.com>

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 <marcel.ziswiler@toradex.com>
Signed-off-by: Oleksandr Suvorov <oleksandr.suvorov@toradex.com>
---

Changes in v2:
- remove 2 patches not meet requirements.
END

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

Comments

Stefano Babic Aug. 6, 2021, 11:03 a.m. UTC | #1
On 23.07.21 08:39, Oleksandr Suvorov wrote:
> From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> 
> 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 <marcel.ziswiler@toradex.com>
> Signed-off-by: Oleksandr Suvorov <oleksandr.suvorov@toradex.com>
> ---
> 
> Changes in v2:
> - remove 2 patches not meet requirements.
> END
> 
>   board/toradex/apalis-tk1/apalis-tk1.c | 13 +++++++++++--
>   1 file changed, 11 insertions(+), 2 deletions(-)
> 
> diff --git a/board/toradex/apalis-tk1/apalis-tk1.c b/board/toradex/apalis-tk1/apalis-tk1.c
> index b97617cfca..f418f2c39f 100644
> --- a/board/toradex/apalis-tk1/apalis-tk1.c
> +++ b/board/toradex/apalis-tk1/apalis-tk1.c
> @@ -38,8 +38,17 @@
>   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");

These above are very long lines making checkpatch unhgappy. Can we 
switch to a multiline ?

Best regards,
Stefano

> +	}
>   
>   	/* PCB Version Indication: V1.2 and later have GPIO_PV0 wired to GND */
>   	gpio_request(TEGRA_GPIO(V, 0), "PCB Version Indication");
>
Marcel Ziswiler Aug. 11, 2021, 9:50 a.m. UTC | #2
On Fri, 2021-08-06 at 13:03 +0200, Stefano Babic wrote:
> On 23.07.21 08:39, Oleksandr Suvorov wrote:
> > From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> > 
> > 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 <marcel.ziswiler@toradex.com>
> > Signed-off-by: Oleksandr Suvorov <oleksandr.suvorov@toradex.com>
> > ---
> > 
> > Changes in v2:
> > - remove 2 patches not meet requirements.
> > END
> > 
> >   board/toradex/apalis-tk1/apalis-tk1.c | 13 +++++++++++--
> >   1 file changed, 11 insertions(+), 2 deletions(-)
> > 
> > diff --git a/board/toradex/apalis-tk1/apalis-tk1.c b/board/toradex/apalis-tk1/apalis-tk1.c
> > index b97617cfca..f418f2c39f 100644
> > --- a/board/toradex/apalis-tk1/apalis-tk1.c
> > +++ b/board/toradex/apalis-tk1/apalis-tk1.c
> > @@ -38,8 +38,17 @@
> >   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");
> 
> These above are very long lines making checkpatch unhgappy. Can we 
> switch to a multiline ?

Yes, I fully agree. I will pick it up and send a v3 shortly as Oleksandr unfortunately meanwhile left our
company.

> Best regards,
> Stefano
> 
> > +       }
> >   
> >         /* PCB Version Indication: V1.2 and later have GPIO_PV0 wired to GND */
> >         gpio_request(TEGRA_GPIO(V, 0), "PCB Version Indication");
diff mbox series

Patch

diff --git a/board/toradex/apalis-tk1/apalis-tk1.c b/board/toradex/apalis-tk1/apalis-tk1.c
index b97617cfca..f418f2c39f 100644
--- a/board/toradex/apalis-tk1/apalis-tk1.c
+++ b/board/toradex/apalis-tk1/apalis-tk1.c
@@ -38,8 +38,17 @@ 
 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");