From: Richard Cheng <icheng@nvidia•com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@oss•qualcomm.com>
Cc: Arnd Bergmann <arnd@arndb•de>,
soc@lists•linux.dev, Russell King <linux@armlinux•org.uk>,
Theodore Ts'o <tytso@mit•edu>,
linux-arm-kernel@lists•infradead.org,
linux-kernel@vger•kernel.org
Subject: Re: [PATCH] ARM: configs: Drop duplicated CONFIG_EXT4_FS
Date: Wed, 3 Jun 2026 16:59:30 +0800 [thread overview]
Message-ID: <ah_sf1lIBD8UcOHs@MWDK4CY14F> (raw)
In-Reply-To: <20260603072726.19404-2-krzysztof.kozlowski@oss.qualcomm.com>
On Wed, Jun 03, 2026 at 09:27:27AM +0800, Krzysztof Kozlowski wrote:
> Remove redundant, duplicated CONFIG_EXT4_FS to fix warnings like:
>
> axm55xx_defconfig:198:warning: override: reassigning to symbol EXT4_FS
>
> Fixes: c065b6046b34 ("Use CONFIG_EXT4_FS instead of CONFIG_EXT3_FS in all of the defconfigs")
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss•qualcomm.com>
> ---
> arch/arm/configs/axm55xx_defconfig | 1 -
> arch/arm/configs/dove_defconfig | 1 -
> arch/arm/configs/ep93xx_defconfig | 1 -
> arch/arm/configs/mmp2_defconfig | 1 -
> arch/arm/configs/mv78xx0_defconfig | 1 -
> 5 files changed, 5 deletions(-)
>
> diff --git a/arch/arm/configs/axm55xx_defconfig b/arch/arm/configs/axm55xx_defconfig
> index 22b189090e15..0952e5e94c5e 100644
> --- a/arch/arm/configs/axm55xx_defconfig
> +++ b/arch/arm/configs/axm55xx_defconfig
> @@ -195,7 +195,6 @@ CONFIG_PL320_MBOX=y
> # CONFIG_IOMMU_SUPPORT is not set
> CONFIG_EXT2_FS=y
> CONFIG_EXT4_FS=y
> -CONFIG_EXT4_FS=y
> CONFIG_AUTOFS_FS=y
> CONFIG_FUSE_FS=y
> CONFIG_CUSE=y
> diff --git a/arch/arm/configs/dove_defconfig b/arch/arm/configs/dove_defconfig
> index e98c35df675e..3d978ce34b54 100644
> --- a/arch/arm/configs/dove_defconfig
> +++ b/arch/arm/configs/dove_defconfig
> @@ -97,7 +97,6 @@ CONFIG_MV_XOR=y
> CONFIG_EXT2_FS=y
> CONFIG_EXT4_FS=y
> # CONFIG_EXT4_FS_XATTR is not set
> -CONFIG_EXT4_FS=y
> CONFIG_ISO9660_FS=y
> CONFIG_JOLIET=y
> CONFIG_UDF_FS=m
> diff --git a/arch/arm/configs/ep93xx_defconfig b/arch/arm/configs/ep93xx_defconfig
> index 9f3c7324d1cf..ce41dc8c435c 100644
> --- a/arch/arm/configs/ep93xx_defconfig
> +++ b/arch/arm/configs/ep93xx_defconfig
> @@ -105,7 +105,6 @@ CONFIG_EP93XX_DMA=y
> CONFIG_EXT2_FS=y
> CONFIG_EXT4_FS=y
> # CONFIG_EXT4_FS_XATTR is not set
> -CONFIG_EXT4_FS=y
> CONFIG_VFAT_FS=y
> CONFIG_TMPFS=y
> CONFIG_JFFS2_FS=y
> diff --git a/arch/arm/configs/mmp2_defconfig b/arch/arm/configs/mmp2_defconfig
> index a9a212abfd69..0ea608c75f22 100644
> --- a/arch/arm/configs/mmp2_defconfig
> +++ b/arch/arm/configs/mmp2_defconfig
> @@ -54,7 +54,6 @@ CONFIG_RTC_DRV_MAX8925=y
> # CONFIG_RESET_CONTROLLER is not set
> CONFIG_EXT2_FS=y
> CONFIG_EXT4_FS=y
> -CONFIG_EXT4_FS=y
> # CONFIG_DNOTIFY is not set
> CONFIG_MSDOS_FS=y
> CONFIG_FAT_DEFAULT_CODEPAGE=437
> diff --git a/arch/arm/configs/mv78xx0_defconfig b/arch/arm/configs/mv78xx0_defconfig
> index d3a26efe766c..c6b46141cb69 100644
> --- a/arch/arm/configs/mv78xx0_defconfig
> +++ b/arch/arm/configs/mv78xx0_defconfig
> @@ -93,7 +93,6 @@ CONFIG_RTC_DRV_M41T80=y
> CONFIG_EXT2_FS=y
> CONFIG_EXT4_FS=y
> # CONFIG_EXT4_FS_XATTR is not set
> -CONFIG_EXT4_FS=m
> CONFIG_ISO9660_FS=m
> CONFIG_JOLIET=y
> CONFIG_UDF_FS=m
> --
> 2.53.0
>
>
Hi Krzysztof,
LGTM.
Though I wonder which compiler did you use and how did you encounter
the warning? on what system if you may share with us?
Reviewed-by: Richard Cheng <icheng@nvidia•com>
Best regards,
Richard Cheng.
next prev parent reply other threads:[~2026-06-03 8:59 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-06-03 7:27 [PATCH] ARM: configs: Drop duplicated CONFIG_EXT4_FS Krzysztof Kozlowski
2026-06-03 8:59 ` Richard Cheng [this message]
2026-06-03 9:12 ` Krzysztof Kozlowski
2026-06-03 9:47 ` Richard Cheng
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=ah_sf1lIBD8UcOHs@MWDK4CY14F \
--to=icheng@nvidia$(echo .)com \
--cc=arnd@arndb$(echo .)de \
--cc=krzysztof.kozlowski@oss$(echo .)qualcomm.com \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
--cc=linux-kernel@vger$(echo .)kernel.org \
--cc=linux@armlinux$(echo .)org.uk \
--cc=soc@lists$(echo .)linux.dev \
--cc=tytso@mit$(echo .)edu \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox