Skip to contentSkip to conversation

AU1100 I2C support

PATCH In review Rodolfo Giometti 4 participants 6 messages Jul 19, 2006 – Jul 24, 2006

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 AU1100 I2C support

Hello,

here a patch to add I2C support to AU1100 processors by using two GPIOs.

Ciao,

Rodolfo

Signature · 4 lines
GNU/Linux Solutions                  e-mail:    [email protected]
Linux Device Driver                             [email protected]
Embedded Systems                     		[email protected]
UNIX programming                     phone:     +39 349 2432127
drivers/i2c/busses/Kconfig +12 -0
index 884320e..222c7fc 100644
@@ -419,6 +419,18 @@ config SCx200_ACB
 	  This support is also available as a module.  If so, the module 
 	  will be called scx200_acb.
 
+config AU1100_I2C
+	tristate "Au1100 I2C using GPIO pins"
+	depends on SOC_AU1100
+	select I2C_ALGOBIT
+	help
+	  Enable the use of two GPIO pins of a Au1100 processor as an I2C bus.
+
+	  If you don't know what to do here, say N.
+
+	  This support is also available as a module.  If so, the module 
+	  will be called i2c-au1100.
+
 config I2C_SIS5595
 	tristate "SiS 5595"
 	depends on I2C && PCI
drivers/i2c/busses/Makefile +1 -0
index ac56df5..adb7576 100644
@@ -43,6 +43,7 @@ obj-$(CONFIG_I2C_VIAPRO)	+= i2c-viapro.o
 obj-$(CONFIG_I2C_VOODOO3)	+= i2c-voodoo3.o
 obj-$(CONFIG_SCx200_ACB)	+= scx200_acb.o
 obj-$(CONFIG_SCx200_I2C)	+= scx200_i2c.o
+obj-$(CONFIG_AU1100_I2C)	+= i2c-au1100.o
 
 ifeq ($(CONFIG_I2C_DEBUG_BUS),y)
 EXTRA_CFLAGS += -DDEBUG
drivers/i2c/busses/i2c-au1100.c new file +139 -0

139 lines not on this page · raw message

#1 lore raw

Hi,

Quoted 2 lines · Rodolfo Giometti

here a patch to add I2C support to AU1100 processors by using two GPIOs.

Is there any reason why it is limited to this very processor and should not work with all au1xxx types?

Daniel

#2 lore raw
Quoted 3 lines · Daniel Mack

Is there any reason why it is limited to this very processor and should not work with all au1xxx types?

To be honest I don't know exacly... I don't know so well other processors in this family. However I think at least au1000 and au1200 should be compatible.

Ciao,

Rodolfo

Signature · 4 lines
GNU/Linux Solutions                  e-mail:    [email protected]
Linux Device Driver                             [email protected]
Embedded Systems                     		[email protected]
UNIX programming                     phone:     +39 349 2432127
#3 lore raw
Quoted 8 lines · Rodolfo Giometti
Quoted 3 lines · Daniel Mack

Is there any reason why it is limited to this very processor and should not work with all au1xxx types?

To be honest I don't know exacly... I don't know so well other processors in this family. However I think at least au1000 and au1200 should be compatible.

I think so, too. Since you only call the generic au_* functions for the GPIO access, all members of this processor family should be supported. I propose to change the patch accordingly.

Daniel

#4 lore raw
Quoted 4 lines · Daniel Mack

I think so, too. Since you only call the generic au_* functions for the GPIO access, all members of this processor family should be supported. I propose to change the patch accordingly.

Do you think I should change strings (functions' name, etc.) from "au1100" to "au1xxx"?

If so, do you prefere a new patch or I can just send you a patch-to-the-patch? :D

Ciao,

Rodolfo

Signature · 4 lines
GNU/Linux Solutions                  e-mail:    [email protected]
Linux Device Driver                             [email protected]
Embedded Systems                     		[email protected]
UNIX programming                     phone:     +39 349 2432127
#5 lore raw
In reply to Daniel Mack

Hello.

Quoted 5 lines · Daniel Mack
Quoted 4 lines · Daniel Mack
Quoted 3 lines

Is there any reason why it is limited to this very processor and should not work with all au1xxx types?

Quoted 7 lines
Quoted 3 lines

To be honest I don't know exacly... I don't know so well other processors in this family. However I think at least au1000 and au1200 should be compatible.

I think so, too. Since you only call the generic au_* functions for the GPIO access, all members of this processor family should be supported. I propose to change the patch accordingly.

Actually, Au1550/1200 have dedicated SMBus contoller and an existing I2C
driver for it, so there''s probably no need for another bit-banging driver for them.

Quoted 1 line

Daniel

WBR, Sergei