summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authormarcel <marcel@FreeBSD.org>2013-08-30 15:26:45 +0000
committermarcel <marcel@FreeBSD.org>2013-08-30 15:26:45 +0000
commiteab5b536cd4a17efa23c34b3183d0a2e133df859 (patch)
tree6b145f0b282a540bcf2dfd20080f71fbe1016380
parente60f4ea26b636b910cff32fdd856960ef1c58dc1 (diff)
downloadFreeBSD-src-eab5b536cd4a17efa23c34b3183d0a2e133df859.zip
FreeBSD-src-eab5b536cd4a17efa23c34b3183d0a2e133df859.tar.gz
A final test with unmodified code has shown that a delay of 150ms
is not giving us a 100% success rate. Bump the delay to 200ms as that seems to do the trick. Note that during testing the delay was added to uart_bus_attach() in uart_core.c. While having the delay in a different place can change the behaviour, it was not expected. Having to bump the delay with another 50ms could therefore be an indication that the problem can not be solved with delays. Reported by: kevlo@ Tested by: kevlo@
-rw-r--r--sys/dev/uart/uart_dev_ns8250.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/sys/dev/uart/uart_dev_ns8250.c b/sys/dev/uart/uart_dev_ns8250.c
index 2fad76c..211d113 100644
--- a/sys/dev/uart/uart_dev_ns8250.c
+++ b/sys/dev/uart/uart_dev_ns8250.c
@@ -465,7 +465,7 @@ ns8250_bus_attach(struct uart_softc *sc)
* accidental manner as before. More analysis is warranted, but
* at least now we fixed a known regression.
*/
- DELAY(150);
+ DELAY(200);
return (0);
}
OpenPOWER on IntegriCloud