Sketch working on Uno, not on ESP8266

Hi all,

I've done quite a bit in Arduino over the years however this is my first attempt at ESP8266 modules.

I have the following script working fine on standard Arduino hardware, however once it's loaded up on an NodeMCU ESP8266 module it doesn't work. LED is always HIGH, nothing outputs to serial monitor and button obviously doesn't do anything. It compiles and uploads fine, just doesn't run as expected.

I'm aware of the usable pins on the ESP, the labeled pin and their respective GPIO number however still nothing.

I assume it's something silly I'm doing wrong, and any assistance is appreciated.

Cheers

Adam


// this constant won't change:
const int  buttonPin = 5;    // the pin that the pushbutton is attached to
const int ledPin = 4;       // the pin that the LED is attached to

// Variables will change:
int buttonPushCounter = 0;   // counter for the number of button presses
int buttonState = 0;         // current state of the button
int lastButtonState = 0;     // previous state of the button

void setup() {
  // initialize the button pin as a input:
  pinMode(buttonPin, INPUT);
  // initialize the LED as an output:
  pinMode(ledPin, OUTPUT);
  // initialize serial communication:
  delay(2000);
  Serial.begin(115200);
  delay(2000);
}


void loop() {
  // read the pushbutton input pin:
  buttonState = digitalRead(buttonPin);

  // compare the buttonState to its previous state
  if (buttonState != lastButtonState) {
    // if the state has changed, increment the counter
    if (buttonState == HIGH) {
      // if the current state is HIGH then the button went from off to on:
      buttonPushCounter++;
      Serial.println("on");
      Serial.print("number of button pushes: ");
      Serial.println(buttonPushCounter);
    } else {
      // if the current state is LOW then the button went from on to off:
      Serial.println("off");
    }
    // Delay a little bit to avoid bouncing
    delay(100);
  }
  // save the current state as the last state, for next time through the loop
  lastButtonState = buttonState;


  // turns on the LED every second button press by checking the modulo of the
  // button push counter. the modulo function gives you the remainder of the
  // division of two numbers:
  if (buttonPushCounter % 2 == 0) {
    digitalWrite(ledPin, HIGH);
  } else {
    digitalWrite(ledPin, LOW);
  }

}

I would bet you're not using the correct pin numbers for the two GPIOs. Which PHYSICAL pins on the device are you using? Take a photo of your wiring, or edit a photo of a NODEMCU, and indicate which pins you're using.

Thanks for your response, Ray.

I'm using GPIO 4 for the led and 5 for the button. Which is labeled pins 1 & 2 on the NodeMCU Board.

*Deleted and reposted as reply

I see you are trying to use GPIO_NUM_4 and GPIO_NUM_5 or are you trying to use the physical pin number?

Pretty sure the numbers you WANT to use are the ones printed on the board. So GPIO4 is D2, and GPIO5 is D1. So, try this:

const int buttonPin = 1; // the pin that the pushbutton is attached to
const int ledPin = 2;

I've tried that as well, still nothing happening or printing to serial when button is pressed.
This time LED is LOW on boot, with no change.
Wiring is as below. (Obviously with different board)

@Idahowalker

Then I suspect your NODEMCU is damaged. I have used those two pins extensively with no problems.

Are you sure you have the board correctly configured in the IDE? You must specify the correct NODEMCU version (most available now seem to be 12E versions). You cannot specify ESP8266.

Start by doing a simple test sketch that simple prints things to the serial port. Then try to blink the on-board LED (if there is one).

Thanks for your serial test suggestion. I found that the pin the LED was on is faulty. I switched it to Built in LED and everything is happy now. Allowed me to easily finish off the full project.

Thanks for your help @RayLivingston & @Idahowalker

Always helps to first prove your basic assumptions with simple tests...

Pretty sure that if you do it like that you should use

const int buttonPin = D1; // the pin that the pushbutton is attached to
const int ledPin = D2;

@Idahowalker @RayLivingston
For everyone's reference, in the final code I used;

int buttonPin = 4; //GPIO4

The project I was working on, if anyone is interested is also below. Thanks again for everyone's assistance on this
ESP2866 Telegram Messenger

This topic was automatically closed 120 days after the last reply. New replies are no longer allowed.