[PATCH] ARM: dts: am335x-boneblack: add i2c1 DT entry

[Li, Yong B]

Hi Tony,

I can submit a V2 version about the "disabled", or just drop this patch?

Thanks,
Yong
-----Original Message-----
From: Tony Lindgren [mailto:tony at atomide.com] 
Sent: Friday, June 3, 2016 1:57 AM
To: Robert Nelson <robertcnelson at gmail.com>
Cc: Li, Yong B <yong.b.li at intel.com>; Benoit Cousson <bcousson at baylibre.com>; Rob Herring <robh+dt at kernel.org>; Pawel Moll <pawel.moll at arm.com>; Mark Rutland <mark.rutland at arm.com>; Ian Campbell <ijc+devicetree at hellion.org.uk>; Kumar Gala <galak at codeaurora.org>; Russell King <linux at arm.linux.org.uk>; linux-omap at vger.kernel.org; devicetree <devicetree at vger.kernel.org>; linux-arm-kernel at lists.infradead.org; linux kernel <linux-kernel at vger.kernel.org>; sdliyong at gmail.com
Subject: Re: [PATCH] ARM: dts: am335x-boneblack: add i2c1 DT entry

* Robert Nelson <robertcnelson at gmail.com> [160602 10:48]:
> On Thu, Jun 2, 2016 at 12:39 PM, Tony Lindgren <tony at atomide.com> wrote:
> >
> > * Robert Nelson <robertcnelson at gmail.com> [160602 10:39]:
> > > On Thu, Jun 2, 2016 at 2:41 AM, Yong Li <yong.b.li at intel.com> wrote:
> > >
> > > > From: Yong Li <sdliyong at gmail.com>
> > > >
> > > > Without this patch, I2C-1 is missing on beaglebone black boards
> > > >
> > > > Signed-off-by: Yong Li <sdliyong at gmail.com>
> > > > ---
> > > >  arch/arm/boot/dts/am335x-boneblack.dts | 15 +++++++++++++++
> > > >  1 file changed, 15 insertions(+)
> > > >
> > >
> > >
> > > This is also 'common' for the white/green, if your really going to 
> > > push this, please add it to:
> > >
> > > am335x-bone-common.dtsi
> >
> > But isn't this exposed on the cape pins? So the use can be whatever 
> > for these pins?
> 
> That is correct, "i2c1" is on the cape connector, with no primary 
> function defined.
> 
> Currently on mainline, the board and cape eeprom i2c bus's are enabled.
> 
> i2c0 = board eeprom
> i2c2 = 'cape' eeprom
> 
> personally, nak by me to add i2c1 by default, should use overlay's to enable it.

Yes sounds like the best we can do is define it but set it with status = "disabled" for now.

Regards,

Tony