[Buildroot] [PATCH v2 1/3] configs/icnova-a20-adb4006: Bump Linux to 6.6.56, U-Boot to 2024.10

[Buildroot] [PATCH v2 1/3] configs/icnova-a20-adb4006: Bump Linux to 6.6.56, U-Boot to 2024.10

[Ludwig Kormann]

Signed-off-by: Ludwig Kormann <ludwig.kormann@ict42.de>
---
 configs/icnova-a20-adb4006_defconfig | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/configs/icnova-a20-adb4006_defconfig b/configs/icnova-a20-adb4006_defconfig
index ba529c8db5..7eac9b429d 100644
--- a/configs/icnova-a20-adb4006_defconfig
+++ b/configs/icnova-a20-adb4006_defconfig
@@ -20,7 +20,7 @@ BR2_PACKAGE_HOST_MTOOLS=y
 # Kernel
 BR2_LINUX_KERNEL=y
 BR2_LINUX_KERNEL_CUSTOM_VERSION=y
-BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="6.6.14"
+BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="6.6.56"
 BR2_LINUX_KERNEL_DEFCONFIG="sunxi"
 BR2_LINUX_KERNEL_CONFIG_FRAGMENT_FILES="board/in-circuit/icnova-a20-adb4006/linux.fragment"
 BR2_LINUX_KERNEL_DTS_SUPPORT=y
@@ -36,11 +36,12 @@ BR2_TARGET_ROOTFS_EXT2_SIZE="128M"
 BR2_TARGET_UBOOT=y
 BR2_TARGET_UBOOT_BUILD_SYSTEM_KCONFIG=y
 BR2_TARGET_UBOOT_CUSTOM_VERSION=y
-BR2_TARGET_UBOOT_CUSTOM_VERSION_VALUE="2024.01"
+BR2_TARGET_UBOOT_CUSTOM_VERSION_VALUE="2024.10"
 BR2_TARGET_UBOOT_BOARD_DEFCONFIG="icnova-a20-adb4006"
 BR2_TARGET_UBOOT_NEEDS_DTC=y
 BR2_TARGET_UBOOT_NEEDS_PYLIBFDT=y
 BR2_TARGET_UBOOT_NEEDS_OPENSSL=y
+BR2_TARGET_UBOOT_NEEDS_GNUTLS=y
 BR2_TARGET_UBOOT_SPL=y
 BR2_TARGET_UBOOT_SPL_NAME="u-boot-sunxi-with-spl.bin"
 BR2_PACKAGE_HOST_UBOOT_TOOLS=y
-- 
2.39.2

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

[Buildroot] [PATCH v2 2/3] configs/icnova-a20-adb4006: enable BR2_DOWNLOAD_FORCE_CHECK_HASHES

[Ludwig Kormann]

Signed-off-by: Ludwig Kormann <ludwig.kormann@ict42.de>
---
 .../patches/linux-headers/linux-headers.hash                  | 1 +
 board/in-circuit/icnova-a20-adb4006/patches/linux/linux.hash  | 2 ++
 board/in-circuit/icnova-a20-adb4006/patches/uboot/uboot.hash  | 2 ++
 configs/icnova-a20-adb4006_defconfig                          | 4 ++++
 4 files changed, 9 insertions(+)
 create mode 120000 board/in-circuit/icnova-a20-adb4006/patches/linux-headers/linux-headers.hash
 create mode 100644 board/in-circuit/icnova-a20-adb4006/patches/linux/linux.hash
 create mode 100644 board/in-circuit/icnova-a20-adb4006/patches/uboot/uboot.hash

