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

[Yong Li]

Thanks Nishanth!

Hi Bruce, based on the discussion, please merge the
5d1a2961adf906f965b00eb8059fd2e0585e0e09 from
git://git.kernel.org/pub/scm/linux/kernel/git/tmlind/linux-omap.git.

Regarding I2C1 support, let me try to find/submit another patch.

Thanks,
Yong

2016-06-02 3:22 GMT+08:00 Bruce Ashfield <bruce.ashfield at windriver.com>:
> 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
>
>>
>>
>