[PATCH] configs: add PineTab defconfig

[PATCH] configs: add PineTab defconfig

[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 <arnaud.ferraris@collabora.com>
---
 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 <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
diff --git a/configs/pinetab_defconfig b/configs/pinetab_defconfig
new file mode 100644
index 0000000000..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

[Andre Przywara]

On Mon,  6 Sep 2021 23:06:13 +0200
Arnaud Ferraris <arnaud.ferraris@collabora.com> 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 <arnaud.ferraris@collabora.com>

Reviewed-by: Andre Przywara <andre.przywara@arm.com>

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 <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
> diff --git a/configs/pinetab_defconfig b/configs/pinetab_defconfig
> new file mode 100644
> index 0000000000..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

[Arnaud Ferraris]

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
> 

[PATCH] configs: add PineTab defconfig

[Icenowy Zheng]

? 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

[PATCH] configs: add PineTab defconfig

[Andre Przywara]

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

[PATCH] configs: add PineTab defconfig

[Andre Przywara]

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

[PATCH] configs: add PineTab defconfig

[Vagrant Cascadian]

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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 227 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20210307/db8a451a/attachment.sig>

[PATCH] configs: add PineTab defconfig

[Peter Robinson]

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

[PATCH] configs: add PineTab defconfig

[Nicolas Boulenguez]

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