[PATCH] configs: add PineTab defconfig
From: Arnaud Ferraris arnaud.ferraris@collabora.com
The PineTab device-tree is already in u-boot, this commit adds the corresponding defconfig, based on pinephone_defconfig.
Signed-off-by: Arnaud Ferraris arnaud.ferraris@collabora.com
--- a/board/sunxi/MAINTAINERS +++ b/board/sunxi/MAINTAINERS @@ -471,6 +471,11 @@ M: Samuel Holland samuel@sholland.org S: Maintained F: configs/pinephone_defconfig
+PINETAB BOARD +M: Arnaud Ferraris arnaud.ferraris@collabora.com +S: Maintained +F: configs/pinetab_defconfig + R16 EVB PARROT BOARD M: Quentin Schulz quentin.schulz@free-electrons.com S: Maintained --- /dev/null +++ b/configs/pinetab_defconfig @@ -0,0 +1,21 @@ +CONFIG_ARM=y +CONFIG_ARCH_SUNXI=y +CONFIG_SPL=y +CONFIG_IDENT_STRING="" +CONFIG_MACH_SUN50I=y +CONFIG_SUNXI_DRAM_LPDDR3_STOCK=y +CONFIG_DRAM_CLK=552 +CONFIG_DRAM_ZQ=3881949 +CONFIG_MMC_SUNXI_SLOT_EXTRA=2 +# CONFIG_VIDEO_DE2 is not set +CONFIG_DEFAULT_DEVICE_TREE="sun50i-a64-pinetab" +# CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set +CONFIG_BOOTDELAY=0 +CONFIG_SYS_CONSOLE_INFO_QUIET=y +# CONFIG_DISPLAY_CPUINFO is not set +# CONFIG_DISPLAY_BOARDINFO is not set +# CONFIG_SPL_RAW_IMAGE_SUPPORT is not set +# CONFIG_SPL_BANNER_PRINT is not set +# CONFIG_SPL_POWER_SUPPORT is not set +# CONFIG_NET is not set +# CONFIG_EFI_LOADER is not set
On Sun, Mar 7, 2021 at 1:24 PM Nicolas Boulenguez nicolas@debian.org wrote:
From: Arnaud Ferraris arnaud.ferraris@collabora.com
The PineTab device-tree is already in u-boot, this commit adds the corresponding defconfig, based on pinephone_defconfig.
Should it have support to deal with the two variants where they have differing screens?
Signed-off-by: Arnaud Ferraris arnaud.ferraris@collabora.com
--- a/board/sunxi/MAINTAINERS +++ b/board/sunxi/MAINTAINERS @@ -471,6 +471,11 @@ M: Samuel Holland samuel@sholland.org S: Maintained F: configs/pinephone_defconfig
+PINETAB BOARD +M: Arnaud Ferraris arnaud.ferraris@collabora.com +S: Maintained +F: configs/pinetab_defconfig
R16 EVB PARROT BOARD M: Quentin Schulz quentin.schulz@free-electrons.com S: Maintained --- /dev/null +++ b/configs/pinetab_defconfig @@ -0,0 +1,21 @@ +CONFIG_ARM=y +CONFIG_ARCH_SUNXI=y +CONFIG_SPL=y +CONFIG_IDENT_STRING="" +CONFIG_MACH_SUN50I=y +CONFIG_SUNXI_DRAM_LPDDR3_STOCK=y +CONFIG_DRAM_CLK=552 +CONFIG_DRAM_ZQ=3881949 +CONFIG_MMC_SUNXI_SLOT_EXTRA=2 +# CONFIG_VIDEO_DE2 is not set +CONFIG_DEFAULT_DEVICE_TREE="sun50i-a64-pinetab" +# CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set +CONFIG_BOOTDELAY=0 +CONFIG_SYS_CONSOLE_INFO_QUIET=y +# CONFIG_DISPLAY_CPUINFO is not set +# CONFIG_DISPLAY_BOARDINFO is not set +# CONFIG_SPL_RAW_IMAGE_SUPPORT is not set +# CONFIG_SPL_BANNER_PRINT is not set +# CONFIG_SPL_POWER_SUPPORT is not set +# CONFIG_NET is not set +# CONFIG_EFI_LOADER is not set
On 2021-03-07, Nicolas Boulenguez wrote:
From: Arnaud Ferraris arnaud.ferraris@collabora.com
The PineTab device-tree is already in u-boot, this commit adds the corresponding defconfig, based on pinephone_defconfig.
Signed-off-by: Arnaud Ferraris arnaud.ferraris@collabora.com
...
--- /dev/null +++ b/configs/pinetab_defconfig
...
+CONFIG_BOOTDELAY=0
Setting bootdelay to 0 it almost impossible to debug issues in a running u-boot.
The default of 2 seconds that distro_bootcmd uses tries to strike a balance between not slowing the boot down too much while still being reasonably able to get into a u-boot shell when something goes wrong.
Individual users or vendors can set this value as they see fit, but this doesn't seem like a good default for mainline u-boot, at least to me.
live well, vagrant
On Sun, 07 Mar 2021 12:34:14 -0800 Vagrant Cascadian vagrant@debian.org wrote:
On 2021-03-07, Nicolas Boulenguez wrote:
From: Arnaud Ferraris arnaud.ferraris@collabora.com
The PineTab device-tree is already in u-boot, this commit adds the corresponding defconfig, based on pinephone_defconfig.
Signed-off-by: Arnaud Ferraris arnaud.ferraris@collabora.com
...
--- /dev/null +++ b/configs/pinetab_defconfig
...
+CONFIG_BOOTDELAY=0
Setting bootdelay to 0 it almost impossible to debug issues in a running u-boot.
The default of 2 seconds that distro_bootcmd uses tries to strike a balance between not slowing the boot down too much while still being reasonably able to get into a u-boot shell when something goes wrong.
Individual users or vendors can set this value as they see fit, but this doesn't seem like a good default for mainline u-boot, at least to me.
Yeah, we just had a similar discussion recently about the Pinephone. I think we keep the default of 2 seconds for the mainline defconfig, by not having any explicit entry in that file, so it reverts to the platform default. People can always change this in their .config, even with automated build systems, by using a simple sed command.
Cheers, Andre
On Sun, 7 Mar 2021 13:53:56 +0100 Nicolas Boulenguez nicolas@debian.org wrote:
Hi,
From: Arnaud Ferraris arnaud.ferraris@collabora.com
The PineTab device-tree is already in u-boot, this commit adds the corresponding defconfig, based on pinephone_defconfig.
Signed-off-by: Arnaud Ferraris arnaud.ferraris@collabora.com
--- a/board/sunxi/MAINTAINERS +++ b/board/sunxi/MAINTAINERS @@ -471,6 +471,11 @@ M: Samuel Holland samuel@sholland.org S: Maintained F: configs/pinephone_defconfig
+PINETAB BOARD +M: Arnaud Ferraris arnaud.ferraris@collabora.com +S: Maintained +F: configs/pinetab_defconfig
Arnaud, do you agree with this? Happy to take your patch via Nicolas, but for the maintainer entry I would like to have some confirmation.
R16 EVB PARROT BOARD M: Quentin Schulz quentin.schulz@free-electrons.com S: Maintained --- /dev/null +++ b/configs/pinetab_defconfig @@ -0,0 +1,21 @@ +CONFIG_ARM=y +CONFIG_ARCH_SUNXI=y +CONFIG_SPL=y +CONFIG_IDENT_STRING=""
Having "Allwinner Technology" here is indeed weird and probably not really justified anymore, given the "support" we see from Allwinner. I wonder if we should scrap this for all boards. Also it makes the line longer than 80 characters.
+CONFIG_MACH_SUN50I=y +CONFIG_SUNXI_DRAM_LPDDR3_STOCK=y +CONFIG_DRAM_CLK=552 +CONFIG_DRAM_ZQ=3881949 +CONFIG_MMC_SUNXI_SLOT_EXTRA=2 +# CONFIG_VIDEO_DE2 is not set +CONFIG_DEFAULT_DEVICE_TREE="sun50i-a64-pinetab" +# CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set +CONFIG_BOOTDELAY=0
I answered in the other email about the boot delay already.
So what is the reason for all those other options below? Is there any particular reason they were all disabled? I can buy CONFIG_NET, but the rest seems unnecessary. There doesn't seem to be a driver for the PineTab panel in U-Boot, so this is solely suppressing a few lines on the serial? Since this would be surely for debug only, I think it's useful to have them, normal users wouldn't see them anyway.
+CONFIG_SYS_CONSOLE_INFO_QUIET=y +# CONFIG_DISPLAY_CPUINFO is not set +# CONFIG_DISPLAY_BOARDINFO is not set +# CONFIG_SPL_RAW_IMAGE_SUPPORT is not set +# CONFIG_SPL_BANNER_PRINT is not set +# CONFIG_SPL_POWER_SUPPORT is not set +# CONFIG_NET is not set +# CONFIG_EFI_LOADER is not set
We should definitely keep EFI_LOADER.
Cheers, Andre
于 2021年3月8日 GMT+08:00 上午8:12:24, Andre Przywara andre.przywara@arm.com 写到:
On Sun, 7 Mar 2021 13:53:56 +0100 Nicolas Boulenguez nicolas@debian.org wrote:
Hi,
From: Arnaud Ferraris arnaud.ferraris@collabora.com
The PineTab device-tree is already in u-boot, this commit adds the
corresponding
defconfig, based on pinephone_defconfig.
Signed-off-by: Arnaud Ferraris arnaud.ferraris@collabora.com
--- a/board/sunxi/MAINTAINERS +++ b/board/sunxi/MAINTAINERS @@ -471,6 +471,11 @@ M: Samuel Holland samuel@sholland.org S: Maintained F: configs/pinephone_defconfig
+PINETAB BOARD +M: Arnaud Ferraris arnaud.ferraris@collabora.com +S: Maintained +F: configs/pinetab_defconfig
Arnaud, do you agree with this? Happy to take your patch via Nicolas, but for the maintainer entry I would like to have some confirmation.
R16 EVB PARROT BOARD M: Quentin Schulz quentin.schulz@free-electrons.com S: Maintained --- /dev/null +++ b/configs/pinetab_defconfig @@ -0,0 +1,21 @@ +CONFIG_ARM=y +CONFIG_ARCH_SUNXI=y +CONFIG_SPL=y +CONFIG_IDENT_STRING=""
Having "Allwinner Technology" here is indeed weird and probably not really justified anymore, given the "support" we see from Allwinner. I wonder if we should scrap this for all boards. Also it makes the line longer than 80 characters.
But if we do so, it should be in Kconfig, not defconfig.
+CONFIG_MACH_SUN50I=y +CONFIG_SUNXI_DRAM_LPDDR3_STOCK=y +CONFIG_DRAM_CLK=552 +CONFIG_DRAM_ZQ=3881949 +CONFIG_MMC_SUNXI_SLOT_EXTRA=2 +# CONFIG_VIDEO_DE2 is not set +CONFIG_DEFAULT_DEVICE_TREE="sun50i-a64-pinetab" +# CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set +CONFIG_BOOTDELAY=0
I answered in the other email about the boot delay already.
So what is the reason for all those other options below? Is there any particular reason they were all disabled? I can buy CONFIG_NET, but the rest seems unnecessary. There doesn't seem to be a driver for the PineTab panel in U-Boot, so this is solely suppressing a few lines on the serial? Since this would be surely for debug only, I think it's useful to have them, normal users wouldn't see them anyway.
+CONFIG_SYS_CONSOLE_INFO_QUIET=y +# CONFIG_DISPLAY_CPUINFO is not set +# CONFIG_DISPLAY_BOARDINFO is not set +# CONFIG_SPL_RAW_IMAGE_SUPPORT is not set +# CONFIG_SPL_BANNER_PRINT is not set +# CONFIG_SPL_POWER_SUPPORT is not set +# CONFIG_NET is not set +# CONFIG_EFI_LOADER is not set
We should definitely keep EFI_LOADER.
Cheers, Andre
Hi,
Le 08/03/2021 à 01:12, Andre Przywara a écrit :
On Sun, 7 Mar 2021 13:53:56 +0100 Nicolas Boulenguez nicolas@debian.org wrote:
Hi,
From: Arnaud Ferraris arnaud.ferraris@collabora.com
The PineTab device-tree is already in u-boot, this commit adds the corresponding defconfig, based on pinephone_defconfig.
Signed-off-by: Arnaud Ferraris arnaud.ferraris@collabora.com
--- a/board/sunxi/MAINTAINERS +++ b/board/sunxi/MAINTAINERS @@ -471,6 +471,11 @@ M: Samuel Holland samuel@sholland.org S: Maintained F: configs/pinephone_defconfig
+PINETAB BOARD +M: Arnaud Ferraris arnaud.ferraris@collabora.com +S: Maintained +F: configs/pinetab_defconfig
Arnaud, do you agree with this? Happy to take your patch via Nicolas, but for the maintainer entry I would like to have some confirmation.
Yes, I'm perfectly fine with this.
R16 EVB PARROT BOARD M: Quentin Schulz quentin.schulz@free-electrons.com S: Maintained --- /dev/null +++ b/configs/pinetab_defconfig @@ -0,0 +1,21 @@ +CONFIG_ARM=y +CONFIG_ARCH_SUNXI=y +CONFIG_SPL=y +CONFIG_IDENT_STRING=""
Having "Allwinner Technology" here is indeed weird and probably not really justified anymore, given the "support" we see from Allwinner. I wonder if we should scrap this for all boards. Also it makes the line longer than 80 characters.
+CONFIG_MACH_SUN50I=y +CONFIG_SUNXI_DRAM_LPDDR3_STOCK=y +CONFIG_DRAM_CLK=552 +CONFIG_DRAM_ZQ=3881949 +CONFIG_MMC_SUNXI_SLOT_EXTRA=2 +# CONFIG_VIDEO_DE2 is not set +CONFIG_DEFAULT_DEVICE_TREE="sun50i-a64-pinetab" +# CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set +CONFIG_BOOTDELAY=0
I answered in the other email about the boot delay already.
So what is the reason for all those other options below? Is there any particular reason they were all disabled? I can buy CONFIG_NET, but the rest seems unnecessary. There doesn't seem to be a driver for the PineTab panel in U-Boot, so this is solely suppressing a few lines on the serial? Since this would be surely for debug only, I think it's useful to have them, normal users wouldn't see them anyway.
I initially created this defconfig from a downstream pinephone_defconfig IIRC, I guess I just carried those over without thinking too much about it. I'll improve the defconfig and post an improved version soon.
Cheers, Arnaud
+CONFIG_SYS_CONSOLE_INFO_QUIET=y +# CONFIG_DISPLAY_CPUINFO is not set +# CONFIG_DISPLAY_BOARDINFO is not set +# CONFIG_SPL_RAW_IMAGE_SUPPORT is not set +# CONFIG_SPL_BANNER_PRINT is not set +# CONFIG_SPL_POWER_SUPPORT is not set +# CONFIG_NET is not set +# CONFIG_EFI_LOADER is not set
We should definitely keep EFI_LOADER.
Cheers, Andre
participants (6)
-
Andre Przywara -
Arnaud Ferraris -
Icenowy Zheng -
Nicolas Boulenguez -
Peter Robinson -
Vagrant Cascadian