laforge submitted this change.

View Change



1 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.

Approvals: fixeria: Looks good to me, approved Jenkins Builder: Verified
ASCI: Correctly set and store 'uplink-release-local' flag

Also this option is enabled by default and show in the VTY if it is
disabled.

Change-Id: I6af44f0dd7ff842de633587fb0dcbe78126d30e1
Related: OS#5364
---
M src/host/layer23/src/common/settings.c
M src/host/layer23/src/mobile/vty_interface.c
2 files changed, 16 insertions(+), 1 deletion(-)

diff --git a/src/host/layer23/src/common/settings.c b/src/host/layer23/src/common/settings.c
index 395d239..e79a6a1 100644
--- a/src/host/layer23/src/common/settings.c
+++ b/src/host/layer23/src/common/settings.c
@@ -116,6 +116,8 @@

INIT_LLIST_HEAD(&set->abbrev);

+ set->uplink_release_local = true;
+
return 0;
}

diff --git a/src/host/layer23/src/mobile/vty_interface.c b/src/host/layer23/src/mobile/vty_interface.c
index 7611c0f..aaf01bd 100644
--- a/src/host/layer23/src/mobile/vty_interface.c
+++ b/src/host/layer23/src/mobile/vty_interface.c
@@ -1326,7 +1326,7 @@
set->any_timeout, VTY_NEWLINE);
if (!l23_vty_hide_default || !set->uplink_release_local)
vty_out(vty, " %suplink-release-local%s",
- (set->uplink_release_local) ? "no " : "", VTY_NEWLINE);
+ (!set->uplink_release_local) ? "no " : "", VTY_NEWLINE);
if (!l23_vty_hide_default || set->asci_allow_any)
vty_out(vty, " %sasci-allow-any%s",
(set->asci_allow_any) ? "" : "no ", VTY_NEWLINE);

To view, visit change 34867. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I6af44f0dd7ff842de633587fb0dcbe78126d30e1
Gerrit-Change-Number: 34867
Gerrit-PatchSet: 2
Gerrit-Owner: jolly <andreas@eversberg.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-MessageType: merged