Attention is currently required from: laforge, pespin. osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/27923 )
Change subject: osmobts: list all features of latest osmobts ......................................................................
Patch Set 2:
(1 comment)
File src/osmo-bsc/bts_trx_vty.c:
https://gerrit.osmocom.org/c/osmo-bsc/+/27923/comment/5528374f_d86764e0 PS2, Line 347: * connected yet (thus not sent the feature vector), so we cannot know for
I think this entire [existing] logic is flawed. […]
My thought process was that with this patch it would at least be consistent between the BTS models that can and the ones that cannot report the features upon connect. But I see your point.
How should this be handled then, should we just remove checks in VTY commands against osmo_bts_has_feature?
-
osmith