Re: [PATCH 4/4] eeprom: at24: Add OF device ID table

From	Javier Martinez Canillas <javier@osg.samsung.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH 4/4] eeprom: at24: Add OF device ID table
Date	2017-03-15 12:10 +0100
Message-ID	<tleYy-6NF-9@gated-at.bofh.it> (permalink)
References	<tkWyB-20n-5@gated-at.bofh.it> <tkWyD-20n-51@gated-at.bofh.it> <tl3JM-7h6-11@gated-at.bofh.it> <tl4Pv-7ZV-7@gated-at.bofh.it> <tlc0F-4wj-1@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Hello Wolfram,

On 03/15/2017 04:58 AM, Wolfram Sang wrote:
> 
>> So there isn't an agreement if is better to just rely in the current behavior
>> (and have a superfluous I2C device ID table) or fix the I2C core (and need a
>> OF device ID table).
> 
> For at24, the i2c_device_id table is not superfluous! It is used outside
> the DT world as well.
>

Yes, I know. I was trying to explain to Andy what's the problem that I want to
solve in general, not talking about this particular driver. Sorry if that was
confusing.

>> Indeed, but these all are compatible strings used by DTS in mainline and so
>> should be in the OF device ID table in order to be matched and the proper
>> modalias reported (once the I2C core is fixed).
> 
> I'd think we should fix the DTS files instead to contain a fallback we
> agree on. Say, we agree on "atmel,at24c01" as a the generic fallback,
> the DTS should contain:
> 
> 	compatible = "<your_vendor>,<your_type>", "atmel,at24c01"
> 
> And we shall only keep compatible values in the source file which differ
> in behaviour fromt the generic case.
>

Do you know who that's familiar with this device and driver can do this? I've
been trying to fix all the drivers that are relying on having an I2C ID table
but whose devices are registered via DT and I've now posted patches for all.

I wanted to do that so this patch can finally land [0] but to be honest I'm
about to give up on this... it seems is causing a lot of churn to maintainers
and many don't see the benefit on having the I2C core to report a proper OF
modalias, and don't think the current behavior is particularly bad.

Unfortunately some maintainers do and don't accept patches adding I2C tables
only to have module autoloading working so I still think it should be fixed.

[0]: https://patchwork.kernel.org/patch/6903991/

Best regards,
-- 
Javier Martinez Canillas
Open Source Group
Samsung Research America

Thread

[PATCH 0/4] misc: Add OF device table to I2C drivers that are missing it Javier Martinez Canillas <javier@osg.samsung.com> - 2017-03-14 16:30 +0100
  [PATCH 1/4] misc: tsl2550: Add OF device ID table Javier Martinez Canillas <javier@osg.samsung.com> - 2017-03-14 16:30 +0100
    Re: [PATCH 1/4] misc: tsl2550: Add OF device ID table Arnd Bergmann <arnd@arndb.de> - 2017-03-14 22:00 +0100
  [PATCH 3/4] eeprom: idt_89hpesx: Add OF device ID table Javier Martinez Canillas <javier@osg.samsung.com> - 2017-03-14 16:30 +0100
  [PATCH 2/4] misc: ds1682: Add OF device ID table Javier Martinez Canillas <javier@osg.samsung.com> - 2017-03-14 16:30 +0100
    Re: [PATCH 2/4] misc: ds1682: Add OF device ID table Arnd Bergmann <arnd@arndb.de> - 2017-03-14 21:50 +0100
      Re: [PATCH 2/4] misc: ds1682: Add OF device ID table Javier Martinez Canillas <javier@osg.samsung.com> - 2017-03-15 01:40 +0100
        Re: [PATCH 2/4] misc: ds1682: Add OF device ID table Javier Martinez Canillas <javier@osg.samsung.com> - 2017-03-15 04:30 +0100
  [PATCH 4/4] eeprom: at24: Add OF device ID table Javier Martinez Canillas <javier@osg.samsung.com> - 2017-03-14 16:30 +0100
    Re: [PATCH 4/4] eeprom: at24: Add OF device ID table Andy Shevchenko <andy.shevchenko@gmail.com> - 2017-03-15 00:10 +0100
      Re: [PATCH 4/4] eeprom: at24: Add OF device ID table Javier Martinez Canillas <javier@osg.samsung.com> - 2017-03-15 01:20 +0100
        Re: [PATCH 4/4] eeprom: at24: Add OF device ID table Wolfram Sang <wsa@the-dreams.de> - 2017-03-15 09:00 +0100
          Re: [PATCH 4/4] eeprom: at24: Add OF device ID table Javier Martinez Canillas <javier@osg.samsung.com> - 2017-03-15 12:10 +0100
            Re: [PATCH 4/4] eeprom: at24: Add OF device ID table Andy Shevchenko <andy.shevchenko@gmail.com> - 2017-03-15 12:30 +0100
              Re: [PATCH 4/4] eeprom: at24: Add OF device ID table Javier Martinez Canillas <javier@osg.samsung.com> - 2017-03-15 12:50 +0100
                Re: [PATCH 4/4] eeprom: at24: Add OF device ID table Andy Shevchenko <andy.shevchenko@gmail.com> - 2017-03-15 23:50 +0100
                Re: [PATCH 4/4] eeprom: at24: Add OF device ID table Javier Martinez Canillas <javier@osg.samsung.com> - 2017-03-16 13:30 +0100
            Re: [PATCH 4/4] eeprom: at24: Add OF device ID table Wolfram Sang <wsa@the-dreams.de> - 2017-03-16 14:10 +0100
              Re: [PATCH 4/4] eeprom: at24: Add OF device ID table Javier Martinez Canillas <javier@osg.samsung.com> - 2017-03-16 14:20 +0100
                Re: [PATCH 4/4] eeprom: at24: Add OF device ID table Wolfram Sang <wsa@the-dreams.de> - 2017-03-16 14:40 +0100
                Re: [PATCH 4/4] eeprom: at24: Add OF device ID table Javier Martinez Canillas <javier@osg.samsung.com> - 2017-03-16 15:10 +0100
                Re: [PATCH 4/4] eeprom: at24: Add OF device ID table Wolfram Sang <wsa@the-dreams.de> - 2017-03-16 16:10 +0100
                Re: [PATCH 4/4] eeprom: at24: Add OF device ID table Javier Martinez Canillas <javier@osg.samsung.com> - 2017-03-16 16:40 +0100
                Re: [PATCH 4/4] eeprom: at24: Add OF device ID table Javier Martinez Canillas <javier@osg.samsung.com> - 2017-03-20 17:50 +0100

csiph-web