summaryrefslogtreecommitdiffstats
path: root/drivers/isdn/hardware
diff options
context:
space:
mode:
authorKarsten Keil <keil@b1-systems.de>2009-07-27 08:20:55 -0700
committerDavid S. Miller <davem@davemloft.net>2009-07-27 08:20:55 -0700
commit663a31ce5bbef2d14fa325023e48bf02b4249f27 (patch)
treeb1b0d69da31cad32b886f4db5ddd0a746026b2a9 /drivers/isdn/hardware
parent70034918930d2e5b68c09bced637228c50d9561a (diff)
downloadblackbird-op-linux-663a31ce5bbef2d14fa325023e48bf02b4249f27.tar.gz
blackbird-op-linux-663a31ce5bbef2d14fa325023e48bf02b4249f27.zip
mISDN: Fix wrong struct name in macro and clarifications
Based on comments from Joe Perches <joe@perches.com>. Thanks. Fix IOFUNC_MEMIO macro. WriteFiFo##name##_MIO use the wrong struct name, this was missed because the macro was only called with this name. Clarify with _func that the defined types are functions. Add names to the parameters for better understanding the purpose. Signed-off-by: Karsten Keil <keil@b1-systems.de> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/isdn/hardware')
-rw-r--r--drivers/isdn/hardware/mISDN/iohelper.h10
-rw-r--r--drivers/isdn/hardware/mISDN/ipac.h16
-rw-r--r--drivers/isdn/hardware/mISDN/isar.h8
3 files changed, 17 insertions, 17 deletions
diff --git a/drivers/isdn/hardware/mISDN/iohelper.h b/drivers/isdn/hardware/mISDN/iohelper.h
index c16a217301e8..b438981107ae 100644
--- a/drivers/isdn/hardware/mISDN/iohelper.h
+++ b/drivers/isdn/hardware/mISDN/iohelper.h
@@ -26,9 +26,9 @@
#ifndef _IOHELPER_H
#define _IOHELPER_H
-typedef u8 (read_reg_t)(void *, u8);
-typedef void (write_reg_t)(void *, u8, u8);
-typedef void (fifo_func_t)(void *, u8, u8 *, int);
+typedef u8 (read_reg_func)(void *hwp, u8 offset);
+typedef void (write_reg_func)(void *hwp, u8 offset, u8 value);
+typedef void (fifo_func)(void *hwp, u8 offset, u8 *datap, int size);
struct _ioport {
u32 port;
@@ -90,7 +90,7 @@ struct _ioport {
*dp++ = readb(((typ *)hw->adr) + off);\
} \
static void WriteFiFo##name##_MIO(void *p, u8 off, u8 *dp, int size) {\
- struct inf_hw *hw = p;\
+ struct hws *hw = p;\
while (size--)\
writeb(*dp++, ((typ *)hw->adr) + off);\
}
@@ -106,4 +106,4 @@ struct _ioport {
ASSIGN_FUNC(typ, IPAC, target);\
} while (0)
-#endif \ No newline at end of file
+#endif
diff --git a/drivers/isdn/hardware/mISDN/ipac.h b/drivers/isdn/hardware/mISDN/ipac.h
index f9601d55dbca..74a6ccf9065c 100644
--- a/drivers/isdn/hardware/mISDN/ipac.h
+++ b/drivers/isdn/hardware/mISDN/ipac.h
@@ -30,10 +30,10 @@ struct isac_hw {
u32 off; /* offset to isac regs */
char *name;
spinlock_t *hwlock; /* lock HW acccess */
- read_reg_t *read_reg;
- write_reg_t *write_reg;
- fifo_func_t *read_fifo;
- fifo_func_t *write_fifo;
+ read_reg_func *read_reg;
+ write_reg_func *write_reg;
+ fifo_func *read_fifo;
+ fifo_func *write_fifo;
int (*monitor)(void *, u32, u8 *, int);
void (*release)(struct isac_hw *);
int (*init)(struct isac_hw *);
@@ -73,10 +73,10 @@ struct ipac_hw {
spinlock_t *hwlock; /* lock HW acccess */
struct module *owner;
u32 type;
- read_reg_t *read_reg;
- write_reg_t *write_reg;
- fifo_func_t *read_fifo;
- fifo_func_t *write_fifo;
+ read_reg_func *read_reg;
+ write_reg_func *write_reg;
+ fifo_func *read_fifo;
+ fifo_func *write_fifo;
void (*release)(struct ipac_hw *);
int (*init)(struct ipac_hw *);
int (*ctrl)(struct ipac_hw *, u32, u_long);
diff --git a/drivers/isdn/hardware/mISDN/isar.h b/drivers/isdn/hardware/mISDN/isar.h
index 092351acd2ab..4a134acd44d0 100644
--- a/drivers/isdn/hardware/mISDN/isar.h
+++ b/drivers/isdn/hardware/mISDN/isar.h
@@ -47,10 +47,10 @@ struct isar_hw {
spinlock_t *hwlock; /* lock HW acccess */
char *name;
struct module *owner;
- read_reg_t *read_reg;
- write_reg_t *write_reg;
- fifo_func_t *read_fifo;
- fifo_func_t *write_fifo;
+ read_reg_func *read_reg;
+ write_reg_func *write_reg;
+ fifo_func *read_fifo;
+ fifo_func *write_fifo;
int (*ctrl)(void *, u32, u_long);
void (*release)(struct isar_hw *);
int (*init)(struct isar_hw *);
OpenPOWER on IntegriCloud