d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Some strange hacks for the clpd6729:

Browse Source 
o It needs to have pcic_isa_intr intrrupt handler
o for pci interrupts, in the func interrupt handler it needs to check the isa
  registers rather than the pci ones for card present.
o better commentary for some of the strangeness of the 6729 on pci
o fix some crunchy comments to better reflect reality.

With this I almost have the WL200 working, but an interrupt storm
after attach is causing problems for reasons unknown.  This code
doesn't seem to break the normal clpd6729 case, and I'd like others
with 6729 cards to try to test it (there were some that were used for
external pccard slots in pci only systems).
This commit is contained in:
imp 2002-07-18 08:05:00 +00:00
parent c882dcd9d1
commit aa54e9aa16
1 changed files with 44 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

57
sys/pccard/pcic_pci.c
View File

@ -1085,18 +1085,24 @@ pcic_pci_probe(device_t dev)
* We only need to route interrupts when we're doing pci * We only need to route interrupts when we're doing pci
* parallel interrupt routing. * parallel interrupt routing.
* *
* Note: The CLPD6729 is a special case. See its init function * We use two different variables for the memory based and I/O
* for an explaination of ISA vs PCI interrupts. XXX Might be other * based cards, so the check here is a little more complex than
* special cases as well. * one would otherwise hope.
*
* XXX The bus code for PCI really should do this for us.
*/ */
if (pcic_intr_path == pcic_iw_pci && if ((pcic_intr_path == pcic_iw_pci &&
device_id != PCI_DEVICE_ID_PCIC_CLPD6729) { device_id != PCI_DEVICE_ID_PCIC_CLPD6729) ||
(pcic_pd6729_intr_path == pcic_iw_pci &&
device_id == PCI_DEVICE_ID_PCIC_CLPD6729)) {
rid = 0; rid = 0;
#ifdef __i386__ #ifdef __i386__
/* /*
* IRQ 0 is invalid on x86, but not other platforms. * IRQ 0 is invalid on x86, but not other platforms.
* If we have irq 0, then write 255 to force a new, non- * If we have irq 0, then write 255 to force a new, non-
* bogus one to be assigned. * bogus one to be assigned. I think that in -current
* the code in this ifdef may be obsolete with the new
* invalid mapping that we're doing in the pci layer -- imp
*/ */
if (pci_get_irq(dev) == 0) { if (pci_get_irq(dev) == 0) {
pci_set_irq(dev, 255); pci_set_irq(dev, 255);
@ -1176,9 +1182,8 @@ pcic_pci_print_config(device_t dev)
} }
/* /*
* General PCI based card dispatch routine. Right now * Generic pci interrupt attach routine. It tries to understand all parts,
* it only understands the Ricoh, CL-PD6832 and TI parts. It does * and do sane things for those parts it does not understand.
* try to do generic things with other parts.
*/ */
static int static int
pcic_pci_attach(device_t dev) pcic_pci_attach(device_t dev)
@ -1216,6 +1221,7 @@ pcic_pci_attach(device_t dev)
&sc->iorid, 0, ~0, 1, RF_ACTIVE | RF_SHAREABLE); &sc->iorid, 0, ~0, 1, RF_ACTIVE | RF_SHAREABLE);
if (sc->iores == NULL) if (sc->iores == NULL)
return (ENOMEM); return (ENOMEM);
sc->flags = PCIC_PD_POWER; sc->flags = PCIC_PD_POWER;
itm = pcic_pci_lookup(device_id, &pcic_pci_devs[0]); itm = pcic_pci_lookup(device_id, &pcic_pci_devs[0]);
for (i = 0; i < 2; i++) { for (i = 0; i < 2; i++) {
@ -1233,6 +1239,14 @@ pcic_pci_attach(device_t dev)
sc->csc_route = sc->func_route = pcic_pd6729_intr_path; sc->csc_route = sc->func_route = pcic_pd6729_intr_path;
if (itm) if (itm)
sc->flags = itm->flags; sc->flags = itm->flags;
/*
* We have to use the ISA interrupt routine for status
* changes since we don't have any "yenta" pci registers.
* We have to do this even when we're using pci type
* interrupts because on these cards the interrupts are
* cleared in the same way that the ISA cards clear them.
*/
intr = pcic_isa_intr;
} else { } else {
sc->memrid = CB_PCI_SOCKET_BASE; sc->memrid = CB_PCI_SOCKET_BASE;
sc->memres = bus_alloc_resource(dev, SYS_RES_MEMORY, sc->memres = bus_alloc_resource(dev, SYS_RES_MEMORY,
@ -1277,7 +1291,8 @@ pcic_pci_attach(device_t dev)
device_printf(dev, device_printf(dev,
"No PCI interrupt routed, trying ISA.\n"); "No PCI interrupt routed, trying ISA.\n");
} else { } else {
intr = pcic_pci_intr; if (intr == NULL)
intr = pcic_pci_intr;
irq = rman_get_start(r); irq = rman_get_start(r);
} }
} }
@ -1295,7 +1310,8 @@ pcic_pci_attach(device_t dev)
} }
device_printf(dev, device_printf(dev,
"Management interrupt on ISA IRQ %ld\n", irq); "Management interrupt on ISA IRQ %ld\n", irq);
intr = pcic_isa_intr; if (intr == NULL)
intr = pcic_isa_intr;
} else { } else {
sc->slot_poll = pcic_timeout; sc->slot_poll = pcic_timeout;
sc->timeout_ch = timeout(sc->slot_poll, sc, hz/2); sc->timeout_ch = timeout(sc->slot_poll, sc, hz/2);
@ -1410,9 +1426,24 @@ pcic_pci_func_intr(void *arg)
struct pcic_softc *sc = (struct pcic_softc *) arg; struct pcic_softc *sc = (struct pcic_softc *) arg;
struct pcic_slot *sp = &sc->slots[0]; struct pcic_slot *sp = &sc->slots[0];
u_int32_t stat; u_int32_t stat;
int doit = 0;
stat = bus_space_read_4(sp->bst, sp->bsh, CB_SOCKET_STATE); /*
if ((stat & CB_SS_CD) == 0 && sc->func_intr != 0) * The 6729 controller is a weird one, and we have to use
* the ISA registers to check to see if the card is there.
* Otherwise we look at the PCI state register to find out
* if the card is there.
*/
if (sp->controller == PCIC_PD6729) {
if ((sp->getb(sp, PCIC_STATUS) & PCIC_CD) == PCIC_CD)
doit = 1;
}
else {
stat = bus_space_read_4(sp->bst, sp->bsh, CB_SOCKET_STATE);
if ((stat & CB_SS_CD) == 0 && sc->func_intr != 0)
doit = 1;
}
if (doit && sc->func_intr != NULL)
sc->func_intr(sc->func_arg); sc->func_intr(sc->func_arg);
} }