Attention is currently required from: neels, laforge, fixeria. osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/27923 )
Change subject: Check VTY config against features reported by BTS ......................................................................
Patch Set 7:
(2 comments)
File src/osmo-bsc/bts.c:
https://gerrit.osmocom.org/c/osmo-bsc/+/27923/comment/67eb271c_82a1bcae PS6, Line 502: if (!bts_gprs_mode_is_compat(bts, bts->gprs.mode)) {
&& instead of nested if?
Done on purpose here, so we have a if (bts->features_known) { ... } block where we can have multiple checks in the future. If it's strongly preferred to use && here however, I'd change it.
File src/osmo-bsc/bts_osmobts.c:
https://gerrit.osmocom.org/c/osmo-bsc/+/27923/comment/5740a709_cffda913 PS6, Line 208: /* Adjust bts_init/bts_model_init in OsmoBTS to report new features.
i wonder what this comment attempts to say, sounds like legacy that we can drop?
I just added this comment recently. What I meant with it is: when OsmoBTS supports a new feature, OsmoBTS code should be adjusted to report this feature. Instead of just adding the feature here to the BTS model, as it had been done sometimes in the past.