Attention is currently required from: neels, msuraev. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/30972 )
Change subject: layer23: Move '(no) shutdown' VTY code to common/vty.c ......................................................................
Patch Set 2:
(3 comments)
File src/host/layer23/src/common/vty.c:
https://gerrit.osmocom.org/c/osmocom-bb/+/30972/comment/5e11e667_05d10b91 PS2, Line 223: placeholder
placeholder? i don't understand...
This function holds the common place where the shared final VTY commandds are put.
https://gerrit.osmocom.org/c/osmocom-bb/+/30972/comment/c8724bbb_052a24c9 PS2, Line 229: vty_out
(i would drop this line)
It was alredy there and it does no harm. As you can see the list of changes I'm doing is already quite large enough. I'm just trying to get everything to a state where I can easily have a VTY and share stuff between modem and mobile.
File src/host/layer23/src/mobile/vty_interface.c:
https://gerrit.osmocom.org/c/osmocom-bb/+/30972/comment/e36e5c2c_58e10ab7 PS2, Line 2951: static int l23_vty_signal_cb(unsigned int subsys, unsigned int signal,
this patch log says "Move" some vty command, but this function seems brand new / not moved from anyw […]
That's part of the changes required to move the command from one place to another, since it is moved from an app specific to a generic/common place.