search

lora-aprs / LoRa_APRS_iGate

This is a LoRa APRS iGate/Digi based on an ESP32
https://www.lora-aprs.info
MIT License
348 stars 112 forks source link

wakeup OLED with button not implemented #111

Closed peterus closed 2 years ago

peterus commented 3 years ago

see https://github.com/lora-aprs/LoRa_APRS_iGate/discussions/110

OK1FWG commented 3 years ago

I am using the devel branch where some changes were implemented and I can confirm, that this is working. But! In the beginning of my testing, I was surprised, that I must have the override pin connected either to GND or 3V3 (depending on the logical condition). I then tried several different GPIOs (with or without PULLUP). This was not what I wanted - simply connect the overridepin to GND should wake up the display. But this wasn't working. I choosed the GPIO13 (has the PULLUP) and had to change this in LoRa_APRS_iGate.cpp: from:

  if (userConfig.display.overwritePin != 0) {
    pinMode(userConfig.display.overwritePin, INPUT);
    pinMode(userConfig.display.overwritePin, INPUT_PULLUP);
  }

to:

if (userConfig.display.overwritePin != 0) {
    pinMode(userConfig.display.overwritePin, INPUT_PULLUP);
  }

I understand the doubled pin input mode settings (you may use different pin than me), but I really wonder why it caused troubles. So my observation. Now I am satisfied ;-)

peterus commented 2 years ago

closing with v21.50.0.