On Tue, Feb 19, 2008 at 03:03:15PM +1100, Simon Burge wrote:
> > Notice that after bus_space_read_4(), it does bus_space_unmap()
> > followed by return (3) regardless of what the AHCI_GHC register
> > contained. I think that "return (3)" should be "return (ret)", that
> > way it will only return 3 if AHCI_GHC & AHCI_GHC_AE != 0. This bug has
> > been there since rev. 1.1, but would have been uncovered by rev. 1.3 on
> > Feb. 11th, which added the check against
> > PCI_SUBCLASS_MASS_STORAGE_RAID.
> Using "return (ret)" fixes this for me - svwsata(4) attaches and
> ahcisata(4) doesn't attempt to.
> svwsata0 at pci1 dev 14 function 0
> svwsata0: ServerWorks HT-1000 SATA Controller (rev. 0x00)
> svwsata0: using irq 11 for native-PCI interrupt
> atabus0 at svwsata0 channel 0
> atabus1 at svwsata0 channel 1
> atabus2 at svwsata0 channel 2
> atabus3 at svwsata0 channel 3
> svwsata0 port 0: device present, speed: 1.5Gb/s
> wd0 at atabus0 drive 0: <FUJITSU MHT2080BH>
> wd0: 76319 MB, 155061 cyl, 16 head, 63 sec, 512 bytes/sect x 156301488
> svwsata0 port 1: PHY offline
> svwsata0 port 2: PHY offline
> svwsata0 port 3: PHY offline
> Manuel - OK for John to commit that fix?
Sure ! thanks !
Manuel Bouyer, LIP6, Universite Paris VI. [email protected]
NetBSD: 26 ans d'experience feront toujours la difference