Re: [PATCH] i2c: Remove I2C_COMPAT config symbol and related code

[Heiner Kallweit <hkallweit1@gmail.com>]

On 01.09.2024 11:06, Wolfram Sang wrote:
> Hi Heiner,
> 
> On Wed, Aug 21, 2024 at 10:13:04PM +0200, Heiner Kallweit wrote:
>> This code was added with 2bb5095affdb ("i2c: Provide compatibility links
>> for i2c adapters"). Commit message stated: Provide compatibility links
>> for [...] the time being. We will remove them after a long transition
>> period.
>> 15 years should have been a long enough transition period.
> 
> Well, in general, I totally agree.
> 
> It seems, however, that this slipped through the cracks. My Debian
> kernelconfig has I2C_COMPAT still enabled, so I am reluctant to remove
> it from one kernel release to the next.
> 
One reason may be that the default for I2C_COMPAT is still y.
We should switch this to n, even though it doesn't help in cases like
make oldconfig.

> I wonder if we need some printout that it is really going away for a few
> kernel releases.
> 
This may be a compromise.
However I wonder how likely it is that somebody uses the latest kernel
and hasn't updated his userspace tools for > 15 yrs.

> Opinions? Jean?
> 
> Happy hacking,
> 
>    Wolfram
> 
Heiner