[src/trunk]: src/sys/arch/arm/broadcom A minimal change to prevent the Raspbe...

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

details:   https://anonhg.NetBSD.org/src/rev/43446a1d8486
branches:  trunk
changeset: 833055:43446a1d8486
user:      thorpej <thorpej%NetBSD.org@localhost>
date:      Thu Jun 07 05:07:28 2018 +0000

description:
A minimal change to prevent the Raspberry Pi i2c driver from
looping forever if the command buffer and data buffer are both
NULL.  XXX This driver needs an overhaul.

Also, preemptively return an error if we get a 10-bit address,
because we don't yet handle them correctly.

diffstat:

 sys/arch/arm/broadcom/bcm2835_bsc.c |  18 ++++++++++++++----
 1 files changed, 14 insertions(+), 4 deletions(-)

diffs (50 lines):

diff -r 9c5c450e6e37 -r 43446a1d8486 sys/arch/arm/broadcom/bcm2835_bsc.c
--- a/sys/arch/arm/broadcom/bcm2835_bsc.c       Thu Jun 07 00:51:41 2018 +0000
+++ b/sys/arch/arm/broadcom/bcm2835_bsc.c       Thu Jun 07 05:07:28 2018 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: bcm2835_bsc.c,v 1.11 2018/05/09 02:53:00 thorpej Exp $ */
+/*     $NetBSD: bcm2835_bsc.c,v 1.12 2018/06/07 05:07:28 thorpej Exp $ */
 
 /*
  * Copyright (c) 2012 Jonathan A. Kollasch
@@ -27,7 +27,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: bcm2835_bsc.c,v 1.11 2018/05/09 02:53:00 thorpej Exp $");
+__KERNEL_RCSID(0, "$NetBSD: bcm2835_bsc.c,v 1.12 2018/06/07 05:07:28 thorpej Exp $");
 
 #if defined(_KERNEL_OPT)
 #include "opt_kernhist.h"
@@ -242,8 +242,15 @@
        size_t len;
        size_t pos;
        int error = 0;
+       int whichbuf = 0;
        const bool isread = I2C_OP_READ_P(op);
 
+       /*
+        * XXX We don't do 10-bit addressing correctly yet.
+        */
+       if (addr > 0x7f)
+               return (ENOTSUP);
+
        flags |= I2C_F_POLL;
 
 #if 0
@@ -312,10 +319,13 @@
        KERNHIST_LOG(bsciichist, "flood bot %#jx %ju",
            (uintptr_t)buf, len, 0, 0);
 
-       if (buf == cmdbuf && !isread) {
+       if (whichbuf == 0 && !isread) {
+               KASSERT(buf == cmdbuf);
+               whichbuf++;
                buf = databuf;
                len = datalen;
-               goto flood_again;
+               if (len)
+                       goto flood_again;
        }
 
        do {