Attention is currently required from: pespin.
Timur Davydov has posted comments on this change by Timur Davydov. ( https://gerrit.osmocom.org/c/osmo-trx/+/42421?usp=email )
Change subject: transceiver: return -EINVAL for invalid SETSLOT ......................................................................
Patch Set 2:
(1 comment)
File Transceiver52M/Transceiver.cpp:
https://gerrit.osmocom.org/c/osmo-trx/+/42421/comment/f57f2d56_ba8dde0d?usp=... : PS1, Line 1090: return rc == -EINVAL ? 0 : rc; /* Ignore invalid values, but report other errors */
why ignore it here? I'd actually propagate the error since it means there was an error, there's no n […]
I've updated it so that an invalid timeslot now results in an error instead of being ignored.