Commit 449012da authored by Scott Wood's avatar Scott Wood Committed by Kumar Gala

[POWERPC] cpm2: Infrastructure code cleanup.

Mostly sparse fixes (__iomem annotations, etc); also, cpm2_immr
is used rather than creating many temporary mappings.
Signed-off-by: default avatarScott Wood <scottwood@freescale.com>
Signed-off-by: default avatarKumar Gala <galak@kernel.crashing.org>
parent 544cdabe
...@@ -33,6 +33,8 @@ ...@@ -33,6 +33,8 @@
#include <linux/mm.h> #include <linux/mm.h>
#include <linux/interrupt.h> #include <linux/interrupt.h>
#include <linux/module.h> #include <linux/module.h>
#include <linux/of.h>
#include <asm/io.h> #include <asm/io.h>
#include <asm/irq.h> #include <asm/irq.h>
#include <asm/mpc8260.h> #include <asm/mpc8260.h>
...@@ -45,13 +47,12 @@ ...@@ -45,13 +47,12 @@
#include <sysdev/fsl_soc.h> #include <sysdev/fsl_soc.h>
static void cpm2_dpinit(void); static void cpm2_dpinit(void);
cpm_cpm2_t *cpmp; /* Pointer to comm processor space */ cpm_cpm2_t __iomem *cpmp; /* Pointer to comm processor space */
/* We allocate this here because it is used almost exclusively for /* We allocate this here because it is used almost exclusively for
* the communication processor devices. * the communication processor devices.
*/ */
cpm2_map_t *cpm2_immr; cpm2_map_t __iomem *cpm2_immr;
intctl_cpm2_t *cpm2_intctl;
#define CPM_MAP_SIZE (0x40000) /* 256k - the PQ3 reserve this amount #define CPM_MAP_SIZE (0x40000) /* 256k - the PQ3 reserve this amount
of space for CPM as it is larger of space for CPM as it is larger
...@@ -60,8 +61,11 @@ intctl_cpm2_t *cpm2_intctl; ...@@ -60,8 +61,11 @@ intctl_cpm2_t *cpm2_intctl;
void void
cpm2_reset(void) cpm2_reset(void)
{ {
cpm2_immr = (cpm2_map_t *)ioremap(CPM_MAP_ADDR, CPM_MAP_SIZE); #ifdef CONFIG_PPC_85xx
cpm2_intctl = cpm2_map(im_intctl); cpm2_immr = ioremap(CPM_MAP_ADDR, CPM_MAP_SIZE);
#else
cpm2_immr = ioremap(get_immrbase(), CPM_MAP_SIZE);
#endif
/* Reclaim the DP memory for our use. /* Reclaim the DP memory for our use.
*/ */
...@@ -91,7 +95,7 @@ cpm2_reset(void) ...@@ -91,7 +95,7 @@ cpm2_reset(void)
void void
cpm_setbrg(uint brg, uint rate) cpm_setbrg(uint brg, uint rate)
{ {
volatile uint *bp; u32 __iomem *bp;
/* This is good enough to get SMCs running..... /* This is good enough to get SMCs running.....
*/ */
...@@ -113,7 +117,8 @@ cpm_setbrg(uint brg, uint rate) ...@@ -113,7 +117,8 @@ cpm_setbrg(uint brg, uint rate)
void void
cpm2_fastbrg(uint brg, uint rate, int div16) cpm2_fastbrg(uint brg, uint rate, int div16)
{ {
volatile uint *bp; u32 __iomem *bp;
u32 val;
if (brg < 4) { if (brg < 4) {
bp = cpm2_map_size(im_brgc1, 16); bp = cpm2_map_size(im_brgc1, 16);
...@@ -123,10 +128,11 @@ cpm2_fastbrg(uint brg, uint rate, int div16) ...@@ -123,10 +128,11 @@ cpm2_fastbrg(uint brg, uint rate, int div16)
brg -= 4; brg -= 4;
} }
bp += brg; bp += brg;
*bp = ((BRG_INT_CLK / rate) << 1) | CPM_BRG_EN; val = ((BRG_INT_CLK / rate) << 1) | CPM_BRG_EN;
if (div16) if (div16)
*bp |= CPM_BRG_DIV16; val |= CPM_BRG_DIV16;
out_be32(bp, val);
cpm2_unmap(bp); cpm2_unmap(bp);
} }
...@@ -135,8 +141,8 @@ int cpm2_clk_setup(enum cpm_clk_target target, int clock, int mode) ...@@ -135,8 +141,8 @@ int cpm2_clk_setup(enum cpm_clk_target target, int clock, int mode)
int ret = 0; int ret = 0;
int shift; int shift;
int i, bits = 0; int i, bits = 0;
cpmux_t *im_cpmux; cpmux_t __iomem *im_cpmux;
u32 *reg; u32 __iomem *reg;
u32 mask = 7; u32 mask = 7;
u8 clk_map [24][3] = { u8 clk_map [24][3] = {
{CPM_CLK_FCC1, CPM_BRG5, 0}, {CPM_CLK_FCC1, CPM_BRG5, 0},
...@@ -228,13 +234,33 @@ static spinlock_t cpm_dpmem_lock; ...@@ -228,13 +234,33 @@ static spinlock_t cpm_dpmem_lock;
* until the memory subsystem goes up... */ * until the memory subsystem goes up... */
static rh_block_t cpm_boot_dpmem_rh_block[16]; static rh_block_t cpm_boot_dpmem_rh_block[16];
static rh_info_t cpm_dpmem_info; static rh_info_t cpm_dpmem_info;
static u8* im_dprambase; static u8 __iomem *im_dprambase;
static void cpm2_dpinit(void) static void cpm2_dpinit(void)
{ {
spin_lock_init(&cpm_dpmem_lock); struct resource r;
#ifdef CONFIG_PPC_CPM_NEW_BINDING
struct device_node *np;
np = of_find_compatible_node(NULL, NULL, "fsl,cpm2");
if (!np)
panic("Cannot find CPM2 node");
im_dprambase = ioremap(CPM_MAP_ADDR, CPM_DATAONLY_BASE + CPM_DATAONLY_SIZE); if (of_address_to_resource(np, 1, &r))
panic("Cannot get CPM2 resource 1");
of_node_put(np);
#else
r.start = CPM_MAP_ADDR;
r.end = r.start + CPM_DATAONLY_BASE + CPM_DATAONLY_SIZE - 1;
#endif
im_dprambase = ioremap(r.start, r.end - r.start + 1);
if (!im_dprambase)
panic("Cannot map DPRAM");
spin_lock_init(&cpm_dpmem_lock);
/* initialize the info header */ /* initialize the info header */
rh_init(&cpm_dpmem_info, 1, rh_init(&cpm_dpmem_info, 1,
...@@ -248,7 +274,7 @@ static void cpm2_dpinit(void) ...@@ -248,7 +274,7 @@ static void cpm2_dpinit(void)
* varies with the processor and the microcode patches activated. * varies with the processor and the microcode patches activated.
* But the following should be at least safe. * But the following should be at least safe.
*/ */
rh_attach_region(&cpm_dpmem_info, CPM_DATAONLY_BASE, CPM_DATAONLY_SIZE); rh_attach_region(&cpm_dpmem_info, 0, r.end - r.start + 1);
} }
/* This function returns an index into the DPRAM area. /* This function returns an index into the DPRAM area.
......
...@@ -48,7 +48,7 @@ ...@@ -48,7 +48,7 @@
#define CPM2_IRQ_PORTC15 48 #define CPM2_IRQ_PORTC15 48
#define CPM2_IRQ_PORTC0 63 #define CPM2_IRQ_PORTC0 63
static intctl_cpm2_t *cpm2_intctl; static intctl_cpm2_t __iomem *cpm2_intctl;
static struct irq_host *cpm2_pic_host; static struct irq_host *cpm2_pic_host;
#define NR_MASK_WORDS ((NR_IRQS + 31) / 32) #define NR_MASK_WORDS ((NR_IRQS + 31) / 32)
......
...@@ -107,7 +107,7 @@ ...@@ -107,7 +107,7 @@
/* Export the base address of the communication processor registers /* Export the base address of the communication processor registers
* and dual port ram. * and dual port ram.
*/ */
extern cpm_cpm2_t *cpmp; /* Pointer to comm processor */ extern cpm_cpm2_t __iomem *cpmp; /* Pointer to comm processor */
extern unsigned long cpm_dpalloc(uint size, uint align); extern unsigned long cpm_dpalloc(uint size, uint align);
extern int cpm_dpfree(unsigned long offset); extern int cpm_dpfree(unsigned long offset);
......
...@@ -23,22 +23,9 @@ ...@@ -23,22 +23,9 @@
#include <asm/mpc85xx.h> #include <asm/mpc85xx.h>
#endif #endif
#define cpm2_map(member) \ #define cpm2_map(member) (&cpm2_immr->member)
({ \ #define cpm2_map_size(member, size) (&cpm2_immr->member)
u32 offset = offsetof(cpm2_map_t, member); \ #define cpm2_unmap(addr) do {} while(0)
void *addr = ioremap (CPM_MAP_ADDR + offset, \
sizeof( ((cpm2_map_t*)0)->member)); \
addr; \
})
#define cpm2_map_size(member, size) \
({ \
u32 offset = offsetof(cpm2_map_t, member); \
void *addr = ioremap (CPM_MAP_ADDR + offset, size); \
addr; \
})
#define cpm2_unmap(addr) iounmap(addr)
#endif #endif
#ifdef CONFIG_8xx #ifdef CONFIG_8xx
......
...@@ -10,6 +10,8 @@ ...@@ -10,6 +10,8 @@
#ifndef __IMMAP_CPM2__ #ifndef __IMMAP_CPM2__
#define __IMMAP_CPM2__ #define __IMMAP_CPM2__
#include <linux/types.h>
/* System configuration registers. /* System configuration registers.
*/ */
typedef struct sys_82xx_conf { typedef struct sys_82xx_conf {
...@@ -642,7 +644,7 @@ typedef struct immap { ...@@ -642,7 +644,7 @@ typedef struct immap {
u8 res11[4096]; u8 res11[4096];
} cpm2_map_t; } cpm2_map_t;
extern cpm2_map_t *cpm2_immr; extern cpm2_map_t __iomem *cpm2_immr;
#endif /* __IMMAP_CPM2__ */ #endif /* __IMMAP_CPM2__ */
#endif /* __KERNEL__ */ #endif /* __KERNEL__ */
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