Re: [PATCH] arm64: defconfig: Enable zram, xfs and loading compressed FW support

[Javier Martinez Canillas]

Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> writes:

> On 21/02/2024 16:31, Arnd Bergmann wrote:
>> On Wed, Feb 21, 2024, at 13:59, Javier Martinez Canillas wrote:
>>> These options are needed by some Linux distributions (e.g: Fedora), so
>>> let's enable them to make it easier for developers using such distros.
>>>
>>> Signed-off-by: Javier Martinez Canillas <javierm@xxxxxxxxxx>
>>> ---
>> 
>> Acked-by: Arnd Bergmann <arnd@xxxxxxxx>
>> 
>>> @@ -1595,6 +1599,7 @@ CONFIG_HTE_TEGRA194_TEST=m
>>>  CONFIG_EXT2_FS=y
>>>  CONFIG_EXT3_FS=y
>>>  CONFIG_EXT4_FS_POSIX_ACL=y
>>> +CONFIG_XFS_FS=m
>>>  CONFIG_BTRFS_FS=m
>>>  CONFIG_BTRFS_FS_POSIX_ACL=y
>> 
>> Unfortunately this will increase build time noticeably, but
>> I agree it is the right thing to do.
>> 
>> Can you send it (with any other acks) to soc@xxxxxxxxxx so
>> I can merge it from there?
>
> Due to XFS, please include:
>
> Nacked-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
>

Don't worry, I won't to re-send anything if you disagree with the change.

> so this will not be precedence used for further patches like this
> (Maxime already used such argument, so I want to be sure it will not be
> used against me in the future)
>

I used the same argument and is a fair one in my opinion. Removing other
filesystems should send the correct message, if that is the agreegment.

-- 
Best regards,

Javier Martinez Canillas
Core Platforms
Red Hat