diff --git a/board/in-circuit/icnova-a20-adb4006/patches/linux-headers/linux-headers.hash b/board/in-circuit/icnova-a20-adb4006/patches/linux-headers/linux-headers.hash
new file mode 120000
index 0000000000..5808d92afe
--- /dev/null
+++ b/board/in-circuit/icnova-a20-adb4006/patches/linux-headers/linux-headers.hash
@@ -0,0 +1 @@
+../linux/linux.hash
\ No newline at end of file
diff --git a/board/in-circuit/icnova-a20-adb4006/patches/linux/linux.hash b/board/in-circuit/icnova-a20-adb4006/patches/linux/linux.hash
new file mode 100644
index 0000000000..98edd13a58
--- /dev/null
+++ b/board/in-circuit/icnova-a20-adb4006/patches/linux/linux.hash
@@ -0,0 +1,2 @@
+# Locally calculated
+sha256  f74812f78e88992c416434cb107639e13a551dbaff36bb90d6346ab16ab71a95  linux-6.6.56.tar.xz
diff --git a/board/in-circuit/icnova-a20-adb4006/patches/uboot/uboot.hash b/board/in-circuit/icnova-a20-adb4006/patches/uboot/uboot.hash
new file mode 100644
index 0000000000..904fa56c02
--- /dev/null
+++ b/board/in-circuit/icnova-a20-adb4006/patches/uboot/uboot.hash
@@ -0,0 +1,2 @@
+# Locally calculated
+sha256  b28daf4ac17e43156363078bf510297584137f6df50fced9b12df34f61a92fb0  u-boot-2024.10.tar.bz2
diff --git a/configs/icnova-a20-adb4006_defconfig b/configs/icnova-a20-adb4006_defconfig
index 7eac9b429d..6b860489ca 100644
--- a/configs/icnova-a20-adb4006_defconfig
+++ b/configs/icnova-a20-adb4006_defconfig
@@ -5,6 +5,10 @@ BR2_ARM_FPU_NEON_VFPV4=y
 # Linux headers same as kernel, a 6.6 LTS series
 BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_6_6=y
 
+# Build
+BR2_GLOBAL_PATCH_DIR="board/in-circuit/icnova-a20-adb4006/patches"
+BR2_DOWNLOAD_FORCE_CHECK_HASHES=y
+
 # System
 BR2_TARGET_GENERIC_HOSTNAME="ICnova A20 ADB4006"
 BR2_TARGET_GENERIC_ISSUE="Welcome to ICnova A20 ADB4006!"
-- 
2.39.2

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

[Buildroot] [PATCH v2 3/3] configs/icnova-a20-adb4006: add uboot patch for reliable bootclk

[Ludwig Kormann]

Up until now cpu clock gets initialized at 384 MHz, which is
the highest supported cpu clock.

Recent A20 batches show an increased percentage of modules
reacting very sensitive to operating conditions outside the
specifications.

The cpu dies very shortly after PLLs, core frequency or cpu
voltage are missconfigured. E.g.:
- uboot SPL selects 384 MHz as cpu clock which requires a cpu
  voltage of at least 1.1 V.
- Linux CPU Frequency scaling with most sun7i dts will reduce
  cpu voltage down to 1.0 V.
- When intiating a reboot or reset from linux the cpu voltage
  may keep the 1.0 V configuration and the cpu dies during SPL
  initialization.

Therefore reduce cpu clock at uboot SPL initialization down
to 144 MHz from 384 MHz.

Use patch until KConfig option in uboot becomes available.

Signed-off-by: Ludwig Kormann <ludwig.kormann@ict42.de>
---
 ...ce-cpu-clock-at-SPL-initialization-t.patch | 48 +++++++++++++++++++
 1 file changed, 48 insertions(+)
 create mode 100644 board/in-circuit/icnova-a20-adb4006/patches/uboot/0001-sunxi-sun4i-Reduce-cpu-clock-at-SPL-initialization-t.patch

