Page MenuHomeFreeBSD

ahci(4): Add another device ID for Marvell 9128 SATA controller: 0x91a3
ClosedPublic

Authored by michaelo on Fri, Apr 17, 9:19 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, May 12, 10:23 AM
Unknown Object (File)
Sun, May 10, 11:02 AM
Unknown Object (File)
Sun, May 10, 9:44 AM
Unknown Object (File)
Sat, May 9, 11:15 PM
Unknown Object (File)
Sat, May 9, 11:10 PM
Unknown Object (File)
Sat, May 9, 6:24 PM
Unknown Object (File)
Sat, May 9, 6:18 PM
Unknown Object (File)
Sat, May 9, 12:07 PM
Subscribers

Details

Summary

This was encountered on a Gigabyte GA-P55A-UD4 motherboard identifying itself
as:
ahci0@pci0:2:0:0: class=0x01018f rev=0x11 hdr=0x00 vendor=0x1b4b device=0x91a3 subvendor=0x1458 subdevice=0xb000

vendor     = 'Marvell Technology Group Ltd.'
class      = mass storage
subclass   = ATA

The physical chip has "88SE9128-NAA2" printed on it.

Similar code has been in Linux for a long time:
https://github.com/torvalds/linux/commit/50be5e3657cd2851a297dc0b3fd459f25829d29b

Co-authored-by: Michael Osipov <michaelo@FreeBSD.org>
PR: 288526

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jhb added inline comments.
sys/dev/ahci/ahci_pci.c
299

I would move this down to here to keep the IDs sorted.

This revision is now accepted and ready to land.Thu, Apr 23, 4:05 PM
sys/dev/ahci/ahci_pci.c
299

I see, I added it up there on purpose to retain the order of the device description, but I am fine with either.

This revision now requires review to proceed.Thu, Apr 23, 4:17 PM
This revision is now accepted and ready to land.Thu, Apr 23, 4:39 PM