]> pilppa.com Git - linux-2.6-omap-h63xx.git/commitdiff
i2c-s3c2410: sda_delay should be in ns, not clock ticks
authorBen Dooks <ben@simtec.co.uk>
Fri, 27 Mar 2009 10:52:13 +0000 (10:52 +0000)
committerBen Dooks <ben-linux@fluff.org>
Tue, 7 Apr 2009 09:18:33 +0000 (10:18 +0100)
The sda_delay field should be specified in ns, not in clock ticks
as when using cpufreq we could be changing the bus rate.

Signed-off-by: Ben Dooks <ben@simtec.co.uk>
arch/arm/plat-s3c/dev-i2c0.c
arch/arm/plat-s3c/dev-i2c1.c
drivers/i2c/busses/i2c-s3c2410.c

index fe327074037ea05d4d996cbe52be7ab9a2deae78..8eecb466ab4c3b630ed78924e7c7a39299045116 100644 (file)
@@ -52,7 +52,7 @@ static struct s3c2410_platform_i2c default_i2c_data0 __initdata = {
        .slave_addr     = 0x10,
        .bus_freq       = 100*1000,
        .max_freq       = 400*1000,
-       .sda_delay      = S3C2410_IICLC_SDA_DELAY5 | S3C2410_IICLC_FILTER_ON,
+       .sda_delay      = 100,
 };
 
 void __init s3c_i2c0_set_platdata(struct s3c2410_platform_i2c *pd)
index 2387fbf57af6da984b49dce28fcf08a55a6afee5..4536e5bb50e5fbad2ffffbd9b1cc1908c752355b 100644 (file)
@@ -49,7 +49,7 @@ static struct s3c2410_platform_i2c default_i2c_data1 __initdata = {
        .slave_addr     = 0x10,
        .bus_freq       = 100*1000,
        .max_freq       = 400*1000,
-       .sda_delay      = S3C2410_IICLC_SDA_DELAY5 | S3C2410_IICLC_FILTER_ON,
+       .sda_delay      = 100,
 };
 
 void __init s3c_i2c1_set_platdata(struct s3c2410_platform_i2c *pd)
index 5b7f95641ba48119d17fe71c7fc02b8c3b16fd2d..9d797561de5fe0eabf56641a5e291c901819fa2b 100644 (file)
@@ -663,6 +663,23 @@ static int s3c24xx_i2c_clockrate(struct s3c24xx_i2c *i2c, unsigned int *got)
 
        writel(iiccon, i2c->regs + S3C2410_IICCON);
 
+       if (s3c24xx_i2c_is2440(i2c)) {
+               unsigned long sda_delay;
+
+               if (pdata->sda_delay) {
+                       sda_delay = (freq / 1000) * pdata->sda_delay;
+                       sda_delay /= 1000000;
+                       sda_delay = DIV_ROUND_UP(sda_delay, 5);
+                       if (sda_delay > 3)
+                               sda_delay = 3;
+                       sda_delay |= S3C2410_IICLC_FILTER_ON;
+               } else
+                       sda_delay = 0;
+
+               dev_dbg(i2c->dev, "IICLC=%08lx\n", sda_delay);
+               writel(sda_delay, i2c->regs + S3C2440_IICLC);
+       }
+
        return 0;
 }
 
@@ -769,11 +786,8 @@ static int s3c24xx_i2c_init(struct s3c24xx_i2c *i2c)
 
        /* check for s3c2440 i2c controller  */
 
-       if (s3c24xx_i2c_is2440(i2c)) {
-               dev_dbg(i2c->dev, "S3C2440_IICLC=%08x\n", pdata->sda_delay);
-
-               writel(pdata->sda_delay, i2c->regs + S3C2440_IICLC);
-       }
+       if (s3c24xx_i2c_is2440(i2c))
+               writel(0x0, i2c->regs + S3C2440_IICLC);
 
        return 0;
 }