search

pioneers / runtime

Firmware for the PiE kit robots and devices
7 stars 2 forks source link

[LOWCAR] make sure lowcar parameter is valid #256

Closed wongquijote closed 1 year ago

wongquijote commented 2 years ago

RESOLVED #253 We should make sure that device_read()/device_write() for all the devices return 0 if the param argument is invalid.

In LimitSwitch::device_read(), we call digitalRead() with the argument param, even if might not refer to a relevant pin.

benliao1 commented 1 year ago

Should merge this one as well, this one is ready to go once the workflows CI is added and we rebase it on master