diff --git a/board/in-circuit/icnova-a20-adb4006/patches/uboot/0001-sunxi-sun4i-Reduce-cpu-clock-at-SPL-initialization-t.patch b/board/in-circuit/icnova-a20-adb4006/patches/uboot/0001-sunxi-sun4i-Reduce-cpu-clock-at-SPL-initialization-t.patch
new file mode 100644
index 0000000000..edaa47ac4e
--- /dev/null
+++ b/board/in-circuit/icnova-a20-adb4006/patches/uboot/0001-sunxi-sun4i-Reduce-cpu-clock-at-SPL-initialization-t.patch
@@ -0,0 +1,48 @@
+From 3cdf8aa3ff45e35a237285c107785bc3d2c6976a Mon Sep 17 00:00:00 2001
+From: Ludwig Kormann <ludwig.kormann@ict42.de>
+Date: Wed, 31 Jan 2024 11:28:19 +0100
+Subject: [PATCH] sunxi: sun4i: Reduce cpu clock at SPL initialization to 144
+ MHz
+
+Up until now cpu clock gets initialized at 384 MHz, which is
+the highest supported cpu clock.
+
+Recent A20 batches show an increased percentage of modules
+reacting very sensitive to operating conditions outside the
+specifications.
+
+The cpu dies very shortly after PLLs, core frequency or cpu
+voltage are missconfigured. E.g.:
+- uboot SPL selects 384 MHz as cpu clock which requires a cpu
+  voltage of at least 1.1 V.
+- Linux CPU Frequency scaling with most sun7i dts will reduce
+  cpu voltage down to 1.0 V.
+- When intiating a reboot or reset from linux the cpu voltage
+  may keep the 1.0 V configuration and the cpu dies during SPL
+  initialization.
+
+Therefore reduce cpu clock at uboot SPL initialization down
+to 144 MHz from 384 MHz.
+
+Signed-off-by: Ludwig Kormann <ludwig.kormann@ict42.de>
+Upstream: Not Applicable
+---
+ arch/arm/include/asm/arch-sunxi/clock_sun4i.h | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/arch/arm/include/asm/arch-sunxi/clock_sun4i.h b/arch/arm/include/asm/arch-sunxi/clock_sun4i.h
+index 2cec91cb20..252c4c693e 100644
+--- a/arch/arm/include/asm/arch-sunxi/clock_sun4i.h
++++ b/arch/arm/include/asm/arch-sunxi/clock_sun4i.h
+@@ -141,7 +141,7 @@ struct sunxi_ccm_reg {
+ #define CCM_PLL1_CFG_SIG_DELT_PAT_EN_SHIFT	2
+ #define CCM_PLL1_CFG_FACTOR_M_SHIFT		0
+ 
+-#define PLL1_CFG_DEFAULT	0xa1005000
++#define PLL1_CFG_DEFAULT	0xa1004c01
+ 
+ #if defined CONFIG_OLD_SUNXI_KERNEL_COMPAT && defined CONFIG_MACH_SUN5I
+ /*
+-- 
+2.39.2
+
-- 
2.39.2

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH v2 3/3] configs/icnova-a20-adb4006: add uboot patch for reliable bootclk

[Fabio Estevam]

Hi Ludwig,

On Wed, Oct 16, 2024 at 9:45 AM Ludwig Kormann <ludwig.kormann@ict42.de> wrote:

> +From 3cdf8aa3ff45e35a237285c107785bc3d2c6976a Mon Sep 17 00:00:00 2001
> +From: Ludwig Kormann <ludwig.kormann@ict42.de>
> +Date: Wed, 31 Jan 2024 11:28:19 +0100
> +Subject: [PATCH] sunxi: sun4i: Reduce cpu clock at SPL initialization to 144
> + MHz
> +
> +Up until now cpu clock gets initialized at 384 MHz, which is
> +the highest supported cpu clock.
> +
> +Recent A20 batches show an increased percentage of modules
> +reacting very sensitive to operating conditions outside the
> +specifications.
> +
> +The cpu dies very shortly after PLLs, core frequency or cpu
> +voltage are missconfigured. E.g.:
> +- uboot SPL selects 384 MHz as cpu clock which requires a cpu
> +  voltage of at least 1.1 V.
> +- Linux CPU Frequency scaling with most sun7i dts will reduce
> +  cpu voltage down to 1.0 V.
> +- When intiating a reboot or reset from linux the cpu voltage
> +  may keep the 1.0 V configuration and the cpu dies during SPL
> +  initialization.
> +
> +Therefore reduce cpu clock at uboot SPL initialization down
> +to 144 MHz from 384 MHz.
> +
> +Signed-off-by: Ludwig Kormann <ludwig.kormann@ict42.de>
> +Upstream: Not Applicable

Why is this not applicable to U-Boot upstream?
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH v2 3/3] configs/icnova-a20-adb4006: add uboot patch for reliable bootclk

[Ludwig Kormann]

Hi Fabio,

resending this mail because I received an error on the last transmission 
and it also does not show in the buildroot mailing archive.

thank you for your review.

Am 16.10.24 um 16:21 schrieb Fabio Estevam:
> Hi Ludwig,
>
> On Wed, Oct 16, 2024 at 9:45 AM Ludwig Kormann 
> <ludwig.kormann@ict42.de> wrote:
>
>> +From 3cdf8aa3ff45e35a237285c107785bc3d2c6976a Mon Sep 17 00:00:00 2001
>> +From: Ludwig Kormann <ludwig.kormann@ict42.de>
>> +Date: Wed, 31 Jan 2024 11:28:19 +0100
>> +Subject: [PATCH] sunxi: sun4i: Reduce cpu clock at SPL 
>> initialization to 144
>> + MHz
>> +
>> +Up until now cpu clock gets initialized at 384 MHz, which is
>> +the highest supported cpu clock.
>> +
>> +Recent A20 batches show an increased percentage of modules
>> +reacting very sensitive to operating conditions outside the
>> +specifications.
>> +
>> +The cpu dies very shortly after PLLs, core frequency or cpu
>> +voltage are missconfigured. E.g.:
>> +- uboot SPL selects 384 MHz as cpu clock which requires a cpu
>> + voltage of at least 1.1 V.
>> +- Linux CPU Frequency scaling with most sun7i dts will reduce
>> + cpu voltage down to 1.0 V.
>> +- When intiating a reboot or reset from linux the cpu voltage
>> + may keep the 1.0 V configuration and the cpu dies during SPL
>> + initialization.
>> +
>> +Therefore reduce cpu clock at uboot SPL initialization down
>> +to 144 MHz from 384 MHz.
>> +
>> +Signed-off-by: Ludwig Kormann <ludwig.kormann@ict42.de>
>> +Upstream: Not Applicable
> Why is this not applicable to U-Boot upstream?
Maybe I'm misunderstanding what the "Upstream"-tag is meant to be used for.

This patch will most likely never be upstreamed to u-boot. I had 
attempted to do this:
https://lists.denx.de/pipermail/u-boot/2024-January/544897.html

But the conclusion was that introducing a KConfig option would be a 
better option, which i started working on in:
https://lists.denx.de/pipermail/u-boot/2024-February/545080.html

But as of now the KConfig option is not available yet, so I'd like to 
use the patch as an intermediate solution for our cpu module.

kind regards
Ludwig

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH v2 1/3] configs/icnova-a20-adb4006: Bump Linux to 6.6.56, U-Boot to 2024.10

[Thomas Petazzoni via buildroot]

On Wed, 16 Oct 2024 14:45:35 +0200
Ludwig Kormann <ludwig.kormann@ict42.de> wrote:

> Signed-off-by: Ludwig Kormann <ludwig.kormann@ict42.de>
> ---
>  configs/icnova-a20-adb4006_defconfig | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)

I have applied the entire series. It would be good to add a readme file
in board/in-circuit/icnova-a20-adb4006/ like we have for other boards.
You can look at other readme.txt files for other boards for examples.

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH v2 3/3] configs/icnova-a20-adb4006: add uboot patch for reliable bootclk

[Thomas Petazzoni via buildroot]

Hello Ludwig,

On Mon, 21 Oct 2024 08:38:06 +0200
Ludwig Kormann <ludwig.kormann@ict42.de> wrote:

> Maybe I'm misunderstanding what the "Upstream"-tag is meant to be used for.
> 
> This patch will most likely never be upstreamed to u-boot. I had 
> attempted to do this:
> https://lists.denx.de/pipermail/u-boot/2024-January/544897.html
> 
> But the conclusion was that introducing a KConfig option would be a 
> better option, which i started working on in:
> https://lists.denx.de/pipermail/u-boot/2024-February/545080.html
> 
> But as of now the KConfig option is not available yet, so I'd like to 
> use the patch as an intermediate solution for our cpu module.

OK. I applied your patch as-is to move forward, but we really need a
solution that is accepted in upstream U-Boot. We can't keep
board-specific patches forever in Buildroot.

Best regards,

Thomas
-- 
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot