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:n...@ti.com]
Sent: Wednesday, June 1, 2016 2:42 AM
To: Ashfield, Bruce (Wind River) <bruce.ashfi...@windriver.com>; Li, Yong B 
<yong.b...@intel.com>; linux-yocto@yoctoproject.org; s...@linux.intel.com; Wold, Saul 
<saul.w...@intel.com>
Cc: sdliy...@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 <n...@ti.com>

Configure i2c1 and 2 at 400KHz.

Signed-off-by: Nishanth Menon <n...@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


--
_______________________________________________
linux-yocto mailing list
linux-yocto@yoctoproject.org
https://lists.yoctoproject.org/listinfo/linux-yocto

Reply via email to