On Wed, Nov 28, 2018 at 8:44 PM Dennis Gilmore dgilmore@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@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 = ð0;
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.
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