Bug Report: enable pins as digital input pins to start reading not working for DI but for DIO

[mascheck55]

Version

0.0.23alpha

Bug description

The function "bool enableDigitalInput(uint8_t pin, bool pullup) {" has in her first line:
if (!bitRead(pinMap[pin].capability, DI)) { ... but DI is 1 and for real DI (not DIO) there should be a 0. Hence I corrected as followes "if (!bitRead(pinMap[pin].capability, 0)) {" and that works also for DIO pins.

Steps to reproduce the bug

mark one pin as DI instead of DIO in the pin mapping file
try to use it as digital input in the myAutomation.h
watch the serial output of te EX-IOExpander and you get "ERROR! pin X not capable of digital input"

Expected behaviour

there schould be created a virtual Pin for Input only

Screenshots

No response

Hardware in use

DCC-EX Commandstation Mega
EX-IOExpander Nano

Additional context

No response

[mascheck55]

I was thinking how the ERROR came in - probably it was a confusion between DIGITAL_INPUT and DI. Right would be : if (!bitRead(pinMap[pin].capability, DIGITAL_INPUT)) {
I have seen a similar construction in other places. For the time being I am looking for the place where I may plug in own code in digitalRead for a couple of pins that I need to configure different. Therefor I would like to add
#define VIRTUAL_INPUT 16
and
#define VI 0x10 // virtual DI pin with own setup and read function
comments to that are most welcome

[Ash-4]

Please use the discord server for discussion on EX-IOExpander.
https://dcc-ex.com/support/index.html#preferred-option-discord

Create a thread in the # ex-ioexpander channel. Perhaps thread title can be:
Add custom code to EX-IOExpander