diff options
author | Shaun Ruffell <sruffell@digium.com> | 2011-02-15 16:02:33 -0600 |
---|---|---|
committer | Shaun Ruffell <sruffell@digium.com> | 2011-04-15 14:18:59 -0500 |
commit | 6a143c104ac4c058faceffba024f848cbf08fa87 (patch) | |
tree | 70d393762968a47ea259149e0416a759d1495ca4 /drivers | |
parent | ba8abe97cc5a1d7973284b254ef9f336d0bdf49e (diff) |
wctdm24xxp: Drop 'flags' from 'struct wctdm_module'.
The voicebus cards only support the S110M FXS modules which are based on
the 3215. The module flags member was only used to hold whether we were
dealing with a 3210 or 3215 SLIC, so we can drop it since we always know
we'll have a 3215 based SLIC.
Signed-off-by: Shaun Ruffell <sruffell@digium.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/dahdi/wctdm24xxp/base.c | 38 | ||||
-rw-r--r-- | drivers/dahdi/wctdm24xxp/wctdm24xxp.h | 4 |
2 files changed, 13 insertions, 29 deletions
diff --git a/drivers/dahdi/wctdm24xxp/base.c b/drivers/dahdi/wctdm24xxp/base.c index 78275fd..eeb1c79 100644 --- a/drivers/dahdi/wctdm24xxp/base.c +++ b/drivers/dahdi/wctdm24xxp/base.c @@ -1202,12 +1202,11 @@ wctdm_proslic_setreg_indirect(struct wctdm *wc, struct wctdm_module *const mod, unsigned char address, unsigned short data) { int res = -1; - /* Translate 3215 addresses */ - if (mod->flags & FLAG_3215) { - address = translate_3215(address); - if (address == 255) - return 0; - } + + address = translate_3215(address); + if (address == 255) + return 0; + if (!wait_access(wc, mod)) { wctdm_setreg(wc, mod, IDA_LO, (u8)(data & 0xFF)); wctdm_setreg(wc, mod, IDA_HI, (u8)((data & 0xFF00)>>8)); @@ -1223,12 +1222,11 @@ wctdm_proslic_getreg_indirect(struct wctdm *wc, struct wctdm_module *const mod, { int res = -1; char *p=NULL; - /* Translate 3215 addresses */ - if (mod->flags & FLAG_3215) { - address = translate_3215(address); - if (address == 255) - return 0; - } + + address = translate_3215(address); + if (address == 255) + return 0; + if (!wait_access(wc, mod)) { wctdm_setreg(wc, mod, IAA, address); if (!wait_access(wc, mod)) { @@ -1276,8 +1274,7 @@ wctdm_proslic_verify_indirect_regs(struct wctdm *wc, struct wctdm_module *mod) } initial = indirect_regs[i].initial; - if ((j != initial) && (!(mod->flags & FLAG_3215) || - (indirect_regs[i].altaddr != 255))) { + if ((j != initial) && (indirect_regs[i].altaddr != 255)) { dev_notice(&wc->vb.pdev->dev, "!!!!!!! %s iREG %X = %X should be %X\n", indirect_regs[i].name, @@ -2199,16 +2196,6 @@ wctdm_proslic_insane(struct wctdm *wc, struct wctdm_module *const mod) return -1; } - if ((blah & 0xf) < 2) { - dev_info(&wc->vb.pdev->dev, - "ProSLIC 3210 version %d is too old\n", blah & 0xf); - return -1; - } - - if (wctdm_getreg(wc, mod, 1) & 0x80) - /* ProSLIC 3215, not a 3210 */ - mod->flags |= FLAG_3215; - blah = wctdm_getreg(wc, mod, 8); if (blah != 0x2) { dev_notice(&wc->vb.pdev->dev, @@ -3935,7 +3922,7 @@ static struct wctdm_span *wctdm_init_span(struct wctdm *wc, int spanno, int chan snprintf(s->span.desc, sizeof(s->span.desc) - 1, "%s Board %d", wc->desc->name, wc->pos + 1); snprintf(s->span.location, sizeof(s->span.location) - 1, "PCI%s Bus %02d Slot %02d", - (wc->mods[0].flags & FLAG_EXPRESS) ? " Express" : "", + (wc->desc->flags & FLAG_EXPRESS) ? " Express" : "", pdev->bus->number, PCI_SLOT(pdev->devfn) + 1); s->span.manufacturer = "Digium"; strncpy(s->span.devicetype, wc->desc->name, sizeof(s->span.devicetype) - 1); @@ -4902,7 +4889,6 @@ __wctdm_init_one(struct pci_dev *pdev, const struct pci_device_id *ent) wc->companding = DAHDI_LAW_DEFAULT; for (i = 0; i < NUM_MODULES; i++) { - wc->mods[i].flags = wc->desc->flags; wc->mods[i].dacssrc = -1; wc->mods[i].card = i; } diff --git a/drivers/dahdi/wctdm24xxp/wctdm24xxp.h b/drivers/dahdi/wctdm24xxp/wctdm24xxp.h index 076f835..9054846 100644 --- a/drivers/dahdi/wctdm24xxp/wctdm24xxp.h +++ b/drivers/dahdi/wctdm24xxp/wctdm24xxp.h @@ -50,8 +50,7 @@ #define OHT_TIMER 6000 /* How long after RING to retain OHT */ -#define FLAG_3215 (1 << 0) -#define FLAG_EXPRESS (1 << 1) +#define FLAG_EXPRESS (1 << 0) #define EFRAME_SIZE 108L #define EFRAME_GAP 20L @@ -228,7 +227,6 @@ struct wctdm_module { enum module_type type; int sethook; /* pending hook state command */ int dacssrc; - int flags; /* bitmap of board-specific + module-specific flags */ int altcs; u8 card; |