diff options
author | Sascha Hauer <s.hauer@pengutronix.de> | 2014-05-16 09:36:53 +0200 |
---|---|---|
committer | Sascha Hauer <s.hauer@pengutronix.de> | 2014-06-04 20:57:24 +0200 |
commit | c50bdcf71e9002b31e45c13973c3db477823387e (patch) | |
tree | 01f206ec6ee338d257ff7e24dcd6c72df14dd05c /arch/arm/mach-imx/iim.c | |
parent | 579dce92c8bcd13a6424f80ed554b7621784297b (diff) | |
download | barebox-c50bdcf71e9002b31e45c13973c3db477823387e.tar.gz barebox-c50bdcf71e9002b31e45c13973c3db477823387e.tar.xz |
ARM: i.MX: iim: introduce private data struct
Instead of duplicating data shared between the banks in a bank
specific struct, use a iim struct and a bank struct.
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'arch/arm/mach-imx/iim.c')
-rw-r--r-- | arch/arm/mach-imx/iim.c | 74 |
1 files changed, 44 insertions, 30 deletions
diff --git a/arch/arm/mach-imx/iim.c b/arch/arm/mach-imx/iim.c index 8e37cd7138..16d6649070 100644 --- a/arch/arm/mach-imx/iim.c +++ b/arch/arm/mach-imx/iim.c @@ -30,16 +30,25 @@ #include <mach/iim.h> #define DRIVERNAME "imx_iim" +#define IIM_NUM_BANKS 8 static int iim_write_enable; static int iim_sense_enable; +struct iim_priv; + +struct iim_bank { + struct cdev cdev; + void __iomem *bankbase; + int bank; + struct iim_priv *iim; +}; + struct iim_priv { struct cdev cdev; void __iomem *base; void __iomem *bankbase; - int bank; - int banksize; + struct iim_bank *bank[IIM_NUM_BANKS]; }; static int do_fuse_sense(void __iomem *reg_base, unsigned int bank, @@ -88,22 +97,23 @@ static ssize_t imx_iim_cdev_read(struct cdev *cdev, void *buf, size_t count, loff_t offset, ulong flags) { ulong size, i; - struct iim_priv *priv = cdev->priv; + struct iim_bank *bank = container_of(cdev, struct iim_bank, cdev); + struct iim_priv *iim = bank->iim; - size = min((loff_t)count, priv->banksize - offset); + size = min((loff_t)count, 32 - offset); if (iim_sense_enable) { for (i = 0; i < size; i++) { int row_val; - row_val = do_fuse_sense(priv->base, - priv->bank, offset + i); + row_val = do_fuse_sense(iim->base, + bank->bank, offset + i); if (row_val < 0) return row_val; ((u8 *)buf)[i] = (u8)row_val; } } else { for (i = 0; i < size; i++) - ((u8 *)buf)[i] = ((u8 *)priv->bankbase)[(offset+i)*4]; + ((u8 *)buf)[i] = ((u8 *)bank->bankbase)[(offset+i)*4]; } return size; @@ -173,22 +183,23 @@ static ssize_t imx_iim_cdev_write(struct cdev *cdev, const void *buf, size_t cou loff_t offset, ulong flags) { ulong size, i; - struct iim_priv *priv = cdev->priv; + struct iim_bank *bank = container_of(cdev, struct iim_bank, cdev); + struct iim_priv *iim = bank->iim; - size = min((loff_t)count, priv->banksize - offset); + size = min((loff_t)count, 32 - offset); if (IS_ENABLED(CONFIG_IMX_IIM_FUSE_BLOW) && iim_write_enable) { for (i = 0; i < size; i++) { int ret; - ret = do_fuse_blow(priv->base, priv->bank, + ret = do_fuse_blow(iim->base, bank->bank, offset + i, ((u8 *)buf)[i]); if (ret < 0) return ret; } } else { for (i = 0; i < size; i++) - ((u8 *)priv->bankbase)[(offset+i)*4] = ((u8 *)buf)[i]; + ((u8 *)bank->bankbase)[(offset+i)*4] = ((u8 *)buf)[i]; } return size; @@ -200,26 +211,25 @@ static struct file_operations imx_iim_ops = { .lseek = dev_lseek_default, }; -static int imx_iim_add_bank(struct device_d *dev, void __iomem *base, int num) +static int imx_iim_add_bank(struct iim_priv *iim, int num) { - struct iim_priv *priv; + struct iim_bank *bank; struct cdev *cdev; - priv = xzalloc(sizeof (*priv)); + bank = xzalloc(sizeof (*bank)); - priv->base = base; - priv->bankbase = priv->base + 0x800 + 0x400 * num; - priv->bank = num; - priv->banksize = 32; - cdev = &priv->cdev; - cdev->dev = dev; + bank->bankbase = iim->base + 0x800 + 0x400 * num; + bank->bank = num; + bank->iim = iim; + cdev = &bank->cdev; cdev->ops = &imx_iim_ops; - cdev->priv = priv; cdev->size = 32; cdev->name = asprintf(DRIVERNAME "_bank%d", num); if (cdev->name == NULL) return -ENOMEM; + iim->bank[num] = bank; + return devfs_create(cdev); } @@ -231,7 +241,7 @@ static int imx_iim_add_bank(struct device_d *dev, void __iomem *base, int num) */ #define MAC_ADDRESS_PROPLEN (3 * sizeof(__be32)) -static void imx_iim_init_dt(struct device_d *dev) +static void imx_iim_init_dt(struct device_d *dev, struct iim_priv *iim) { char mac[6]; const __be32 *prop; @@ -266,25 +276,29 @@ static void imx_iim_init_dt(struct device_d *dev) } } #else -static inline void imx_iim_init_dt(struct device_d *dev) +static inline void imx_iim_init_dt(struct device_d *dev, struct iim_priv *iim) { } #endif static int imx_iim_probe(struct device_d *dev) { - int i; - void __iomem *base; + struct iim_priv *iim; + int i, ret; + + iim = xzalloc(sizeof(*iim)); - base = dev_request_mem_region(dev, 0); - if (!base) + iim->base = dev_request_mem_region(dev, 0); + if (!iim->base) return -EBUSY; - for (i = 0; i < 8; i++) { - imx_iim_add_bank(dev, base, i); + for (i = 0; i < IIM_NUM_BANKS; i++) { + ret = imx_iim_add_bank(iim, i); + if (ret) + return ret; } - imx_iim_init_dt(dev); + imx_iim_init_dt(dev, iim); if (IS_ENABLED(CONFIG_IMX_IIM_FUSE_BLOW)) dev_add_param_bool(dev, "permanent_write_enable", |