Attention is currently required from: pespin. fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/30992 )
Change subject: layer23: Introduce APN VTY node ......................................................................
Patch Set 2:
(3 comments)
File doc/examples/modem/modem.cfg:
https://gerrit.osmocom.org/c/osmocom-bb/+/30992/comment/d994406f_2ef00411 PS2, Line 12: no shutdown 'no shutdown' again? I understand the idea that the user may want to switch between APNs, or even activate several at the same time (if this is possible at all), but something like 'active' or 'enabled' would be more intuitive, IMO.
File src/host/layer23/include/osmocom/bb/common/apn.h:
https://gerrit.osmocom.org/c/osmocom-bb/+/30992/comment/03f80523_b141dbf1 PS2, Line 62: //struct in6_addr v6_lladdr; TODO?
File src/host/layer23/include/osmocom/bb/common/ms.h:
https://gerrit.osmocom.org/c/osmocom-bb/+/30992/comment/5129f065_06866b2c PS2, Line 94: gprs_settings Ideally, all settings should be in struct gsm_settings.