Re: [PATCH] Use Kconfig flag to remove support of deprecated BE2/BE3 adapters

From: Ivan Vecera
Date: Tue Aug 07 2018 - 07:44:57 EST


On 6.8.2018 22:46, David Miller wrote:
> From: Petr Oros <poros@xxxxxxxxxx>
> Date: Mon, 6 Aug 2018 14:12:28 +0200
>
>> Add flags to remove support of deprecated BE2/BE3 adapters.
>> BE2 disable will reduce .ko size by 2kb and BE3 by 3kb.
>> Disable both will reduce .ko size by 9kb.
>>
>> With dissabled support is also removed coresponding PCI IDs
>> and codepath with [BE2|BE3|BEx]_chip checks.
>>
>> New help style in Kconfig
>>
>> Signed-off-by: Petr Oros <poros@xxxxxxxxxx>
>
> Sorry, I'm not too hot about this.

Why? This patch does not remove support for older chips, it just gives an
ability to disable support for certain chip families. In other words an user is
able to customize the driver for his/her needs - it could be fine to add this
ability also for the rest of chip families.

> Imagine being one of the people who has one of these cards.

Such people leave this configs as they are (enabled by default).

> Pulling out detection and working'ness of devices from a driver
> is a big step backwards, and I'm sorry I will don't want to be
> part of something that facilitates this.
>
> The S390 folks tried something similar in the past and I reject
> those changes too.

This patch is practically the same as "a1b8714593b6 ("net/mlx4: Use Kconfig flag
to remove support of old gen2 Mellanox devices")" for mlx4 and that was accepted
without any objections.

Thanks,
Ivan