From: "Vincent Stehlé" <vincent.stehle@arm.com>
To: Niklas Cassel <niklas.cassel@wdc.com>
Cc: Romain Naour <romain.naour@gmail.com>,
Damien Le Moal <damien.lemoal@opensource.wdc.com>,
buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH 3/3] board/qemu/aarch64-sbsa: drop unused kernel config file
Date: Thu, 2 Mar 2023 14:12:45 +0100 [thread overview]
Message-ID: <ZACgzebwfOVNcfdC@debian> (raw)
In-Reply-To: <20230301155810.3222459-4-niklas.cassel@wdc.com>
On Wed, Mar 01, 2023 at 04:58:10PM +0100, Niklas Cassel wrote:
> The config file board/qemu/aarch64-sbsa/linux.config has
> never been in use by configs/qemu_aarch64_sbsa_defconfig,
> neither via BR2_LINUX_KERNEL_CUSTOM_CONFIG_FILE, nor via
> BR2_LINUX_KERNEL_CONFIG_FRAGMENT_FILES.
>
> Thus the config file is and has always been unused.
Hi Niklas,
This fragment seems to be used by edk2 python unit test
`support/testing/tests/boot/test_edk2.py'.
I missed that during my testing of the defconfigs.
Best regards,
Vincent.
> Remove it such that no one might get the silly idea that
> it is actually used for something.
>
> Signed-off-by: Niklas Cassel <niklas.cassel@wdc.com>
> ---
> board/qemu/aarch64-sbsa/linux.config | 68 ----------------------------
> 1 file changed, 68 deletions(-)
> delete mode 100644 board/qemu/aarch64-sbsa/linux.config
>
> diff --git a/board/qemu/aarch64-sbsa/linux.config b/board/qemu/aarch64-sbsa/linux.config
> deleted file mode 100644
> index a60cce02ec..0000000000
> --- a/board/qemu/aarch64-sbsa/linux.config
> +++ /dev/null
> @@ -1,68 +0,0 @@
> -CONFIG_SYSVIPC=y
> -CONFIG_POSIX_MQUEUE=y
> -CONFIG_NO_HZ_IDLE=y
> -CONFIG_HIGH_RES_TIMERS=y
> -CONFIG_TASKSTATS=y
> -CONFIG_SCHED_AUTOGROUP=y
> -CONFIG_PROFILING=y
> -CONFIG_ARCH_VEXPRESS=y
> -CONFIG_COMPAT=y
> -CONFIG_FW_CFG_SYSFS=y
> -CONFIG_FW_CFG_SYSFS_CMDLINE=y
> -CONFIG_ACPI=y
> -CONFIG_MODULES=y
> -CONFIG_MODULE_UNLOAD=y
> -CONFIG_BLK_DEV_BSGLIB=y
> -CONFIG_BINFMT_MISC=y
> -CONFIG_TRANSPARENT_HUGEPAGE=y
> -CONFIG_NET=y
> -CONFIG_PACKET=y
> -CONFIG_PACKET_DIAG=y
> -CONFIG_UNIX=y
> -CONFIG_NET_KEY=y
> -CONFIG_INET=y
> -CONFIG_IP_MULTICAST=y
> -CONFIG_IP_ADVANCED_ROUTER=y
> -CONFIG_BRIDGE=m
> -CONFIG_NET_SCHED=y
> -CONFIG_VSOCKETS=y
> -CONFIG_PCI=y
> -CONFIG_PCI_HOST_GENERIC=y
> -CONFIG_DEVTMPFS=y
> -CONFIG_DEVTMPFS_MOUNT=y
> -CONFIG_VIRTIO_BLK=y
> -CONFIG_BLK_DEV_SD=y
> -CONFIG_CHR_DEV_SG=y
> -CONFIG_SCSI_CONSTANTS=y
> -CONFIG_SCSI_LOGGING=y
> -CONFIG_SCSI_SCAN_ASYNC=y
> -CONFIG_SCSI_VIRTIO=y
> -CONFIG_ATA=y
> -CONFIG_SATA_AHCI_PLATFORM=y
> -CONFIG_NETDEVICES=y
> -CONFIG_DUMMY=y
> -CONFIG_MACVLAN=y
> -CONFIG_VIRTIO_NET=y
> -CONFIG_NLMON=y
> -CONFIG_SERIAL_AMBA_PL011=y
> -CONFIG_SERIAL_AMBA_PL011_CONSOLE=y
> -CONFIG_VIRTIO_CONSOLE=y
> -CONFIG_HW_RANDOM=y
> -CONFIG_HW_RANDOM_VIRTIO=y
> -CONFIG_TCG_TPM=y
> -CONFIG_TCG_TIS=y
> -CONFIG_DRM=y
> -CONFIG_DRM_VIRTIO_GPU=y
> -CONFIG_RTC_CLASS=y
> -CONFIG_RTC_DRV_PL031=y
> -CONFIG_VIRTIO_PCI=y
> -CONFIG_VIRTIO_MMIO=y
> -CONFIG_VIRTIO_MMIO_CMDLINE_DEVICES=y
> -CONFIG_MAILBOX=y
> -CONFIG_PL320_MBOX=y
> -CONFIG_ARM_SMMU_V3=y
> -CONFIG_EXT4_FS=y
> -CONFIG_FUSE_FS=y
> -CONFIG_VIRTIO_FS=y
> -CONFIG_TMPFS=y
> -CONFIG_TMPFS_POSIX_ACL=y
> --
> 2.39.2
>
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
next prev parent reply other threads:[~2023-03-02 13:12 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-03-01 15:58 [Buildroot] [PATCH 0/3] misc aarch64-sbsa cleanups Niklas Cassel via buildroot
2023-03-01 15:58 ` [Buildroot] [PATCH 1/3] board/qemu/aarch64-sbsa: use PARTLABEL to specify root Niklas Cassel via buildroot
2023-03-01 15:58 ` [Buildroot] [PATCH 2/3] board/qemu/aarch64-sbsa: use the default console specified by ACPI Niklas Cassel via buildroot
2023-03-01 15:58 ` [Buildroot] [PATCH 3/3] board/qemu/aarch64-sbsa: drop unused kernel config file Niklas Cassel via buildroot
2023-03-02 13:12 ` Vincent Stehlé [this message]
2023-03-02 12:59 ` [Buildroot] [PATCH 0/3] misc aarch64-sbsa cleanups Vincent Stehlé
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ZACgzebwfOVNcfdC@debian \
--to=vincent.stehle@arm.com \
--cc=buildroot@buildroot.org \
--cc=damien.lemoal@opensource.wdc.com \
--cc=niklas.cassel@wdc.com \
--cc=romain.naour@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.