Re: [PATCH 0/9] mfd: Remove redundant i2c_device_id

From: Keerthy
Date: Tue Jun 06 2017 - 07:01:48 EST




On Tuesday 06 June 2017 04:14 PM, Tony Lindgren wrote:
> * Keerthy <j-keerthy@xxxxxx> [170606 00:52]:
>> On Tuesday 06 June 2017 01:00 PM, Keerthy wrote:
>>> On Tuesday 06 June 2017 12:47 PM, Lee Jones wrote:
>>>>>> Are all of these drivers DT only?
>>>>>
>>>>> At least i did not find any platform code calling on to these drivers.
>>>>> No dependency on OF as such. Hence sent these 9.
>>>>
>>>> I can't accept these changes on any drivers which does not depend on
>>>> OF as a hard requirement.
>>>
>>> Okay. So that is a per-requisite. At least palmas, tps65218, lp873x,
>>> tps65217 imho can have the OF dependency. I will post patches for that
>>> and then will resurrect this series.
>>>
>>> Thanks for your feedback.
>>
>> Tony,
>>
>> I do not see any platform specific code for tps65218, tps65217 and
>> palmas. I believe they are all completely Device Tree based. Can i make
>> them depend on OF? It helps me clear up some unused data from their mfd
>> drivers.
>
> Yes all of arch/arm/mach-omap2 is device tree only nowadays.
> There may be other SoCs or architectures using these too,
> but grepping for platform_data should reveal that.

Yes Tony. I did that for all the above mentioned devices. I did not find
any references apart from device tree folder under arch/ folder.

Apart from device tree folder and drivers/ folder i do not see any
instances of palmas, tps65218, tps65917, lp873x.

Shall i go ahead and introduce dependency on OF?

>
> Regards,
>
> Tony
>