Commit a0832798 authored by Tzachi Perelstein's avatar Tzachi Perelstein Committed by Russell King

[I2C] Split mv643xx I2C platform support

The motivation for this change is to allow other chips, like the
Marvell Orion ARM SoC family, to use the existing i2c-mv64xxx driver.
Signed-off-by: default avatarTzachi Perelstein <tzachi@marvell.com>
Acked-by: default avatarNicolas Pitre <nico@marvell.com>
Acked-by: default avatarDale Farnsworth <dale@farnsworth.org>
Acked-by: default avatarMark A. Greer <mgreer@mvista.com>
Acked-by: default avatarJean Delvare <khali@linux-fr.org>
parent 60ce1c20
...@@ -648,7 +648,7 @@ config I2C_PCA_ISA ...@@ -648,7 +648,7 @@ config I2C_PCA_ISA
config I2C_MV64XXX config I2C_MV64XXX
tristate "Marvell mv64xxx I2C Controller" tristate "Marvell mv64xxx I2C Controller"
depends on MV64X60 && EXPERIMENTAL depends on (MV64X60 || ARCH_ORION) && EXPERIMENTAL
help help
If you say yes to this option, support will be included for the 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. built-in I2C interface on the Marvell 64xxx line of host bridges.
......
/* /*
* Driver for the i2c controller on the Marvell line of host bridges for MIPS * Driver for the i2c controller on the Marvell line of host bridges
* and PPC (e.g, gt642[46]0, mv643[46]0, mv644[46]0). * (e.g, gt642[46]0, mv643[46]0, mv644[46]0, and Orion SoC family).
* *
* Author: Mark A. Greer <mgreer@mvista.com> * Author: Mark A. Greer <mgreer@mvista.com>
* *
...@@ -14,7 +14,7 @@ ...@@ -14,7 +14,7 @@
#include <linux/spinlock.h> #include <linux/spinlock.h>
#include <linux/i2c.h> #include <linux/i2c.h>
#include <linux/interrupt.h> #include <linux/interrupt.h>
#include <linux/mv643xx.h> #include <linux/mv643xx_i2c.h>
#include <linux/platform_device.h> #include <linux/platform_device.h>
#include <asm/io.h> #include <asm/io.h>
...@@ -86,6 +86,7 @@ struct mv64xxx_i2c_data { ...@@ -86,6 +86,7 @@ struct mv64xxx_i2c_data {
u32 cntl_bits; u32 cntl_bits;
void __iomem *reg_base; void __iomem *reg_base;
u32 reg_base_p; u32 reg_base_p;
u32 reg_size;
u32 addr1; u32 addr1;
u32 addr2; u32 addr2;
u32 bytes_left; u32 bytes_left;
...@@ -463,17 +464,20 @@ static int __devinit ...@@ -463,17 +464,20 @@ static int __devinit
mv64xxx_i2c_map_regs(struct platform_device *pd, mv64xxx_i2c_map_regs(struct platform_device *pd,
struct mv64xxx_i2c_data *drv_data) struct mv64xxx_i2c_data *drv_data)
{ {
struct resource *r; int size;
struct resource *r = platform_get_resource(pd, IORESOURCE_MEM, 0);
if ((r = platform_get_resource(pd, IORESOURCE_MEM, 0)) && if (!r)
request_mem_region(r->start, MV64XXX_I2C_REG_BLOCK_SIZE, return -ENODEV;
drv_data->adapter.name)) {
drv_data->reg_base = ioremap(r->start, size = r->end - r->start + 1;
MV64XXX_I2C_REG_BLOCK_SIZE);
drv_data->reg_base_p = r->start; if (!request_mem_region(r->start, size, drv_data->adapter.name))
} else return -EBUSY;
return -ENOMEM;
drv_data->reg_base = ioremap(r->start, size);
drv_data->reg_base_p = r->start;
drv_data->reg_size = size;
return 0; return 0;
} }
...@@ -483,8 +487,7 @@ mv64xxx_i2c_unmap_regs(struct mv64xxx_i2c_data *drv_data) ...@@ -483,8 +487,7 @@ mv64xxx_i2c_unmap_regs(struct mv64xxx_i2c_data *drv_data)
{ {
if (drv_data->reg_base) { if (drv_data->reg_base) {
iounmap(drv_data->reg_base); iounmap(drv_data->reg_base);
release_mem_region(drv_data->reg_base_p, release_mem_region(drv_data->reg_base_p, drv_data->reg_size);
MV64XXX_I2C_REG_BLOCK_SIZE);
} }
drv_data->reg_base = NULL; drv_data->reg_base = NULL;
......
...@@ -15,6 +15,7 @@ ...@@ -15,6 +15,7 @@
#include <asm/types.h> #include <asm/types.h>
#include <linux/mv643xx_eth.h> #include <linux/mv643xx_eth.h>
#include <linux/mv643xx_i2c.h>
/****************************************/ /****************************************/
/* Processor Address Space */ /* Processor Address Space */
...@@ -863,7 +864,6 @@ ...@@ -863,7 +864,6 @@
/* I2C Registers */ /* I2C Registers */
/****************************************/ /****************************************/
#define MV64XXX_I2C_CTLR_NAME "mv64xxx_i2c"
#define MV64XXX_I2C_OFFSET 0xc000 #define MV64XXX_I2C_OFFSET 0xc000
#define MV64XXX_I2C_REG_BLOCK_SIZE 0x0020 #define MV64XXX_I2C_REG_BLOCK_SIZE 0x0020
...@@ -968,14 +968,6 @@ struct mpsc_pdata { ...@@ -968,14 +968,6 @@ struct mpsc_pdata {
u32 brg_clk_freq; u32 brg_clk_freq;
}; };
/* i2c Platform Device, Driver Data */
struct mv64xxx_i2c_pdata {
u32 freq_m;
u32 freq_n;
u32 timeout; /* In milliseconds */
u32 retries;
};
/* Watchdog Platform Device, Driver Data */ /* Watchdog Platform Device, Driver Data */
#define MV64x60_WDT_NAME "mv64x60_wdt" #define MV64x60_WDT_NAME "mv64x60_wdt"
......
/*
* This program is free software; you can redistribute it and/or modify it
* under the terms of the GNU General Public License as published by the
* Free Software Foundation; either version 2 of the License, or (at your
* option) any later version.
*/
#ifndef _MV64XXX_I2C_H_
#define _MV64XXX_I2C_H_
#include <linux/types.h>
#define MV64XXX_I2C_CTLR_NAME "mv64xxx_i2c"
/* i2c Platform Device, Driver Data */
struct mv64xxx_i2c_pdata {
u32 freq_m;
u32 freq_n;
u32 timeout; /* In milliseconds */
u32 retries;
};
#endif /*_MV64XXX_I2C_H_*/
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment