svn commit: r271594 - head/sys/arm/arm
Ian Lepore
ian at FreeBSD.org
Sun Sep 14 17:36:57 UTC 2014
Author: ian
Date: Sun Sep 14 17:36:57 2014
New Revision: 271594
URL: http://svnweb.freebsd.org/changeset/base/271594
Log:
Fix an undefined variable that was accidentally not causing an error.
The code had references to both intr_offset and intr_parent variable names
as referring to the parent interrupt node. The intr_parent variable
wasn't actually defined anywhere, but the only references to it were as
an argument to a macro that didn't use that argument in expansion, so
the undefined variable accidentally didn't cause an error.
The intr_parent name makes more sense in context, so change all occurrances
of intr_offset to intr_parent.
Modified:
head/sys/arm/arm/nexus.c
Modified: head/sys/arm/arm/nexus.c
==============================================================================
--- head/sys/arm/arm/nexus.c Sun Sep 14 16:46:30 2014 (r271593)
+++ head/sys/arm/arm/nexus.c Sun Sep 14 17:36:57 2014 (r271594)
@@ -356,16 +356,16 @@ nexus_ofw_map_intr(device_t dev, device_
pcell_t *intr)
{
fdt_pic_decode_t intr_decode;
- phandle_t intr_offset;
+ phandle_t intr_parent;
int i, rv, interrupt, trig, pol;
- intr_offset = OF_node_from_xref(iparent);
+ intr_parent = OF_node_from_xref(iparent);
for (i = 0; i < icells; i++)
intr[i] = cpu_to_fdt32(intr[i]);
for (i = 0; fdt_pic_table[i] != NULL; i++) {
intr_decode = fdt_pic_table[i];
- rv = intr_decode(intr_offset, intr, &interrupt, &trig, &pol);
+ rv = intr_decode(intr_parent, intr, &interrupt, &trig, &pol);
if (rv == 0) {
/* This was recognized as our PIC and decoded. */
More information about the svn-src-all
mailing list