svn commit: r341811 - head/sys/dev/ahci

[Xin LI]

Author: delphij
Date: Tue Dec 11 05:10:22 2018
New Revision: 341811
URL: https://svnweb.freebsd.org/changeset/base/341811

Log:
  Remove questionable initialization for ICH8M, rely on BIOS to properly
  initialize the controller.
  
  According to the datasheet, the old code checks if port 2 (P2E, 0x4) was
  the only enabled port (except port 0, which was ignored by mask 0xfe),
  and issue a write to the PCS register to disable all but port 0, right
  before ahci_ctlr_reset.
  
  Some other operating systems would issue a port enable to all ports, but
  since the current code only does the special initialization for ICH8M,
  it entirely and rely on BIOS to do the right thing (the alternative
  would be https://reviews.freebsd.org/D18300?id=50922 , should we see
  reports that we really need to do it).
  
  Reviewed by:	mav
  MFC after:	3 months
  Differential Revision:	https://reviews.freebsd.org/D18300

Modified:
  head/sys/dev/ahci/ahci_pci.c

Modified: head/sys/dev/ahci/ahci_pci.c
==============================================================================
--- head/sys/dev/ahci/ahci_pci.c	Tue Dec 11 02:54:36 2018	(r341810)
+++ head/sys/dev/ahci/ahci_pci.c	Tue Dec 11 05:10:22 2018	(r341811)
@@ -358,10 +358,7 @@ static int
 ahci_pci_ctlr_reset(device_t dev)
 {
 
-	if (pci_read_config(dev, PCIR_DEVVENDOR, 4) == 0x28298086 &&
-	    (pci_read_config(dev, 0x92, 1) & 0xfe) == 0x04)
-		pci_write_config(dev, 0x92, 0x01, 1);
-	return ahci_ctlr_reset(dev);
+	return(ahci_ctlr_reset(dev));
 }
 
 static int