ESP32 C3 digitalRead() returns always 0, should return the pin level

[helmut64]

When I do a digitalWrite() to set the GPIO pin (e.g. 2) to HIGH, a digitalRead(2) returns always 0
It should return the pin level as it does not the regular ESP32 or the S2
Arduino-esp32 version: 2.0.0-alpha1
Regards Helmut

[atanisoft]

This is expected behavior as the pin is configured as output only. Reading an output pin will always return zero. The pin must be configured as INPUT_OUTPUT for the read to work. Unfortunately there is no Arduino API to configure a pin as INPUT_OUTPUT, you can however call:

gpio_set_direction(PIN, GPIO_MODE_INPUT_OUTPUT);

where PIN is the target GPIO number.

[helmut64]

Hi Mike, thank you for that advise, your workaround works.
However I am using the Arduino API, digitalRead on the output set GPIO's works on Arduino (ESP32, Atmel-D21, ESP32-S2, etc). On Mbed OS this is also supported. I believe it would be great if the C3 behaves like the other boards/platforms.

Again, thank you for your advise.
PS: I am using like many others reading the GPIO and setting the inverted value.
Regards from Hannover Helmut

[chegewara]

There is simple explanation. ESP32 C3 just does not support it.
You can use this simple sketch, which is using esp-idf API, to test it:

void setup() {
  // put your setup code here, to run once:
  Serial.begin(115200);
  pinMode(7, OUTPUT);
  digitalWrite(7, 0);
  delay(1000);
  Serial.print("pin level: ");
  Serial.println(gpio_get_level((gpio_num_t)7));
}

void loop() {
  // put your main code here, to run repeatedly:
  delay(1000);
  digitalWrite(7, 1);
  Serial.print("pin level: ");
  Serial.println(gpio_get_level((gpio_num_t)7));
  digitalWrite(7, 0);
  delay(1000);
  Serial.print("pin level: ");
  Serial.println(gpio_get_level((gpio_num_t)7));
}

Like you said, it works on S2 and does not work on C3, and it cant be changed or fixed in arduino as long as it wont be "fixed" in esp-idf. This also may be C3 hardware design and cant be fixed.

@igrr

[helmut64]

Again, I understand that it is not working. Ff it can be fixed in the IDF, I would recommend to do so.

[chegewara]

Then you should report it in esp-idf, not here, if you understand it.

[helmut64]

But here is the place for the ESP32-C3-Arduino, and it is an Arduino behaviour. I will have a look into the IDF source to check out what is behind.

[chegewara]

I did show you example code with plain esp-idf API, it is not arduino, which shows the same output:
gpio_get_level((gpio_num_t)7)

[atanisoft]

This CAN work in arduino-esp32 but the support code for it does not exist yet. A new pin mode will be necessary (INPUT_OUTPUT or similar name) which is a combination of the current INPUT and OUTPUT modes. The underlying ESP-IDF code does not support reading the state of an OUTPUT only pin and that is why the current arduino code does not work as you might expect.

As for why this works for ESP32 and ESP32-S2 but not ESP32-C3 that is due to the ESP32-C3 code using ESP-IDF APIs whereas the others are using a local implementation which does not work for the ESP32-C3 (and long term should be removed entirely in favor of ESP-IDF APIs).

[chegewara]

@atanisoft this is esp-idf API, which works on esp32 with pin OUTPUT only:
gpio_get_level((gpio_num_t)7)
the same API does not work on C3, so the problem is not arduino perse, and difference in esp-idf on different models.

[atanisoft]

@chegewara it shouldn't work even in ESP-IDF unless the pin mode is INPUT_OUTPUT since the HAL code references the hardware IN register which is not updated as part of OUTPUT only pin manipulation.

[chegewara]

Maybe it should not work, but it works. Either way it may be consider bug and should be reported in esp-idf, not here.

To be honest until now ive been thinking the same way, that reading OUTPUT pin should not be possible and i never before even did try it, since there is INPUT_OUTPUT option.

[helmut64]

A common technic ist digitalWrite(!digitalWrite()) to invert the current output.
For me this was just once reason why existing software did not work with the C3.

[lbernstone]

I think the expected arduino behavior here is that you can digitalRead an output pin. That probably means the implementation is INPUT_OUPUT once the core is fully refactored on the idf hal.

[atanisoft]

The ESP32-C3 uses ESP-IDF APIs but ESP32 and ESP32-S2 use direct register access. This issue should be used to standardize on ESP-IDF APIs AND default OUTPUT mode to be INPUT_OUTPUT.

[chegewara]

@atanisoft are you sure? As far as i understand C3 is not using esp-idf, but i may be wrong:
https://github.com/espressif/arduino-esp32/blob/master/cores/esp32/esp32-hal-gpio.c#L324