[src/trunk]: src/sys/dev/ic Remove the AHCI_QUIRK_SKIP_RESET quirk now that t...

[jmcneill <jmcneill%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/7e6ff26cf44a
branches:  trunk
changeset: 948385:7e6ff26cf44a
user:      jmcneill <jmcneill%NetBSD.org@localhost>
date:      Mon Dec 28 14:08:42 2020 +0000

description:
Remove the AHCI_QUIRK_SKIP_RESET quirk now that the underlying issue is
fixed.

diffstat:

 sys/arch/arm/nvidia/tegra_ahcisata.c |  5 ++---
 sys/dev/ic/ahcisata_core.c           |  8 ++------
 sys/dev/ic/ahcisatavar.h             |  5 ++---
 3 files changed, 6 insertions(+), 12 deletions(-)

diffs (81 lines):

diff -r a306ec887ed9 -r 7e6ff26cf44a sys/arch/arm/nvidia/tegra_ahcisata.c
--- a/sys/arch/arm/nvidia/tegra_ahcisata.c      Mon Dec 28 13:59:50 2020 +0000
+++ b/sys/arch/arm/nvidia/tegra_ahcisata.c      Mon Dec 28 14:08:42 2020 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: tegra_ahcisata.c,v 1.12 2018/12/14 12:29:22 skrll Exp $ */
+/* $NetBSD: tegra_ahcisata.c,v 1.13 2020/12/28 14:08:42 jmcneill Exp $ */
 
 /*-
  * Copyright (c) 2015 Jared D. McNeill <jmcneill%invisible.ca@localhost>
@@ -27,7 +27,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: tegra_ahcisata.c,v 1.12 2018/12/14 12:29:22 skrll Exp $");
+__KERNEL_RCSID(0, "$NetBSD: tegra_ahcisata.c,v 1.13 2020/12/28 14:08:42 jmcneill Exp $");
 
 #include <sys/param.h>
 #include <sys/bus.h>
@@ -202,7 +202,6 @@
                aprint_error(": couldn't map ahci registers: %d\n", error);
                return;
        }
-       sc->sc.sc_ahci_quirks = AHCI_QUIRK_SKIP_RESET;
 
        aprint_naive("\n");
        aprint_normal(": SATA\n");
diff -r a306ec887ed9 -r 7e6ff26cf44a sys/dev/ic/ahcisata_core.c
--- a/sys/dev/ic/ahcisata_core.c        Mon Dec 28 13:59:50 2020 +0000
+++ b/sys/dev/ic/ahcisata_core.c        Mon Dec 28 14:08:42 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: ahcisata_core.c,v 1.91 2020/12/28 11:05:54 jmcneill Exp $      */
+/*     $NetBSD: ahcisata_core.c,v 1.92 2020/12/28 14:08:42 jmcneill Exp $      */
 
 /*
  * Copyright (c) 2006 Manuel Bouyer.
@@ -26,7 +26,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: ahcisata_core.c,v 1.91 2020/12/28 11:05:54 jmcneill Exp $");
+__KERNEL_RCSID(0, "$NetBSD: ahcisata_core.c,v 1.92 2020/12/28 14:08:42 jmcneill Exp $");
 
 #include <sys/types.h>
 #include <sys/malloc.h>
@@ -872,9 +872,6 @@
                KASSERT(sc->sc_ahci_cap & AHCI_CAP_SPM);
        }
 
-       if (sc->sc_ahci_quirks & AHCI_QUIRK_SKIP_RESET)
-               goto skip_reset;
-
        /* polled command, assume interrupts are disabled */
 
        cmd_h = &achp->ahcic_cmdh[c_slot];
@@ -946,7 +943,6 @@
                goto end;
        }
 
-skip_reset:
        /*
         * wait 31s for BSY to clear
         * This should not be needed, but some controllers clear the
diff -r a306ec887ed9 -r 7e6ff26cf44a sys/dev/ic/ahcisatavar.h
--- a/sys/dev/ic/ahcisatavar.h  Mon Dec 28 13:59:50 2020 +0000
+++ b/sys/dev/ic/ahcisatavar.h  Mon Dec 28 14:08:42 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: ahcisatavar.h,v 1.25 2020/12/25 08:57:38 skrll Exp $   */
+/*     $NetBSD: ahcisatavar.h,v 1.26 2020/12/28 14:08:42 jmcneill Exp $        */
 
 /*
  * Copyright (c) 2006 Manuel Bouyer.
@@ -58,8 +58,7 @@
 #define AHCI_PCI_QUIRK_FORCE   __BIT(0)  /* force attach */
 #define AHCI_PCI_QUIRK_BAD64   __BIT(1)  /* broken 64-bit DMA */
 #define AHCI_QUIRK_BADPMP      __BIT(2)  /* broken PMP support, ignore */
-#define AHCI_QUIRK_SKIP_RESET  __BIT(4)  /* skip drive reset sequence */
-#define AHCI_QUIRK_BADNCQ      __BIT(5)  /* possibly broken NCQ support, ignore */
+#define AHCI_QUIRK_BADNCQ      __BIT(3)  /* possibly broken NCQ support, ignore */
 
        uint32_t sc_ahci_cap;   /* copy of AHCI_CAP */
        int sc_ncmds; /* number of command slots */