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

[Bruce Ashfield]

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.

Bruce

>
> Thanks,
> Yong
> -----Original Message-----
> From: Nishanth Menon [mailto:nm at ti.com]
> Sent: Wednesday, June 1, 2016 2:42 AM
> To: Ashfield, Bruce (Wind River) <bruce.ashfield at windriver.com>; Li, Yong B <yong.b.li at intel.com>; linux-yocto at yoctoproject.org; sgw at linux.intel.com; Wold, Saul <saul.wold at intel.com>
> Cc: sdliyong at gmail.com
> Subject: Re: [PATCH] ARM: dts: am335x-boneblack: configure i2c1 and 2
>
> On 05/31/2016 12:36 PM, Bruce Ashfield wrote:
>> On 2016-05-31 3:00 AM, Yong Li wrote:
>>> From: Nishanth Menon <nm at ti.com>
>>>
>>> Configure i2c1 and 2 at 400KHz.
>>>
>>> Signed-off-by: Nishanth Menon <nm at ti.com>
>>>
>>> Upstream-status: Inappropriate, not author
>>
>> Since you aren't the author, is there a public link that you can
>> reference for the source of the patch ?
>>
>> We should have that, so we can show that it is unmodified as part of
>> the merge, and for tracking purposes.
>
>
> If you folks want to support Capes, you might be better off picking up upstream commit for the same.
>
> 5d1a2961adf9 ARM: dts: Beaglebone i2c definitions
>
>
> The patch was probably done on earlier kernel, I cannot see this on TI product 4.1/4.4 kernels
>
> --
> Regards,
> Nishanth Menon
>