Re: [PATCH] configs: add PineTab defconfig
On Mon, 6 Sep 2021 23:06:13 +0200 Arnaud Ferraris wrote: > The PineTab device-tree is already in u-boot, this commit adds the > corresponding > defconfig, based on pinephone_defconfig. > > Signed-off-by: Arnaud Ferraris Reviewed-by: Andre Przywara Thanks, Andre > --- > board/sunxi/MAINTAINERS | 5 + > configs/pinetab_defconfig | 10 ++ > 2 files changed, 15 insertions(+) > create mode 100644 configs/pinetab_defconfig > > diff --git a/board/sunxi/MAINTAINERS b/board/sunxi/MAINTAINERS > index 4fc26077b2..2543c94de7 100644 > --- a/board/sunxi/MAINTAINERS > +++ b/board/sunxi/MAINTAINERS > @@ -471,6 +471,11 @@ M: Samuel Holland > S: Maintained > F: configs/pinephone_defconfig > > +PINETAB BOARD > +M: Arnaud Ferraris > +S: Maintained > +F: configs/pinetab_defconfig > + > R16 EVB PARROT BOARD > M: Quentin Schulz > S: Maintained > diff --git a/configs/pinetab_defconfig b/configs/pinetab_defconfig > new file mode 100644 > index 00..0cc24146b3 > --- /dev/null > +++ b/configs/pinetab_defconfig > @@ -0,0 +1,10 @@ > +CONFIG_ARM=y > +CONFIG_ARCH_SUNXI=y > +CONFIG_DEFAULT_DEVICE_TREE="sun50i-a64-pinetab" > +CONFIG_SPL=y > +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_SYS_MALLOC_CLEAR_ON_INIT is not set
[PATCH] configs: add PineTab defconfig
The PineTab device-tree is already in u-boot, this commit adds the corresponding defconfig, based on pinephone_defconfig. Signed-off-by: Arnaud Ferraris --- board/sunxi/MAINTAINERS | 5 + configs/pinetab_defconfig | 10 ++ 2 files changed, 15 insertions(+) create mode 100644 configs/pinetab_defconfig diff --git a/board/sunxi/MAINTAINERS b/board/sunxi/MAINTAINERS index 4fc26077b2..2543c94de7 100644 --- a/board/sunxi/MAINTAINERS +++ b/board/sunxi/MAINTAINERS @@ -471,6 +471,11 @@ M: Samuel Holland S: Maintained F: configs/pinephone_defconfig +PINETAB BOARD +M: Arnaud Ferraris +S: Maintained +F: configs/pinetab_defconfig + R16 EVB PARROT BOARD M: Quentin Schulz S: Maintained diff --git a/configs/pinetab_defconfig b/configs/pinetab_defconfig new file mode 100644 index 00..0cc24146b3 --- /dev/null +++ b/configs/pinetab_defconfig @@ -0,0 +1,10 @@ +CONFIG_ARM=y +CONFIG_ARCH_SUNXI=y +CONFIG_DEFAULT_DEVICE_TREE="sun50i-a64-pinetab" +CONFIG_SPL=y +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_SYS_MALLOC_CLEAR_ON_INIT is not set -- 2.33.0
Re: [PATCH] configs: add PineTab defconfig
Hi, Le 08/03/2021 à 01:12, Andre Przywara a écrit : > On Sun, 7 Mar 2021 13:53:56 +0100 > Nicolas Boulenguez wrote: > > Hi, > >> From: Arnaud Ferraris >> >> The PineTab device-tree is already in u-boot, this commit adds the >> corresponding >> defconfig, based on pinephone_defconfig. >> >> Signed-off-by: Arnaud Ferraris >> >> --- a/board/sunxi/MAINTAINERS >> +++ b/board/sunxi/MAINTAINERS >> @@ -471,6 +471,11 @@ M: Samuel Holland >> S: Maintained >> F: configs/pinephone_defconfig >> >> +PINETAB BOARD >> +M: Arnaud Ferraris >> +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 >> 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 >
Re: [PATCH] configs: add PineTab defconfig
于 2021年3月8日 GMT+08:00 上午8:12:24, Andre Przywara 写到: >On Sun, 7 Mar 2021 13:53:56 +0100 >Nicolas Boulenguez wrote: > >Hi, > >> From: Arnaud Ferraris >> >> The PineTab device-tree is already in u-boot, this commit adds the >corresponding >> defconfig, based on pinephone_defconfig. >> >> Signed-off-by: Arnaud Ferraris >> >> --- a/board/sunxi/MAINTAINERS >> +++ b/board/sunxi/MAINTAINERS >> @@ -471,6 +471,11 @@ M: Samuel Holland >> S: Maintained >> F: configs/pinephone_defconfig >> >> +PINETAB BOARD >> +M: Arnaud Ferraris >> +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 >> 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
Re: [PATCH] configs: add PineTab defconfig
On Sun, 7 Mar 2021 13:53:56 +0100 Nicolas Boulenguez wrote: Hi, > From: Arnaud Ferraris > > The PineTab device-tree is already in u-boot, this commit adds the > corresponding > defconfig, based on pinephone_defconfig. > > Signed-off-by: Arnaud Ferraris > > --- a/board/sunxi/MAINTAINERS > +++ b/board/sunxi/MAINTAINERS > @@ -471,6 +471,11 @@ M: Samuel Holland > S: Maintained > F: configs/pinephone_defconfig > > +PINETAB BOARD > +M: Arnaud Ferraris > +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 > 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
Re: [PATCH] configs: add PineTab defconfig
On Sun, 07 Mar 2021 12:34:14 -0800 Vagrant Cascadian wrote: > On 2021-03-07, Nicolas Boulenguez wrote: > > From: Arnaud Ferraris > > > > The PineTab device-tree is already in u-boot, this commit adds the > > corresponding > > defconfig, based on pinephone_defconfig. > > > > Signed-off-by: Arnaud Ferraris > ... > > --- /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
Re: [PATCH] configs: add PineTab defconfig
On 2021-03-07, Nicolas Boulenguez wrote: > From: Arnaud Ferraris > > The PineTab device-tree is already in u-boot, this commit adds the > corresponding > defconfig, based on pinephone_defconfig. > > Signed-off-by: Arnaud Ferraris ... > --- /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 signature.asc Description: PGP signature
Re: [PATCH] configs: add PineTab defconfig
On Sun, Mar 7, 2021 at 1:24 PM Nicolas Boulenguez wrote: > > From: Arnaud Ferraris > > 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 > > --- a/board/sunxi/MAINTAINERS > +++ b/board/sunxi/MAINTAINERS > @@ -471,6 +471,11 @@ M: Samuel Holland > S: Maintained > F: configs/pinephone_defconfig > > +PINETAB BOARD > +M: Arnaud Ferraris > +S: Maintained > +F: configs/pinetab_defconfig > + > R16 EVB PARROT BOARD > M: Quentin Schulz > 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
[PATCH] configs: add PineTab defconfig
From: Arnaud Ferraris The PineTab device-tree is already in u-boot, this commit adds the corresponding defconfig, based on pinephone_defconfig. Signed-off-by: Arnaud Ferraris --- a/board/sunxi/MAINTAINERS +++ b/board/sunxi/MAINTAINERS @@ -471,6 +471,11 @@ M: Samuel Holland S: Maintained F: configs/pinephone_defconfig +PINETAB BOARD +M: Arnaud Ferraris +S: Maintained +F: configs/pinetab_defconfig + R16 EVB PARROT BOARD M: Quentin Schulz 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