Incorrect value for OUTPUT_OPEN_DRAIN in esp32-hal-gpio.h

[TMSL]

Board

ESP32S3 Dev Kit C

Device Description

It appears OUTPUT_OPEN_DRAIN should be 0x13 instead of 0x12.
In ESP32-hal-gpio.h:

//GPIO FUNCTIONS
#define INPUT 0x01
// Changed OUTPUT from 0x02 to behave the same as Arduino pinMode(pin,OUTPUT)
// where you can read the state of pin even when it is set as OUTPUT
#define OUTPUT 0x03 <=== OUTPUT
#define PULLUP 0x04
#define INPUT_PULLUP 0x05
#define PULLDOWN 0x08
#define INPUT_PULLDOWN 0x09
#define OPEN_DRAIN 0x10 <=== OPEN_DRAIN
#define OUTPUT_OPEN_DRAIN 0x12 <=== should be 0x13
#define ANALOG 0xC0

Hardware Configuration

tested with pullup resistor on gpio 38

Version

v2.0.11

IDE Name

Arduino 2.2.0

Operating System

Linux Mint

Flash frequency

80 MHz

PSRAM enabled

yes

Upload speed

115200

Description

Setting "OUTPUT_OPEN_DRAIN (= 0x12)" in pinmode does not work, but using OUTPUT | OPEN_DRAIN ( = 0x13) does.

Sketch

#define OUTPIN 38
hw_timer_t *My_timer = NULL;

void IRAM_ATTR onTimer(){

digitalWrite(OUTPIN, !digitalRead(OUTPIN));
}

void setup() {

// FAILS:
pinMode(OUTPIN, OUTPUT_OPEN_DRAIN); // to get OPEN DRAIN use "OUTPUT | OPEN_DRAIN"

// create 1MHz counter by setting prescaler to 80 (ESP32 w/80MHz clock)
My_timer = timerBegin(0, 80, true);
timerAttachInterrupt(My_timer, &onTimer, true);
timerAlarmWrite(My_timer, 5, true); //1000000 counts @ 1MHz -> 1sec, 5->5usec
timerAlarmEnable(My_timer); //Enable the configured timer
}

void loop() {
}

Debug Message

none

Other Steps to Reproduce

none

I have checked existing issues, online documentation and the Troubleshooting Guide

• I confirm I have checked existing issues, online documentation and Troubleshooting guide.

[P-R-O-C-H-Y]

@TMSL I assume that it's right to be 0x12, as the pin should be only OUTPUT_OPEN_DRAIN and not INPUT_OUTPUT_OPEN_DRAIN as you use it. If you want to read the pin you have to call pinMode( PIN , (INPUT | OUTPUT_OPEN_DRAIN)); or pinMode( PIN , (OUTPUT | OPEN_DRAIN )); as you did. You can still use it without any issues.

[P-R-O-C-H-Y]

Closing this as duplicate of #7022.

[TMSL]

When the definition of OUTPUT was changed to 0x03 from 0x02 (per comment in .h file), it implicitly made OUTPUT set the pin to INPUT at the same time. (since INPUT is defined as 0x01 and the settings are treated as OR'd together).

But when the OUTPUT was changed to 0x03, the same automatically setting INPUT behavior was NOT added to the OUTPUT_OPEN_DRAIN definition. The problem with OUTPUT_OPEN_DRAIN being 0x12 is that it FAILS (because the code does not configure an output unless the value for INPUT is also SET). I.e. using pinmode with OUTPUT simultaneously sets INPUT, but using OUTPUT_OPEN_DRAIN in pinmode does not.

There are two ways get an OPEN DRAIN output to work: Either you need to use OUTPUT_OPEN_DRAIN | INPUT ( per issue #7022 ) or you need to use OUTPUT | OPEN_DRAIN (per #8590). But, again, unlike OUTPUT, using OUTPUT_OPEN_DRAIN (0x12) BY ITSELF DOES NOT WORK. Only something that produces 0x13 does.

[TMSL]

P.S. - this is not a duplicate of #7022. #7022 wanted to set the pin to open drain output AND input at the same time. But I only wanted to set the pin to open drain output. BUT the 'bug' is that setting 0x12 FAILS unless I also set it as INPUT.

[P-R-O-C-H-Y]

@TMSL Sorry about closing it that fast. My bad. I will try to do some tests today, so from what you wrote, the output_opendrain is not working at all? Or what exactly is not working?

I can open a PR with the fix you suggested today, if it's reasonable to get in into 2.0.12 release.

[TMSL]

Thanks. Yes, when I used OUTPUT_OPEN_DRAIN alone I got no output signal from the GPIO pin at all, even though I had a pullup resistor connected. When I used OUTPUT it worked fine as a driven signal. When I changed the code to OUTPUT | OPEN_DRAIN it worked and I verifed that it was operating as OPEN_DRAIN by seeing the signal go away when I removed the pullup.
My test program was based on the RGB LED example. I viewed the oscillating signal on GPIO 48 using an oscilloscope.