Re: ATA CS5530 (cyrix) driver panic (ata_cyrix_setmode())

2003-03-09 Thread Bruce R. Montague
Soeren Schmidt wrote: > the following patch should solve that: Yes, it does, Thanks! All's well that... ends. - bruce To Unsubscribe: send mail to [EMAIL PROTECTED] with "unsubscribe freebsd-current" in the body of the message

Re: ATA CS5530 (cyrix) driver panic (ata_cyrix_setmode())

2003-03-09 Thread Soeren Schmidt
It seems Bruce R. Montague wrote: > > /* is busmastering supported ? */ > if ((cmd & (PCIM_CMD_PORTEN | PCIM_CMD_BUSMASTEREN)) == > (PCIM_CMD_PORTEN | PCIM_CMD_BUSMASTEREN)) { > > failed because "cmd" was 0x01 instead of 0x05 (the > PORTEN | BUSMASTEREN is 0x05). The 5530 datasheet > (well, SC120

Re: ATA CS5530 (cyrix) driver panic (ata_cyrix_setmode())

2003-03-09 Thread Bruce R. Montague
Hi, regarding the CS5530 driver initialization (ata cyrix) panic, the problem of channel "dma" and "r_bmio" fields being 0 is due to the hardware PCI config cmd register not having the expected value. In "ata-pci.c / ata_pci_attach()" the code fragment: /* is busmastering supported ? */ if ((cmd

Re: ATA CS5530 (cyrix) driver panic (ata_cyrix_setmode())

2003-03-09 Thread Julian Elischer
Hi I didn;t see the original problem, just the reply.. Just one thing to be aware of when using the 5530. I used a 5530 on the Interjet-II. It has a terrible bug where it fails if the data being transferred to/from the disk by DMA is not alligned on a 16 byte boundary. programs such as newfs and

Re: ATA CS5530 (cyrix) driver panic (ata_cyrix_setmode())

2003-03-09 Thread Bruce R. Montague
Hi, re the ata driver and CS5530, Soren Schmidt asked: > Could you please dump the pci reg 0x20 (with pciconf) and > verify that it is endeed 0 ? Ok (it's _not_ 0): - geode# pciconf -r pci0:18:2 0x20 fc01 geode# pciconf -r pci0:18:2 32 fc01 - Is this 0x20 on F2, F2BAR4? I

Re: ATA CS5530 (cyrix) driver panic (ata_cyrix_setmode())

2003-03-09 Thread Soeren Schmidt
It seems Bruce R. Montague wrote: > > Routine "ata_cyrix_setmode()" in "ata-chipset.c" > appears to be assuming that "channel->dma" has a > valid pointer to a "struct ata_dma_funcs", and > that "channel->r_bmio" is a valid bus resource > id. This is not the case, both "channel->dma" and > "channe

Re: ATA CS5530 (cyrix) driver panic (ata_cyrix_setmode())

2003-03-09 Thread Soeren Schmidt
It seems Bruce R. Montague wrote: > I'll look into it, thanks for reporting! > Hi, the current -current ata driver panics at boot > when using the CS5530 (National GX1, ex-"cyrix"). > This driver worked in the past on -current, likely > up until the major rework that appears to be > underway as

ATA CS5530 (cyrix) driver panic (ata_cyrix_setmode())

2003-03-09 Thread Bruce R. Montague
Hi, the current -current ata driver panics at boot when using the CS5530 (National GX1, ex-"cyrix"). This driver worked in the past on -current, likely up until the major rework that appears to be underway as of 20-Feb-2003 (that is, the creation of "ata-chipset.c", etc.) Routine "ata_cyrix_set