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




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

Reply via email to