* [Buildroot] [PATCH] configs/andes_ae350_45: remove defconfig, broken
@ 2025-08-18 20:32 Thomas Petazzoni via buildroot
2025-08-19 9:39 ` Leo Liang via buildroot
0 siblings, 1 reply; 5+ messages in thread
From: Thomas Petazzoni via buildroot @ 2025-08-18 20:32 UTC (permalink / raw)
To: buildroot; +Cc: Thomas Petazzoni, Leo Yu-Chi Liang
This configuration doesn't build anymore as its kernel version has
been removed from https://github.com/andestech/linux/.
This has been failing for over 2 months without anyone bothering to
fix it, so let's drop this defconfig.
Fixes:
https://gitlab.com/buildroot.org/buildroot/-/jobs/11012272348
Cc: Leo Yu-Chi Liang <ycliang@andestech.com>
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
---
.checkpackageignore | 4 -
DEVELOPERS | 2 -
board/andes/ae350/genimage_sdcard.cfg | 26 -----
...-Support-DTS-of-ftsdc010-driver-for-.patch | 38 -------
...-tree-blob-address-at-8-byte-boundar.patch | 42 -------
board/andes/ae350/post-build.sh | 3 -
board/andes/ae350/readme.txt | 106 ------------------
.../boot/extlinux/extlinux.conf | 4 -
board/andes/ae350/uboot.config.fragment | 6 -
configs/andes_ae350_45_defconfig | 40 -------
10 files changed, 271 deletions(-)
delete mode 100644 board/andes/ae350/genimage_sdcard.cfg
delete mode 100644 board/andes/ae350/patches/uboot/0001-mmc-ftsdc010_mci-Support-DTS-of-ftsdc010-driver-for-.patch
delete mode 100644 board/andes/ae350/patches/uboot/0002-spl-Align-device-tree-blob-address-at-8-byte-boundar.patch
delete mode 100755 board/andes/ae350/post-build.sh
delete mode 100644 board/andes/ae350/readme.txt
delete mode 100644 board/andes/ae350/rootfs_overlay/boot/extlinux/extlinux.conf
delete mode 100644 board/andes/ae350/uboot.config.fragment
delete mode 100644 configs/andes_ae350_45_defconfig
diff --git a/.checkpackageignore b/.checkpackageignore
index 554c05ef70..8b21f78357 100644
--- a/.checkpackageignore
+++ b/.checkpackageignore
@@ -1,7 +1,4 @@
board/amarula/vyasa/post-build.sh Shellcheck
-board/andes/ae350/patches/uboot/0001-mmc-ftsdc010_mci-Support-DTS-of-ftsdc010-driver-for-.patch lib_patch.Upstream
-board/andes/ae350/patches/uboot/0002-spl-Align-device-tree-blob-address-at-8-byte-boundar.patch lib_patch.Upstream
-board/andes/ae350/post-build.sh Shellcheck
board/arcturus/aarch64-ucls1012a/post-build.sh Shellcheck
board/arcturus/aarch64-ucls1012a/post-image.sh Shellcheck
board/aspeed/common/post-image.sh Shellcheck
@@ -101,7 +98,6 @@ boot/syslinux/0014-Fix-build-with-binutils-note-gnu-property-section.patch lib_p
boot/syslinux/0016-Workaround-multiple-definition-of-symbol-errors.patch lib_patch.Upstream
boot/syslinux/0017-Replace-builtin-strlen-that-appears-to-get-optimized.patch lib_patch.Upstream
configs/am574x_idk_defconfig lib_defconfig.ForceCheckHash
-configs/andes_ae350_45_defconfig lib_defconfig.ForceCheckHash
configs/arcturus_ucls1012a_defconfig lib_defconfig.ForceCheckHash
configs/arcturus_ucp1020_defconfig lib_defconfig.ForceCheckHash
configs/aspeed_ast2500evb_defconfig lib_defconfig.ForceCheckHash
diff --git a/DEVELOPERS b/DEVELOPERS
index a28f3e8d57..f50a92fc56 100644
--- a/DEVELOPERS
+++ b/DEVELOPERS
@@ -2601,8 +2601,6 @@ F: package/systemd/
F: package/tcf-agent/
N: Leo Yu-Chi Liang <ycliang@andestech.com>
-F: board/andes
-F: configs/andes_ae350_45_defconfig
F: package/andes-spi-burn/
F: package/kmon/
diff --git a/board/andes/ae350/genimage_sdcard.cfg b/board/andes/ae350/genimage_sdcard.cfg
deleted file mode 100644
index 58c6f8f156..0000000000
--- a/board/andes/ae350/genimage_sdcard.cfg
+++ /dev/null
@@ -1,26 +0,0 @@
-image boot.vfat {
- vfat {
- files = {
- "u-boot-spl.bin",
- "u-boot.itb",
- "ae350_ax45mp.dtb",
- }
- }
- size = 2M
-}
-
-image sdcard.img {
- hdimage {
- }
-
- partition boot {
- partition-type = 0xC
- image = "boot.vfat"
- }
-
- partition rootfs {
- partition-type = 0x83
- bootable = true
- image = "rootfs.ext4"
- }
-}
diff --git a/board/andes/ae350/patches/uboot/0001-mmc-ftsdc010_mci-Support-DTS-of-ftsdc010-driver-for-.patch b/board/andes/ae350/patches/uboot/0001-mmc-ftsdc010_mci-Support-DTS-of-ftsdc010-driver-for-.patch
deleted file mode 100644
index be89a1563f..0000000000
--- a/board/andes/ae350/patches/uboot/0001-mmc-ftsdc010_mci-Support-DTS-of-ftsdc010-driver-for-.patch
+++ /dev/null
@@ -1,38 +0,0 @@
-From 16aad5594e08550295ea3c12c1c9ed6f64774748 Mon Sep 17 00:00:00 2001
-From: Rick Chen <rick@andestech.com>
-Date: Tue, 29 Mar 2022 13:41:10 +0800
-Subject: [PATCH] mmc: ftsdc010_mci: Support DTS of ftsdc010 driver for
- generic dma
-
-The ftsdc010 driver has been implemented for generic dma in Linux
-kernel. And its compatible is andestech,atfsdc010g to distinguish
-the legacy andestech,atfsdc010 which is not for generic dma.
-
-Althought the ftsdc010_mci driver in U-Boot does not use dma, but
-it still can work well with the mmc node for generic dma. So add
-the compatible string to support it.
-
-Signed-off-by: Rick Chen <rick@andestech.com>
-
-Upstream-Status: Pending
-
-Signed-off-by: Yu Chien Peter Lin <peterlin@andestech.com>
----
- drivers/mmc/ftsdc010_mci.c | 1 +
- 1 file changed, 1 insertion(+)
-
-diff --git a/drivers/mmc/ftsdc010_mci.c b/drivers/mmc/ftsdc010_mci.c
-index 570d54cf9d..65b1d447a8 100644
---- a/drivers/mmc/ftsdc010_mci.c
-+++ b/drivers/mmc/ftsdc010_mci.c
-@@ -460,6 +460,7 @@ int ftsdc010_mmc_bind(struct udevice *dev)
-
- static const struct udevice_id ftsdc010_mmc_ids[] = {
- { .compatible = "andestech,atfsdc010" },
-+ { .compatible = "andestech,atfsdc010g" },
- { }
- };
-
---
-2.34.1
-
diff --git a/board/andes/ae350/patches/uboot/0002-spl-Align-device-tree-blob-address-at-8-byte-boundar.patch b/board/andes/ae350/patches/uboot/0002-spl-Align-device-tree-blob-address-at-8-byte-boundar.patch
deleted file mode 100644
index 5080554ca1..0000000000
--- a/board/andes/ae350/patches/uboot/0002-spl-Align-device-tree-blob-address-at-8-byte-boundar.patch
+++ /dev/null
@@ -1,42 +0,0 @@
-From 933ad8a59f7fd9b2088badc3e97167d750a40b5a Mon Sep 17 00:00:00 2001
-From: Bin Meng <bmeng.cn@gmail.com>
-Date: Mon, 12 Jul 2021 11:52:31 +0800
-Subject: [PATCH] spl: Align device tree blob address at 8-byte boundary
-
-Since libfdt v1.6.1, a new requirement on the device tree address via:
-
- commit 5e735860c478 ("libfdt: Check for 8-byte address alignment in fdt_ro_probe_()")
-
-must be met that the device tree must be loaded in to memory at an
-8-byte aligned address.
-
-Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
-
-This patch was imported from U-boot patchwork:
-https://patchwork.ozlabs.org/project/uboot/patch/20210712035231.26475-1-bmeng.cn@gmail.com/
-
-Signed-off-by: Yu Chien Peter Lin <peterlin@andestech.com>
----
- common/spl/spl_fit.c | 6 ++++++
- 1 file changed, 6 insertions(+)
-
-diff --git a/common/spl/spl_fit.c b/common/spl/spl_fit.c
-index a35be529..a76ad14a 100644
---- a/common/spl/spl_fit.c
-+++ b/common/spl/spl_fit.c
-@@ -382,6 +382,12 @@ static int spl_fit_append_fdt(struct spl_image_info *spl_image,
- */
- image_info.load_addr = spl_image->load_addr + spl_image->size;
-
-+ /*
-+ * Since libfdt v1.6.1, the device tree must be loaded in to memory
-+ * at an 8-byte aligned address.
-+ */
-+ image_info.load_addr = roundup(image_info.load_addr, 8);
-+
- /* Figure out which device tree the board wants to use */
- node = spl_fit_get_image_node(ctx, FIT_FDT_PROP, index++);
- if (node < 0) {
---
-2.34.1
-
diff --git a/board/andes/ae350/post-build.sh b/board/andes/ae350/post-build.sh
deleted file mode 100755
index 84187a064b..0000000000
--- a/board/andes/ae350/post-build.sh
+++ /dev/null
@@ -1,3 +0,0 @@
-#!/bin/sh
-cp $BINARIES_DIR/Image $TARGET_DIR/boot
-cp $BINARIES_DIR/ae350_ax45mp.dtb $TARGET_DIR/boot
diff --git a/board/andes/ae350/readme.txt b/board/andes/ae350/readme.txt
deleted file mode 100644
index 36561fc777..0000000000
--- a/board/andes/ae350/readme.txt
+++ /dev/null
@@ -1,106 +0,0 @@
-Intro
-=====
-
-Andestech AE350 Platform
-
-The AE350 prototype demonstrates the AE350 platform on the FPGA.
-
-How to build it
-===============
-
-Configure Buildroot
--------------------
-
- $ make andes_ae350_45_defconfig
-
-If you want to customize your configuration:
-
- $ make menuconfig
-
-Build everything
-----------------
-Note: you will need to access to the network, since Buildroot will
-download the packages' sources.
-
- $ make
-
-Result of the build
--------------------
-
-After building, you should obtain the following files:
-
- output/images/
- |-- ae350_ax45mp.dtb
- |-- boot.vfat
- |-- fw_dynamic.bin
- |-- fw_dynamic.elf
- |-- Image
- |-- rootfs.ext2
- |-- rootfs.ext4 -> rootfs.ext2
- |-- sdcard.img
- |-- u-boot-spl.bin
- `-- u-boot.itb
-
-How to update the bootloader and device-tree
-============================================
-
-To update the bootloader and device tree, make sure you have
-an ICEman (Andes OpenOCD [1]) and AICE [2] connection set up
-as below:
-
- Local Host Local/Remote Host
- .-----------------. .--------------.
- | buildroot images| | |
- | | ICEman host <IP:PORT>
- | .----------. | | .--------. |
- | | SPI_burn |<---+--socket--+->| ICEman | |
- | '----------' | | '--.-----' |
- '-----------------' '-----|--------'
- |
- USB
- .--------------. |
- | target | .-----v-----.
- | board <----JTAG---| AICE |
- | | '-----------'
- '--------------'
-
-[1] https://github.com/andestech/ICEman
-[2] https://www.andestech.com/en/products-solutions/andeshape-platforms/aice-micro/
-
-The Andes SPI_burn tool will be located in output/host/bin. Use
-the following commands to update the bootloader and device tree:
-
- $ SPI_burn --host $ICE_IP --port $ICE_BURNER_PORT --addr 0x0 -i u-boot-spl.bin
- $ SPI_burn --host $ICE_IP --port $ICE_BURNER_PORT --addr 0x10000 -i u-boot.itb
- $ SPI_burn --host $ICE_IP --port $ICE_BURNER_PORT --addr 0xf0000 -i ae350_ax45mp.dtb
-
-Note that the --addr option specifies the offset starting from
-the flash base address 0x80000000 and set by U-Boot configurations.
-e.g.
-u-boot-spl.bin : CONFIG_SPL_TEXT_BASE=0x80000000
-u-boot.itb : CONFIG_SPL_LOAD_FIT_ADDRESS=0x80010000
-ae350_ax45mp.dtb: CONFIG_SYS_FDT_BASE=0x800f0000
-
-How to write the SD card
-========================
-
-Copy the sdcard.img to a SD card with "dd":
-
- $ sudo dd if=sdcard.img of=/dev/sdX bs=4096
- $ sudo sync
-
-Your SD card partition should be:
-
- Disk /dev/sdb: 14.48 GiB, 15552479232 bytes, 30375936 sectors
- Disk model: Multi-Card
- Units: sectors of 1 * 512 = 512 bytes
- Sector size (logical/physical): 512 bytes / 512 bytes
- I/O size (minimum/optimal): 512 bytes / 512 bytes
- Disklabel type: dos
- Disk identifier: 0x00000000
-
- Device Boot Start End Sectors Size Id Type
- /dev/sdb1 1 4096 4096 2M c W95 FAT32 (LBA)
- /dev/sdb2 * 4097 126976 122880 60M 83 Linux
-
-Insert SD card and reset the board, it should boot Linux from mmc.
diff --git a/board/andes/ae350/rootfs_overlay/boot/extlinux/extlinux.conf b/board/andes/ae350/rootfs_overlay/boot/extlinux/extlinux.conf
deleted file mode 100644
index be19e196a8..0000000000
--- a/board/andes/ae350/rootfs_overlay/boot/extlinux/extlinux.conf
+++ /dev/null
@@ -1,4 +0,0 @@
-label linux
- kernel /boot/Image
- fdt /boot/ae350_ax45mp.dtb
- append earlycon=sbi root=/dev/mmcblk0p2 rootwait
diff --git a/board/andes/ae350/uboot.config.fragment b/board/andes/ae350/uboot.config.fragment
deleted file mode 100644
index 4497ed4c74..0000000000
--- a/board/andes/ae350/uboot.config.fragment
+++ /dev/null
@@ -1,6 +0,0 @@
-CONFIG_SPL_FS_FAT=y
-CONFIG_SPL_MMC=y
-CONFIG_SPL_OPENSBI_LOAD_ADDR=0x0
-CONFIG_SPL_OPENSBI_SCRATCH_OPTIONS=0x0
-CONFIG_DISPLAY_CPUINFO=y
-CONFIG_DISPLAY_BOARDINFO=y
diff --git a/configs/andes_ae350_45_defconfig b/configs/andes_ae350_45_defconfig
deleted file mode 100644
index 7b5a6f73c6..0000000000
--- a/configs/andes_ae350_45_defconfig
+++ /dev/null
@@ -1,40 +0,0 @@
-BR2_riscv=y
-BR2_RISCV_ISA_RVC=y
-BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_6_0=y
-BR2_GLOBAL_PATCH_DIR="board/andes/ae350/patches"
-BR2_TARGET_GENERIC_GETTY_PORT="ttyS0"
-BR2_ROOTFS_OVERLAY="board/andes/ae350/rootfs_overlay"
-BR2_ROOTFS_POST_BUILD_SCRIPT="board/andes/ae350/post-build.sh"
-BR2_ROOTFS_POST_IMAGE_SCRIPT="support/scripts/genimage.sh"
-BR2_ROOTFS_POST_SCRIPT_ARGS="-c board/andes/ae350/genimage_sdcard.cfg"
-BR2_LINUX_KERNEL=y
-BR2_LINUX_KERNEL_CUSTOM_TARBALL=y
-BR2_LINUX_KERNEL_CUSTOM_TARBALL_LOCATION="$(call github,andestech,linux,v6.0.y_ae350-ax45mp)/linux-v6.0.y_ae350-ax45mp.tar.gz"
-BR2_LINUX_KERNEL_DEFCONFIG="ae350_ax45mp"
-BR2_LINUX_KERNEL_DTS_SUPPORT=y
-BR2_LINUX_KERNEL_INTREE_DTS_NAME="andes/ae350_ax45mp"
-BR2_TARGET_ROOTFS_EXT2=y
-BR2_TARGET_ROOTFS_EXT2_4=y
-# BR2_TARGET_ROOTFS_TAR is not set
-BR2_TARGET_OPENSBI=y
-BR2_TARGET_OPENSBI_CUSTOM_TARBALL=y
-BR2_TARGET_OPENSBI_CUSTOM_TARBALL_LOCATION="$(call github,riscv,opensbi,22f38ee6c658a660083aa45c4ec6c72f66a17260)/opensbi-22f38ee6c658a660083aa45c4ec6c72f66a17260.tar.gz"
-BR2_TARGET_OPENSBI_PLAT="andes/ae350"
-# BR2_TARGET_OPENSBI_INSTALL_JUMP_IMG is not set
-BR2_TARGET_UBOOT=y
-BR2_TARGET_UBOOT_BUILD_SYSTEM_KCONFIG=y
-BR2_TARGET_UBOOT_CUSTOM_TARBALL=y
-BR2_TARGET_UBOOT_CUSTOM_TARBALL_LOCATION="$(call github,u-boot,u-boot,a5dfa3b8a0f7ad555495bad1386613d2de4ba619)/u-boot-a5dfa3b8a0f7ad555495bad1386613d2de4ba619.tar.gz"
-BR2_TARGET_UBOOT_BOARD_DEFCONFIG="ae350_rv64_spl_xip"
-BR2_TARGET_UBOOT_CONFIG_FRAGMENT_FILES="board/andes/ae350/uboot.config.fragment"
-BR2_TARGET_UBOOT_NEEDS_PYLIBFDT=y
-BR2_TARGET_UBOOT_NEEDS_OPENSSL=y
-BR2_TARGET_UBOOT_NEEDS_OPENSBI=y
-# BR2_TARGET_UBOOT_FORMAT_BIN is not set
-BR2_TARGET_UBOOT_FORMAT_CUSTOM=y
-BR2_TARGET_UBOOT_FORMAT_CUSTOM_NAME="u-boot.itb"
-BR2_TARGET_UBOOT_SPL=y
-BR2_PACKAGE_HOST_ANDES_SPI_BURN=y
-BR2_PACKAGE_HOST_DOSFSTOOLS=y
-BR2_PACKAGE_HOST_GENIMAGE=y
-BR2_PACKAGE_HOST_MTOOLS=y
--
2.50.1
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [Buildroot] [PATCH] configs/andes_ae350_45: remove defconfig, broken
2025-08-18 20:32 [Buildroot] [PATCH] configs/andes_ae350_45: remove defconfig, broken Thomas Petazzoni via buildroot
@ 2025-08-19 9:39 ` Leo Liang via buildroot
2025-08-19 9:54 ` Thomas Petazzoni via buildroot
0 siblings, 1 reply; 5+ messages in thread
From: Leo Liang via buildroot @ 2025-08-19 9:39 UTC (permalink / raw)
To: Thomas Petazzoni; +Cc: buildroot
On Mon, Aug 18, 2025 at 10:32:22PM +0200, Thomas Petazzoni wrote:
> [EXTERNAL MAIL]
>
> This configuration doesn't build anymore as its kernel version has
> been removed from https://github.com/andestech/linux/.
>
> This has been failing for over 2 months without anyone bothering to
> fix it, so let's drop this defconfig.
>
> Fixes:
>
> https://gitlab.com/buildroot.org/buildroot/-/jobs/11012272348
>
> Cc: Leo Yu-Chi Liang <ycliang@andestech.com>
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
> ---
> .checkpackageignore | 4 -
> DEVELOPERS | 2 -
> board/andes/ae350/genimage_sdcard.cfg | 26 -----
> ...-Support-DTS-of-ftsdc010-driver-for-.patch | 38 -------
> ...-tree-blob-address-at-8-byte-boundar.patch | 42 -------
> board/andes/ae350/post-build.sh | 3 -
> board/andes/ae350/readme.txt | 106 ------------------
> .../boot/extlinux/extlinux.conf | 4 -
> board/andes/ae350/uboot.config.fragment | 6 -
> configs/andes_ae350_45_defconfig | 40 -------
> 10 files changed, 271 deletions(-)
> delete mode 100644 board/andes/ae350/genimage_sdcard.cfg
> delete mode 100644 board/andes/ae350/patches/uboot/0001-mmc-ftsdc010_mci-Support-DTS-of-ftsdc010-driver-for-.patch
> delete mode 100644 board/andes/ae350/patches/uboot/0002-spl-Align-device-tree-blob-address-at-8-byte-boundar.patch
> delete mode 100755 board/andes/ae350/post-build.sh
> delete mode 100644 board/andes/ae350/readme.txt
> delete mode 100644 board/andes/ae350/rootfs_overlay/boot/extlinux/extlinux.conf
> delete mode 100644 board/andes/ae350/uboot.config.fragment
> delete mode 100644 configs/andes_ae350_45_defconfig
Hi Thomas,
My apologies for not catching the CI failure earlier;
my email filter was misconfigured.
I've fixed the Linux tarball link and confirmed that the defconfig builds.
Would it be possible to drop this patch?
Best regards,
Leo
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Buildroot] [PATCH] configs/andes_ae350_45: remove defconfig, broken
2025-08-19 9:39 ` Leo Liang via buildroot
@ 2025-08-19 9:54 ` Thomas Petazzoni via buildroot
2025-08-19 10:17 ` Leo Liang via buildroot
0 siblings, 1 reply; 5+ messages in thread
From: Thomas Petazzoni via buildroot @ 2025-08-19 9:54 UTC (permalink / raw)
To: Leo Liang; +Cc: buildroot
Hello Leo,
On Tue, 19 Aug 2025 17:39:09 +0800
Leo Liang <ycliang@andestech.com> wrote:
> My apologies for not catching the CI failure earlier;
> my email filter was misconfigured.
OK.
> I've fixed the Linux tarball link and confirmed that the defconfig builds.
When you say you "fixed the Linux tarball", you mean you re-added the
missing branch to your Github repo?
Note that the defconfig is apparently using a branch, which is not
correct. It should either use a tag or a commit hash. Indeed, branches
can change over time, while our defconfigs should be reproducible.
> Would it be possible to drop this patch?
I'll drop the patch, but aside from the branch vs. tag issue above,
there are other issues to fix in your defconfig:
Patches lack an "Upstream" tag:
board/andes/ae350/patches/uboot/0001-mmc-ftsdc010_mci-Support-DTS-of-ftsdc010-driver-for-.patch lib_patch.Upstream
board/andes/ae350/patches/uboot/0002-spl-Align-device-tree-blob-address-at-8-byte-boundar.patch lib_patch.Upstream
The post-build shell script doesn't pass Shellcheck:
board/andes/ae350/post-build.sh Shellcheck
Hashes are missing for the defconfig:
configs/andes_ae350_45_defconfig lib_defconfig.ForceCheckHash
Could you please address all those issues to keep the defconfig in a
properly maintained state?
Thanks a lot!
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
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Buildroot] [PATCH] configs/andes_ae350_45: remove defconfig, broken
2025-08-19 9:54 ` Thomas Petazzoni via buildroot
@ 2025-08-19 10:17 ` Leo Liang via buildroot
2025-08-19 17:08 ` Thomas Petazzoni via buildroot
0 siblings, 1 reply; 5+ messages in thread
From: Leo Liang via buildroot @ 2025-08-19 10:17 UTC (permalink / raw)
To: Thomas Petazzoni; +Cc: buildroot
Hi Thomas,
On Tue, Aug 19, 2025 at 11:54:59AM +0200, Thomas Petazzoni wrote:
> [EXTERNAL MAIL]
>
> Hello Leo,
>
> On Tue, 19 Aug 2025 17:39:09 +0800
> Leo Liang <ycliang@andestech.com> wrote:
>
> > My apologies for not catching the CI failure earlier;
> > my email filter was misconfigured.
>
> OK.
>
> > I've fixed the Linux tarball link and confirmed that the defconfig builds.
>
> When you say you "fixed the Linux tarball", you mean you re-added the
> missing branch to your Github repo?
Yes, I have re-added the missing branch.
>
> Note that the defconfig is apparently using a branch, which is not
> correct. It should either use a tag or a commit hash. Indeed, branches
> can change over time, while our defconfigs should be reproducible.
Got it!
>
> > Would it be possible to drop this patch?
>
> I'll drop the patch, but aside from the branch vs. tag issue above,
> there are other issues to fix in your defconfig:
>
> Patches lack an "Upstream" tag:
>
> board/andes/ae350/patches/uboot/0001-mmc-ftsdc010_mci-Support-DTS-of-ftsdc010-driver-for-.patch lib_patch.Upstream
> board/andes/ae350/patches/uboot/0002-spl-Align-device-tree-blob-address-at-8-byte-boundar.patch lib_patch.Upstream
>
> The post-build shell script doesn't pass Shellcheck:
>
> board/andes/ae350/post-build.sh Shellcheck
>
> Hashes are missing for the defconfig:
>
> configs/andes_ae350_45_defconfig lib_defconfig.ForceCheckHash
>
> Could you please address all those issues to keep the defconfig in a
> properly maintained state?
Roger that!
I will prepare a patchset ASAP to address these issues.
Thank you for all the reminders!
> Thanks a lot!
>
> Thomas
> --
> Thomas Petazzoni, co-owner and CEO, Bootlin
> Embedded Linux and Kernel engineering and training
> https://bootlin.com
Thanks again,
Leo
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Buildroot] [PATCH] configs/andes_ae350_45: remove defconfig, broken
2025-08-19 10:17 ` Leo Liang via buildroot
@ 2025-08-19 17:08 ` Thomas Petazzoni via buildroot
0 siblings, 0 replies; 5+ messages in thread
From: Thomas Petazzoni via buildroot @ 2025-08-19 17:08 UTC (permalink / raw)
To: Leo Liang; +Cc: buildroot
Hello Leo,
On Tue, 19 Aug 2025 18:17:03 +0800
Leo Liang <ycliang@andestech.com> wrote:
> Roger that!
> I will prepare a patchset ASAP to address these issues.
> Thank you for all the reminders!
Thanks, looking forward to seeing your patches!
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
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2025-08-19 17:08 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-08-18 20:32 [Buildroot] [PATCH] configs/andes_ae350_45: remove defconfig, broken Thomas Petazzoni via buildroot
2025-08-19 9:39 ` Leo Liang via buildroot
2025-08-19 9:54 ` Thomas Petazzoni via buildroot
2025-08-19 10:17 ` Leo Liang via buildroot
2025-08-19 17:08 ` Thomas Petazzoni via buildroot
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.