Skip to contentSkip to conversation

i2c: mv64xxx: Add support Armada-XP SoC

PATCH New Nobuhiro Iwamatsu 3 participants 5 messages Oct 30, 2012 – Oct 31, 2012 I2C subsystem host drivers

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 i2c: mv64xxx: Add support Armada-XP SoC

The Armada-XP SoC has the same I2C controller as mv64xxxi-i2c. This validates to use mv64xxx in this SoC.

---
 drivers/i2c/busses/Kconfig |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/i2c/busses/Kconfig +1 -1
index 65dd599..ff53cf4 100644
@@ -474,7 +474,7 @@ config I2C_MPC
 
 config I2C_MV64XXX
 	tristate "Marvell mv64xxx I2C Controller"
-	depends on (MV64X60 || PLAT_ORION)
+	depends on (MV64X60 || PLAT_ORION || MACH_ARMADA_370_XP)
 	help
 	  If you say yes to this option, support will be included for the
 	  built-in I2C interface on the Marvell 64xxx line of host bridges.
Signature · 1 line
1.7.10.4
#1 lore raw

PATCH 1/1 i2c: mv64xxx: Add support Armada-XP SoC

Dear Nobuhiro Iwamatsu,

Quoted 13 lines · Nobuhiro Iwamatsu

The Armada-XP SoC has the same I2C controller as mv64xxxi-i2c. This validates to use mv64xxx in this SoC.

---
 drivers/i2c/busses/Kconfig |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/i2c/busses/Kconfig +0 -0
index 65dd599..ff53cf4 100644

drivers/i2c/busses/Kconfig config I2C_MPC · line 480

Quoted 8 lines
drivers/i2c/busses/Kconfig +1 -1
@@ -474,7 +474,7 @@ config I2C_MPC
 
 config I2C_MV64XXX
 	tristate "Marvell mv64xxx I2C Controller"
-	depends on (MV64X60 || PLAT_ORION)
+	depends on (MV64X60 || PLAT_ORION || MACH_ARMADA_370_XP)
 	help
 	  If you say yes to this option, support will be included for the
 	  built-in I2C interface on the Marvell 64xxx line of host bridges.

This is not needed: ARCH_MVEBU already selects PLAT_ORION.

Best regards,

Thomas

Signature · 4 lines
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
#2 lore raw

PATCH 1/1 i2c: mv64xxx: Add support Armada-XP SoC

Hi,

Quoted 17 lines · Thomas Petazzoni

Dear Nobuhiro Iwamatsu,

Quoted 14 lines · Nobuhiro Iwamatsu

The Armada-XP SoC has the same I2C controller as mv64xxxi-i2c. This validates to use mv64xxx in this SoC.

---
 drivers/i2c/busses/Kconfig |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/i2c/busses/Kconfig +0 -0
index 65dd599..ff53cf4 100644
@@ -474,7 +474,7 @@ config I2C_MPC
 
Quoted 10 lines

drivers/i2c/busses/Kconfig config I2C_MPC · line 480

Quoted 7 lines
drivers/i2c/busses/Kconfig +1 -1
 config I2C_MV64XXX
      tristate "Marvell mv64xxx I2C Controller"
-     depends on (MV64X60 || PLAT_ORION)
+     depends on (MV64X60 || PLAT_ORION || MACH_ARMADA_370_XP)
      help
        If you say yes to this option, support will be included for the
        built-in I2C interface on the Marvell 64xxx line of host bridges.

This is not needed: ARCH_MVEBU already selects PLAT_ORION.

Oh, I see. Thanks!

Nobuhiro

Signature · 3 lines
Nobuhiro Iwamatsu
   iwamatsu at {nigauri.org / debian.org}
   GPG ID: 40AD1FA6
#3 lore raw

PATCH 1/1 i2c: mv64xxx: Add support Armada-XP SoC

In reply to Nobuhiro Iwamatsu

Hello.

Quoted 13 lines · Nobuhiro Iwamatsu

The Armada-XP SoC has the same I2C controller as mv64xxxi-i2c. This validates to use mv64xxx in this SoC.

---
 drivers/i2c/busses/Kconfig |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/i2c/busses/Kconfig +0 -0
index 65dd599..ff53cf4 100644

drivers/i2c/busses/Kconfig config I2C_MPC · line 477

Quoted 5 lines
drivers/i2c/busses/Kconfig +1 -1
@@ -474,7 +474,7 @@ config I2C_MPC
 
 config I2C_MV64XXX
 	tristate "Marvell mv64xxx I2C Controller"
-	depends on (MV64X60 || PLAT_ORION)
+	depends on (MV64X60 || PLAT_ORION || MACH_ARMADA_370_XP)

Could remove useless parens, while at it...

WBR, Sergei

#4 lore raw

PATCH 1/1 i2c: mv64xxx: Add support Armada-XP SoC

Hi,

Quoted 18 lines · Sergei Shtylyov

Hello.

Quoted 14 lines · Nobuhiro Iwamatsu

The Armada-XP SoC has the same I2C controller as mv64xxxi-i2c. This validates to use mv64xxx in this SoC.

---
 drivers/i2c/busses/Kconfig |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/i2c/busses/Kconfig +0 -0
index 65dd599..ff53cf4 100644
@@ -474,7 +474,7 @@ config I2C_MPC
 
Quoted 7 lines

drivers/i2c/busses/Kconfig config I2C_MPC · line 477

Quoted 4 lines
drivers/i2c/busses/Kconfig +1 -1
 config I2C_MV64XXX
      tristate "Marvell mv64xxx I2C Controller"
-     depends on (MV64X60 || PLAT_ORION)
+     depends on (MV64X60 || PLAT_ORION || MACH_ARMADA_370_XP)

Could remove useless parens, while at it...

OK, I will send a patch which revise about this.

Signature · 3 lines
Nobuhiro Iwamatsu
   iwamatsu at {nigauri.org / debian.org}
   GPG ID: 40AD1FA6