From b6e2b839038f7036739b6611961f3c93969c9405 Mon Sep 17 00:00:00 2001
From: Rick Carlino <rick.carlino@gmail.com>
Date: Thu, 9 Apr 2020 09:21:14 -0500
Subject: [PATCH] Revert GPIO changes

---
 VERSION                                              |  2 +-
 .../platform/target/gpio/circuits_gpio_handler.ex    | 12 ++++--------
 2 files changed, 5 insertions(+), 9 deletions(-)

diff --git a/VERSION b/VERSION
index 0bbd88ac..fcecff17 100644
--- a/VERSION
+++ b/VERSION
@@ -1 +1 @@
-9.2.2-rc6
+9.2.2-rc7
diff --git a/farmbot_os/platform/target/gpio/circuits_gpio_handler.ex b/farmbot_os/platform/target/gpio/circuits_gpio_handler.ex
index aeaea7ae..b45050f4 100644
--- a/farmbot_os/platform/target/gpio/circuits_gpio_handler.ex
+++ b/farmbot_os/platform/target/gpio/circuits_gpio_handler.ex
@@ -3,7 +3,6 @@ defmodule FarmbotOS.Platform.Target.PinBindingWorker.CircuitsGPIOHandler do
 
   @behaviour FarmbotCore.AssetWorker.FarmbotCore.Asset.PinBinding
   require Logger
-  require FarmbotCore.Logger
   use GenServer
   alias Circuits.GPIO
 
@@ -23,13 +22,10 @@ defmodule FarmbotOS.Platform.Target.PinBindingWorker.CircuitsGPIOHandler do
     Logger.info("CircuitsGPIOHandler #{pin_number} init")
     {:ok, pin} = GPIO.open(pin_number, :input)
     :ok = GPIO.set_interrupts(pin, :rising)
-    # this has been checked on v1.3 and v1.5 hardware and it seems to be fine.
-    # but that was with `:pulldown`.
-    :ok = GPIO.set_pull_mode(pin, :none)
-    result = %{pin_number: pin_number, pin: pin, fun: fun, debounce: nil}
-    str = "*** " <> inspect(result)
-    FarmbotCore.Logger.info(1, str)
-    {:ok, result}
+    # this has been checked on v1.3 and v1.5 hardware
+    # and it seems to be fine.
+    :ok = GPIO.set_pull_mode(pin, :pulldown)
+    {:ok, %{pin_number: pin_number, pin: pin, fun: fun, debounce: nil}}
   end
 
   def handle_info(:timeout, state) do