[U-Boot] [PATCH 3/7] arm: mvebu: helios4: Enable I2C

[Dennis Gilmore]

El jue, 29-11-2018 a las 00:42 +0700, Aditya Prayoga escribió:
> On Wed, Nov 28, 2018 at 8:44 PM Dennis Gilmore <dgilmore at redhat.com>
> wrote:
> > On Wed, 2018-11-28 at 16:59 +0800, Aditya Prayoga wrote:
> > > Enable Marvell I2C driver and set default bus to external I2C
> > > bus.
> > > Define I2C aliases in device tree so it can be recognized by the
> > > driver.
> > > 
> > > Signed-off-by: Aditya Prayoga <aditya at kobol.io>
> > > ---
> > >  arch/arm/dts/armada-388-helios4.dts | 2 ++
> > >  configs/helios4_defconfig           | 3 +++
> > >  2 files changed, 5 insertions(+)
> > > 
> > > diff --git a/arch/arm/dts/armada-388-helios4.dts
> > > b/arch/arm/dts/armada-388-helios4.dts
> > > index a154e0f..711fc22 100644
> > > --- a/arch/arm/dts/armada-388-helios4.dts
> > > +++ b/arch/arm/dts/armada-388-helios4.dts
> > > @@ -24,6 +24,8 @@
> > >       aliases {
> > >               /* So that mvebu u-boot can update the MAC
> > > addresses */
> > >               ethernet1 = &eth0;
> > > +             i2c0 = &i2c0;
> > > +             i2c1 = &i2c1;
> > 
> > have you sent the dts changes to the linux kernel also?
> 
> No, I haven't. Should I also send it to Linux kernel? WIthout those
> aliases, Linux
> can detect the i2c buses.

we should put them in -u-boot.dtsi files if they are not needed then. 
u-boot has been trying to make sure that they can sync devicetree files
from the kernel without modification.

Dennis

> Aditya
> 
> > Dennis
> > >       };
> > > 
> > >       chosen {
> > > diff --git a/configs/helios4_defconfig
> > > b/configs/helios4_defconfig
> > > index e30eb3d..8ac8978 100644
> > > --- a/configs/helios4_defconfig
> > > +++ b/configs/helios4_defconfig
> > > @@ -39,6 +39,9 @@ CONFIG_NET_RANDOM_ETHADDR=y
> > >  CONFIG_SPL_OF_TRANSLATE=y
> > >  CONFIG_SCSI_AHCI=y
> > >  CONFIG_DM_I2C=y
> > > +CONFIG_SYS_I2C_MVTWSI=y
> > > +CONFIG_I2C_SET_DEFAULT_BUS_NUM=y
> > > +CONFIG_I2C_DEFAULT_BUS_NUMBER=0x1
> > >  CONFIG_MMC_SDHCI=y
> > >  CONFIG_MMC_SDHCI_SDMA=y
> > >  CONFIG_MMC_SDHCI_MV=y