Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 10 Jul 2010 13:46:14 +0000 (UTC)
From:      Alexander Motin <mav@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r209872 - in head/sys/dev/ata: . chipsets
Message-ID:  <201007101346.o6ADkEOx070799@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: mav
Date: Sat Jul 10 13:46:14 2010
New Revision: 209872
URL: http://svn.freebsd.org/changeset/base/209872

Log:
  Make hw.ata.ata_dma_check_80pin tunable affect not only device side, but
  also controller side cable checks. Make respective sysctl writable.
  
  PR:		kern/143462

Modified:
  head/sys/dev/ata/ata-all.c
  head/sys/dev/ata/chipsets/ata-acerlabs.c
  head/sys/dev/ata/chipsets/ata-amd.c
  head/sys/dev/ata/chipsets/ata-highpoint.c
  head/sys/dev/ata/chipsets/ata-intel.c
  head/sys/dev/ata/chipsets/ata-ite.c
  head/sys/dev/ata/chipsets/ata-jmicron.c
  head/sys/dev/ata/chipsets/ata-marvell.c
  head/sys/dev/ata/chipsets/ata-promise.c
  head/sys/dev/ata/chipsets/ata-siliconimage.c
  head/sys/dev/ata/chipsets/ata-sis.c

Modified: head/sys/dev/ata/ata-all.c
==============================================================================
--- head/sys/dev/ata/ata-all.c	Sat Jul 10 11:40:31 2010	(r209871)
+++ head/sys/dev/ata/ata-all.c	Sat Jul 10 13:46:14 2010	(r209872)
@@ -105,7 +105,7 @@ SYSCTL_INT(_hw_ata, OID_AUTO, ata_dma, C
 	   "ATA disk DMA mode control");
 TUNABLE_INT("hw.ata.ata_dma_check_80pin", &ata_dma_check_80pin);
 SYSCTL_INT(_hw_ata, OID_AUTO, ata_dma_check_80pin,
-	   CTLFLAG_RDTUN, &ata_dma_check_80pin, 1,
+	   CTLFLAG_RW, &ata_dma_check_80pin, 1,
 	   "Check for 80pin cable before setting ATA DMA mode");
 TUNABLE_INT("hw.ata.atapi_dma", &atapi_dma);
 SYSCTL_INT(_hw_ata, OID_AUTO, atapi_dma, CTLFLAG_RDTUN, &atapi_dma, 0,

Modified: head/sys/dev/ata/chipsets/ata-acerlabs.c
==============================================================================
--- head/sys/dev/ata/chipsets/ata-acerlabs.c	Sat Jul 10 11:40:31 2010	(r209871)
+++ head/sys/dev/ata/chipsets/ata-acerlabs.c	Sat Jul 10 13:46:14 2010	(r209872)
@@ -286,7 +286,7 @@ ata_ali_setmode(device_t dev, int target
         mode = min(mode, ctlr->chip->max_dma);
 
 	if (ctlr->chip->cfg2 & ALI_NEW && ctlr->chip->chiprev < 0xc7) {
-		if (mode > ATA_UDMA2 &&
+		if (ata_dma_check_80pin && mode > ATA_UDMA2 &&
 		    pci_read_config(parent, 0x4a, 1) & (1 << ch->unit)) {
 			ata_print_cable(dev, "controller");
 			mode = ATA_UDMA2;

Modified: head/sys/dev/ata/chipsets/ata-amd.c
==============================================================================
--- head/sys/dev/ata/chipsets/ata-amd.c	Sat Jul 10 11:40:31 2010	(r209871)
+++ head/sys/dev/ata/chipsets/ata-amd.c	Sat Jul 10 13:46:14 2010	(r209872)
@@ -119,7 +119,7 @@ ata_amd_setmode(device_t dev, int target
 
 	mode = min(mode, ctlr->chip->max_dma);
 	if (ctlr->chip->cfg1 & AMD_CABLE) {
-		if (mode > ATA_UDMA2 &&
+		if (ata_dma_check_80pin && mode > ATA_UDMA2 &&
 		    !(pci_read_config(parent, 0x42, 1) & (1 << devno))) {
 			ata_print_cable(dev, "controller");
 			mode = ATA_UDMA2;

Modified: head/sys/dev/ata/chipsets/ata-highpoint.c
==============================================================================
--- head/sys/dev/ata/chipsets/ata-highpoint.c	Sat Jul 10 11:40:31 2010	(r209871)
+++ head/sys/dev/ata/chipsets/ata-highpoint.c	Sat Jul 10 13:46:14 2010	(r209872)
@@ -216,7 +216,7 @@ ata_highpoint_check_80pin(device_t dev, 
     res = pci_read_config(parent, 0x5a, 1) & (ch->unit ? 0x1:0x2);
     pci_write_config(parent, reg, val, 1);
 
-    if (mode > ATA_UDMA2 && res) {
+    if (ata_dma_check_80pin && mode > ATA_UDMA2 && res) {
 	ata_print_cable(dev, "controller");
 	mode = ATA_UDMA2;
     }

Modified: head/sys/dev/ata/chipsets/ata-intel.c
==============================================================================
--- head/sys/dev/ata/chipsets/ata-intel.c	Sat Jul 10 11:40:31 2010	(r209871)
+++ head/sys/dev/ata/chipsets/ata-intel.c	Sat Jul 10 13:46:14 2010	(r209872)
@@ -336,7 +336,8 @@ ata_intel_new_setmode(device_t dev, int 
 	u_int8_t utimings[] = { 0x00, 0x01, 0x02, 0x01, 0x02, 0x01, 0x02 };
 
 	mode = min(mode, ctlr->chip->max_dma);
-	if (mode > ATA_UDMA2 && !(reg54 & (0x10 << devno))) {
+	if (ata_dma_check_80pin && mode > ATA_UDMA2 &&
+	    !(reg54 & (0x10 << devno))) {
 		ata_print_cable(dev, "controller");
 		mode = ATA_UDMA2;
 	}

Modified: head/sys/dev/ata/chipsets/ata-ite.c
==============================================================================
--- head/sys/dev/ata/chipsets/ata-ite.c	Sat Jul 10 11:40:31 2010	(r209871)
+++ head/sys/dev/ata/chipsets/ata-ite.c	Sat Jul 10 13:46:14 2010	(r209872)
@@ -139,7 +139,8 @@ ata_ite_821x_setmode(device_t dev, int t
 
 	mode = min(mode, ctlr->chip->max_dma);
 	/* check the CBLID bits for 80 conductor cable detection */
-	if (mode > ATA_UDMA2 && (pci_read_config(parent, 0x40, 2) &
+	if (ata_dma_check_80pin && mode > ATA_UDMA2 &&
+	    (pci_read_config(parent, 0x40, 2) &
 			     (ch->unit ? (1<<3) : (1<<2)))) {
 		ata_print_cable(dev, "controller");
 		mode = ATA_UDMA2;
@@ -186,7 +187,8 @@ ata_ite_8213_setmode(device_t dev, int t
 
 	mode = min(mode, ctlr->chip->max_dma);
 
-	if (mode > ATA_UDMA2 && !(reg54 & (0x10 << target))) {
+	if (ata_dma_check_80pin && mode > ATA_UDMA2 &&
+	    !(reg54 & (0x10 << target))) {
 		ata_print_cable(dev, "controller");
 		mode = ATA_UDMA2;
 	}

Modified: head/sys/dev/ata/chipsets/ata-jmicron.c
==============================================================================
--- head/sys/dev/ata/chipsets/ata-jmicron.c	Sat Jul 10 11:40:31 2010	(r209871)
+++ head/sys/dev/ata/chipsets/ata-jmicron.c	Sat Jul 10 13:46:14 2010	(r209872)
@@ -149,7 +149,8 @@ ata_jmicron_setmode(device_t dev, int ta
 
 	mode = min(mode, ctlr->chip->max_dma);
 	/* check for 80pin cable present */
-	if (mode > ATA_UDMA2 && pci_read_config(dev, 0x40, 1) & 0x08) {
+	if (ata_dma_check_80pin && mode > ATA_UDMA2 &&
+	    pci_read_config(dev, 0x40, 1) & 0x08) {
 		ata_print_cable(dev, "controller");
 		mode = ATA_UDMA2;
 	}

Modified: head/sys/dev/ata/chipsets/ata-marvell.c
==============================================================================
--- head/sys/dev/ata/chipsets/ata-marvell.c	Sat Jul 10 11:40:31 2010	(r209871)
+++ head/sys/dev/ata/chipsets/ata-marvell.c	Sat Jul 10 13:46:14 2010	(r209872)
@@ -188,7 +188,8 @@ ata_marvell_setmode(device_t dev, int ta
 
 	mode = min(mode, ctlr->chip->max_dma);
 	/* Check for 80pin cable present. */
-	if (mode > ATA_UDMA2 && ATA_IDX_INB(ch, ATA_BMDEVSPEC_0) & 0x01) {
+	if (ata_dma_check_80pin && mode > ATA_UDMA2 &&
+	    ATA_IDX_INB(ch, ATA_BMDEVSPEC_0) & 0x01) {
 		ata_print_cable(dev, "controller");
 		mode = ATA_UDMA2;
 	}

Modified: head/sys/dev/ata/chipsets/ata-promise.c
==============================================================================
--- head/sys/dev/ata/chipsets/ata-promise.c	Sat Jul 10 11:40:31 2010	(r209871)
+++ head/sys/dev/ata/chipsets/ata-promise.c	Sat Jul 10 13:46:14 2010	(r209872)
@@ -470,7 +470,8 @@ ata_promise_setmode(device_t dev, int ta
     switch (ctlr->chip->cfg1) {
     case PR_OLD:
     case PR_NEW:
-	if (mode > ATA_UDMA2 && (pci_read_config(parent, 0x50, 2) &
+	if (ata_dma_check_80pin && mode > ATA_UDMA2 &&
+	    (pci_read_config(parent, 0x50, 2) &
 				 (ch->unit ? 1 << 11 : 1 << 10))) {
 	    ata_print_cable(dev, "controller");
 	    mode = ATA_UDMA2;
@@ -479,7 +480,7 @@ ata_promise_setmode(device_t dev, int ta
 
     case PR_TX:
 	ATA_IDX_OUTB(ch, ATA_BMDEVSPEC_0, 0x0b);
-	if (mode > ATA_UDMA2 &&
+	if (ata_dma_check_80pin && mode > ATA_UDMA2 &&
 	    ATA_IDX_INB(ch, ATA_BMDEVSPEC_1) & 0x04) {
 	    ata_print_cable(dev, "controller");
 	    mode = ATA_UDMA2;
@@ -487,7 +488,7 @@ ata_promise_setmode(device_t dev, int ta
 	break;
    
     case PR_MIO:
-	if (mode > ATA_UDMA2 &&
+	if (ata_dma_check_80pin && mode > ATA_UDMA2 &&
 	    (ATA_INL(ctlr->r_res2,
 		     (ctlr->chip->cfg2 & PR_SX4X ? 0x000c0260 : 0x0260) +
 		     (ch->unit << 7)) & 0x01000000)) {

Modified: head/sys/dev/ata/chipsets/ata-siliconimage.c
==============================================================================
--- head/sys/dev/ata/chipsets/ata-siliconimage.c	Sat Jul 10 11:40:31 2010	(r209871)
+++ head/sys/dev/ata/chipsets/ata-siliconimage.c	Sat Jul 10 13:46:14 2010	(r209872)
@@ -389,7 +389,8 @@ ata_sii_setmode(device_t dev, int target
 	mode = min(mode, ctlr->chip->max_dma);
 
 	if (ctlr->chip->cfg2 & SII_SETCLK) {
-	    if (mode > ATA_UDMA2 && (pci_read_config(parent, 0x79, 1) &
+	    if (ata_dma_check_80pin && mode > ATA_UDMA2 &&
+		(pci_read_config(parent, 0x79, 1) &
 				 (ch->unit ? 0x02 : 0x01))) {
 		ata_print_cable(dev, "controller");
 		mode = ATA_UDMA2;

Modified: head/sys/dev/ata/chipsets/ata-sis.c
==============================================================================
--- head/sys/dev/ata/chipsets/ata-sis.c	Sat Jul 10 11:40:31 2010	(r209871)
+++ head/sys/dev/ata/chipsets/ata-sis.c	Sat Jul 10 13:46:14 2010	(r209872)
@@ -243,13 +243,13 @@ ata_sis_setmode(device_t dev, int target
 	mode = min(mode, ctlr->chip->max_dma);
 
 	if (ctlr->chip->cfg1 == SIS_133NEW) {
-		if (mode > ATA_UDMA2 &&
+		if (ata_dma_check_80pin && mode > ATA_UDMA2 &&
 		        pci_read_config(parent, ch->unit ? 0x52 : 0x50,2) & 0x8000) {
 		        ata_print_cable(dev, "controller");
 		        mode = ATA_UDMA2;
 		}
 	} else {
-		if (mode > ATA_UDMA2 &&
+		if (ata_dma_check_80pin && mode > ATA_UDMA2 &&
 		    pci_read_config(parent, 0x48, 1)&(ch->unit ? 0x20 : 0x10)) {
 		    ata_print_cable(dev, "controller");
 		    mode = ATA_UDMA2;



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201007101346.o6ADkEOx070799>