Module Name: src
Committed By: mrg
Date: Fri Sep 3 01:23:33 UTC 2021
Modified Files:
src/sys/arch/arm/rockchip: rk_platform.c
src/sys/dev/ic: ahcisata_core.c
Log Message:
revert "make ahcisata(4) work on rk3399 (rockpro64)" from 23-06-2021.
this problem is known to affect more than AHCI, and a more general
solution has been commited now.
To generate a diff of this commit:
cvs rdiff -u -r1.13 -r1.14 src/sys/arch/arm/rockchip/rk_platform.c
cvs rdiff -u -r1.100 -r1.101 src/sys/dev/ic/ahcisata_core.c
Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.
Modified files:
Index: src/sys/arch/arm/rockchip/rk_platform.c
diff -u src/sys/arch/arm/rockchip/rk_platform.c:1.13 src/sys/arch/arm/rockchip/rk_platform.c:1.14
--- src/sys/arch/arm/rockchip/rk_platform.c:1.13 Wed Jun 23 00:56:41 2021
+++ src/sys/arch/arm/rockchip/rk_platform.c Fri Sep 3 01:23:33 2021
@@ -1,4 +1,4 @@
-/* $NetBSD: rk_platform.c,v 1.13 2021/06/23 00:56:41 mrg Exp $ */
+/* $NetBSD: rk_platform.c,v 1.14 2021/09/03 01:23:33 mrg Exp $ */
/*-
* Copyright (c) 2018 Jared McNeill <[email protected]>
@@ -31,7 +31,7 @@
#include "opt_console.h"
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: rk_platform.c,v 1.13 2021/06/23 00:56:41 mrg Exp $");
+__KERNEL_RCSID(0, "$NetBSD: rk_platform.c,v 1.14 2021/09/03 01:23:33 mrg Exp $");
#include <sys/param.h>
#include <sys/bus.h>
@@ -39,7 +39,6 @@ __KERNEL_RCSID(0, "$NetBSD: rk_platform.
#include <sys/device.h>
#include <sys/termios.h>
-#include <prop/proplib.h>
#include <dev/fdt/fdtvar.h>
#include <arm/fdt/arm_fdtvar.h>
@@ -72,17 +71,6 @@ rk_platform_init_attach_args(struct fdt_
static void
rk_platform_device_register(device_t self, void *aux)
{
- prop_dictionary_t dict = device_properties(self);
-
- if (device_is_a(self, "ahcisata")) {
- /*
- * Marvel 9230 AHCI SATA controllers take between 1213 and 1216
- * milliseconds to reset, exceeding the AHCI spec of 1000.
- */
- if (!prop_dictionary_set_uint32(dict, "ahci-reset-ms", 2000))
- printf("%s: Failed to set \"ahci-reset-ms\" property"
- " on ahcisata\n", __func__);
- }
}
static void
Index: src/sys/dev/ic/ahcisata_core.c
diff -u src/sys/dev/ic/ahcisata_core.c:1.100 src/sys/dev/ic/ahcisata_core.c:1.101
--- src/sys/dev/ic/ahcisata_core.c:1.100 Sat Aug 7 16:19:11 2021
+++ src/sys/dev/ic/ahcisata_core.c Fri Sep 3 01:23:33 2021
@@ -1,4 +1,4 @@
-/* $NetBSD: ahcisata_core.c,v 1.100 2021/08/07 16:19:11 thorpej Exp $ */
+/* $NetBSD: ahcisata_core.c,v 1.101 2021/09/03 01:23:33 mrg Exp $ */
/*
* Copyright (c) 2006 Manuel Bouyer.
@@ -26,7 +26,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: ahcisata_core.c,v 1.100 2021/08/07 16:19:11 thorpej Exp $");
+__KERNEL_RCSID(0, "$NetBSD: ahcisata_core.c,v 1.101 2021/09/03 01:23:33 mrg Exp $");
#include <sys/types.h>
#include <sys/malloc.h>
@@ -144,31 +144,19 @@ static int
ahci_reset(struct ahci_softc *sc)
{
int i;
- uint32_t timeout_ms = 1000; /* default to 1s timeout */
- prop_dictionary_t dict;
/* reset controller */
AHCI_WRITE(sc, AHCI_GHC, AHCI_GHC_HR);
-
- /* some systems (rockchip rk3399) need extra reset time for ahcisata. */
- dict = device_properties(sc->sc_atac.atac_dev);
- if (dict)
- prop_dictionary_get_uint32(dict, "ahci-reset-ms", &timeout_ms);
-
- /* wait for reset to complete */
- for (i = 0; i < timeout_ms; i++) {
+ /* wait up to 1s for reset to complete */
+ for (i = 0; i < 1000; i++) {
delay(1000);
if ((AHCI_READ(sc, AHCI_GHC) & AHCI_GHC_HR) == 0)
break;
}
- if ((AHCI_READ(sc, AHCI_GHC) & AHCI_GHC_HR) != 0) {
- aprint_error_dev(sc->sc_atac.atac_dev, "reset failed\n");
+ if ((AHCI_READ(sc, AHCI_GHC) & AHCI_GHC_HR)) {
+ aprint_error("%s: reset failed\n", AHCINAME(sc));
return -1;
}
- if (i > 1000) {
- aprint_normal_dev(sc->sc_atac.atac_dev,
- "reset took %d milliseconds\n", i);
- }
/* enable ahci mode */
ahci_enable(sc);
