From mboxrd@z Thu Jan  1 00:00:00 1970
From: Dirk Behme <dirk.behme@googlemail.com>
Date: Mon, 03 Nov 2008 21:27:43 +0100
Subject: [U-Boot] [PATCH 09/13 v5] ARM: OMAP3: Add I2C support
In-Reply-To: <20081103001842.GB10086@game.jcrosoft.org>
References: <490df394.1438560a.2e57.26df@mx.google.com>
	<490df3a8.0c07560a.1694.2f83@mx.google.com>
	<20081103001842.GB10086@game.jcrosoft.org>
Message-ID: <490F5EBF.9080102@googlemail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Jean-Christophe PLAGNIOL-VILLARD wrote:
> On 19:38 Sun 02 Nov     , dirk.behme at googlemail.com wrote:
> 
>>Subject: [PATCH 09/13 v5] ARM: OMAP3: Add I2C support
>>
>>From: Dirk Behme <dirk.behme@gmail.com>
>>
>>Add I2C support
>>
>>Signed-off-by: Dirk Behme <dirk.behme@gmail.com>
>>
>>---
>>Changes in version v5:
>>
>>- Split functional changes and coding style clean up as proposed by Jean-Christophe PLAGNIOL-VILLARD.
>>
>>Changes in version v2:
>>
>>- Remove SMC911X network init as proposed by Ben Warren. Thanks!
>>
>> drivers/i2c/Makefile       |    1 +
>> drivers/i2c/omap24xx_i2c.c |   24 ++++++++++++++++++++++++
>> 2 files changed, 25 insertions(+)
>>
>>Index: u-boot-main/drivers/i2c/Makefile
>>===================================================================
>>--- u-boot-main.orig/drivers/i2c/Makefile
>>+++ u-boot-main/drivers/i2c/Makefile
>>@@ -29,6 +29,7 @@ COBJS-$(CONFIG_FSL_I2C) += fsl_i2c.o
>> COBJS-$(CONFIG_I2C_MXC) += mxc_i2c.o
>> COBJS-$(CONFIG_DRIVER_OMAP1510_I2C) += omap1510_i2c.o
>> COBJS-$(CONFIG_DRIVER_OMAP24XX_I2C) += omap24xx_i2c.o
>>+COBJS-$(CONFIG_DRIVER_OMAP34XX_I2C) += omap24xx_i2c.o
>> COBJS-$(CONFIG_SOFT_I2C) += soft_i2c.o
>> COBJS-$(CONFIG_TSI108_I2C) += tsi108_i2c.o
>> 
>>Index: u-boot-main/drivers/i2c/omap24xx_i2c.c
>>===================================================================
>>--- u-boot-main.orig/drivers/i2c/omap24xx_i2c.c
>>+++ u-boot-main/drivers/i2c/omap24xx_i2c.c
>>@@ -25,6 +25,8 @@
>> #include <asm/arch/i2c.h>
>> #include <asm/io.h>
>> 
>>+#define inb(a) __raw_readb(a)
>>+#define outb(a, v) __raw_writeb(a, v)
>> #define inw(a) __raw_readw(a)
>> #define outw(a,v) __raw_writew(a,v)
> 
> This 4 macro is supposed to be defined in io.h

Even if I have to touch a global/common file for this?

>> 
>>@@ -112,7 +114,11 @@ static int i2c_read_byte (u8 devaddr, u8
>> 
>> 		status = wait_for_pin ();
>> 		if (status & I2C_STAT_RRDY) {
>>+#if defined(CONFIG_OMAP243X) || defined(CONFIG_OMAP34XX)
>>+			*value = inb(I2C_DATA);
> 
> please respect the file style, add a space before '('
> and so on

You prefer file style over global coding style? As I understand it 
"inb(" is global coding style, and I tried to be in sync with global 
coding style at least with code I add.

>>+#else
>> 			*value = inw (I2C_DATA);
>>+#endif
>> 			udelay (20000);
> 
> Best Regards,
> J.
>