Date: Fri, 3 Jan 2014 21:38:33 +0000 (UTC) From: Ian Lepore <ian@FreeBSD.org> To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: svn commit: r260245 - head/sys/arm/ti/am335x Message-ID: <201401032138.s03LcX8x057357@svn.freebsd.org>
next in thread | raw e-mail | index | archive | help
Author: ian Date: Fri Jan 3 21:38:33 2014 New Revision: 260245 URL: http://svnweb.freebsd.org/changeset/base/260245 Log: Fix a typo that caused a loop to run beyond the end of the array it was searching. If you didn't configure a timer capture pin you'd get a data abort as it wandered into the weeds, now you get a nice warning message about your config, as originally intended. Modified: head/sys/arm/ti/am335x/am335x_dmtimer.c Modified: head/sys/arm/ti/am335x/am335x_dmtimer.c ============================================================================== --- head/sys/arm/ti/am335x/am335x_dmtimer.c Fri Jan 3 20:47:51 2014 (r260244) +++ head/sys/arm/ti/am335x/am335x_dmtimer.c Fri Jan 3 21:38:33 2014 (r260245) @@ -372,7 +372,7 @@ am335x_dmtimer_pps_init(device_t dev, st * yet from ti_scm.h. */ timer_num = 0; - for (i = 0; nitems(padinfo) && timer_num == 0; ++i) { + for (i = 0; i < nitems(padinfo) && timer_num == 0; ++i) { if (ti_scm_padconf_get(padinfo[i].ballname, &padmux, &padstate) == 0) { if (strcasecmp(padinfo[i].muxname, padmux) == 0 &&
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201401032138.s03LcX8x057357>