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. -- To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/30972 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmocom-bb Gerrit-Branch: master Gerrit-Change-Id: Ib5c9b6f3efa255d67980945db9f98dd8a112af0e Gerrit-Change-Number: 30972 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: msuraev <[email protected]> Gerrit-CC: neels <[email protected]> Gerrit-Attention: neels <[email protected]> Gerrit-Attention: msuraev <[email protected]> Gerrit-Comment-Date: Mon, 16 Jan 2023 09:56:15 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: neels <[email protected]> Gerrit-MessageType: comment
