[linux-yocto] [PATCH] ARM: dts: am335x-boneblack: configure i2c1 and 2

[Bruce Ashfield]

On 2016-06-01 02:33 PM, Nishanth Menon wrote:
> On 06/01/2016 01:29 PM, Bruce Ashfield wrote:
>> On 2016-05-31 10:43 PM, Li, Yong B wrote:
>>> Thanks Nishanth for your information. I find the 5d1a2961adf9 commit in linux-omap. It seems to me that it only enables the I2C2 bus. Is it correct? We want to enable both the I2C1 and I2C2 buses for external i2c devices.
>>>
>>
>> As long as it works for what you need (i.e. you've tested it), and
>> the patch is from some public repo that I can refernece. I'm ok
>> with merging it.
>>
>>> Hi Bruce, the original source for the patch is https://github.com/nmenon/powertool/blob/master/kernel-patches/0001-v3.15.0-ARM-dts-am335x-boneblack-configure-i2c1-and-2.patch
>>
>> That's fine with me, can you update the commit log and re-submit the
>> patch ? Preferably with a short summary of how you tested the change
>> as well.
>
>
> the patch was created by me on an ancient kernel previously because
> there was no dt overlay support. neither i2c1 nor i2c2 are necessary
> for BBB to function. as the original author of the patch, I have to
> request a NAK. it was specifically done for a power measurement tool
> that i had written which runs on BBB (it uses i2c to read INA226
> measurement IC)

Aha! Thanks for the history.

We definitely want/need the modern support for the buses and addons.
So I'll drop this merge.

Bruce

>
>