DT support in i2c

[Rob Herring]

On 04/09/2012 12:26 AM, jonsmirl at gmail.com wrote:
> On Sun, Apr 8, 2012 at 9:17 PM, Rob Herring <robherring2 at gmail.com> wrote:
>> On 04/07/2012 06:26 PM, jonsmirl at gmail.com wrote:
>>> Grant, moving  of_i2c.c into drivers/i2c would parallel your spi/gpio patches.
>>>
>>> of_i2c_register_devices() needs to be moved up into i2c-core. Right
>>> now the ten bus drivers are individually calling it.
>>
>> I posted patches to do just this a while back and the i2c maintainers
>> didn't like the idea. Feel free to try again. :)
> 
> Try resubmitting the patch, but only submit this change. Looking back
> at the original submission Jean was concerned about other aspects of
> your series and it looks like he ignored this part.

Really? Seems pretty clear to me:

> But anyway, I don't buy the whole thing. We currently have a nice 
> separation between OF and the underlying bus types. Breaking it all 
> just for the sake of saving one call in 4 drivers (out of ~70 in the 
> current kernel tree) doesn't seem worth it, at all. It would make 
> things harder to maintain, too (I definitely do not want to maintain 
> OF-specific code). 

Maybe when i2c is the only thing left and there are dozens of OF enabled
i2c drivers he'll feel differently. In the meantime, I've got more
pressing things to spend time on.

Rob