Merge pull request #396 from cyberman54/patch-1

terrillmoore · web-flow · commit 7315dc14d89c · 2019-09-01T12:02:18.000-04:00
bugfix: add missing pinmode initialization in hal_interrupt_init()
diff --git a/src/hal/hal.cpp b/src/hal/hal.cpp
@@ -130,6 +130,7 @@ static void hal_interrupt_init() {
       if (plmic_pins->dio[i] == LMIC_UNUSED_PIN)
           continue;
 
+      pinMode(plmic_pins->dio[i], INPUT);
       attachInterrupt(digitalPinToInterrupt(plmic_pins->dio[i]), interrupt_fns[i], RISING);
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -130,6 +130,7 @@ static void hal_interrupt_init() {`
`130`	`130`	`if (plmic_pins->dio[i] == LMIC_UNUSED_PIN)`
`131`	`131`	`continue;`
`132`	`132`
	`133`	`+ pinMode(plmic_pins->dio[i], INPUT);`
`133`	`134`	`attachInterrupt(digitalPinToInterrupt(plmic_pins->dio[i]), interrupt_fns[i], RISING);`
`134`	`135`	`}`
`135`	`136`	`}`