Attention is currently required from: fixeria.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/33726 )
Change subject: osmo-bts-trx: document/clarify the meaning of BUFMAX=24 ......................................................................
Patch Set 1: Code-Review+1
(1 comment)
File include/osmo-bts/scheduler.h:
https://gerrit.osmocom.org/c/osmo-bts/+/33726/comment/e0495da3_c6e6ad46 PS1, Line 135: struct l1sched_meas_set buf[24]; /* up to 24 (BUFMAX) entries */
Because `BUFMAX` is defined in a local header of osmo-bts-trx, while this header is in the common di […]
As you see, buf if this 24 is directly related to BUFMAX then I really think it makes sense to have some define used here too, even if that means moving the defines to somewhere under common/.
-
pespin