aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDaniel Willmann <daniel@totalueberwachung.de>2011-08-25 16:37:45 +0200
committerHolger Hans Peter Freyther <zecke@selfish.org>2012-09-11 16:20:46 +0200
commit6088f149b57fd21fd45f61f7976ed0fd61a2611e (patch)
treeae06555394c2b6989c767853063d727d0213ba87
parent65924a5fe85c34a46ad8b163b43cd2aa4de3fa94 (diff)
libctrl, osmo-bsc: Get rid of net prefix
net is now implicit in the root node
-rw-r--r--openbsc/include/openbsc/control_cmd.h1
-rw-r--r--openbsc/src/libctrl/control_if.c13
-rw-r--r--openbsc/src/osmo-bsc/osmo_bsc_ctrl.c4
3 files changed, 5 insertions, 13 deletions
diff --git a/openbsc/include/openbsc/control_cmd.h b/openbsc/include/openbsc/control_cmd.h
index da598c517..3dc5b546a 100644
--- a/openbsc/include/openbsc/control_cmd.h
+++ b/openbsc/include/openbsc/control_cmd.h
@@ -14,7 +14,6 @@ struct ctrl_handle;
enum ctrl_node_type {
CTRL_NODE_ROOT, /* Root elements */
- CTRL_NODE_NET, /* Network specific (net.) */
CTRL_NODE_BTS, /* BTS specific (net.btsN.) */
CTRL_NODE_TRX, /* TRX specific (net.btsN.trxM.) */
CTRL_NODE_TS, /* TS specific (net.btsN.trxM.tsI.) */
diff --git a/openbsc/src/libctrl/control_if.c b/openbsc/src/libctrl/control_if.c
index e76b4e5f1..beb5b432a 100644
--- a/openbsc/src/libctrl/control_if.c
+++ b/openbsc/src/libctrl/control_if.c
@@ -139,9 +139,8 @@ int ctrl_cmd_handle(struct ctrl_cmd *cmd, void *data)
char *token, *request;
long num;
int i, j, ret, node;
- struct gsm_network *gsmnet = data;
- struct gsm_network *net = NULL;
+ struct gsm_network *net = data;
struct gsm_bts *bts = NULL;
struct gsm_bts_trx *trx = NULL;
struct gsm_bts_trx_ts *ts = NULL;
@@ -149,8 +148,8 @@ int ctrl_cmd_handle(struct ctrl_cmd *cmd, void *data)
ret = CTRL_CMD_ERROR;
cmd->reply = "Someone forgot to fill in the reply.";
- cmd->node = NULL;
node = CTRL_NODE_ROOT;
+ cmd->node = net;
request = talloc_strdup(tall_bsc_ctx, cmd->variable);
if (!request)
@@ -173,13 +172,7 @@ int ctrl_cmd_handle(struct ctrl_cmd *cmd, void *data)
/* TODO: We need to make sure that the following chars are digits
* and/or use strtol to check if number conversion was successful
* Right now something like net.bts_stats will not work */
- if (!strcmp(token, "net")) {
- net = gsmnet;
- if (!net)
- goto err_missing;
- cmd->node = net;
- node = CTRL_NODE_NET;
- } else if (!strcmp(token, "bts")) {
+ if (!strcmp(token, "bts")) {
if (!net)
goto err_missing;
i++;
diff --git a/openbsc/src/osmo-bsc/osmo_bsc_ctrl.c b/openbsc/src/osmo-bsc/osmo_bsc_ctrl.c
index ba8c1d264..5b2f4ffab 100644
--- a/openbsc/src/osmo-bsc/osmo_bsc_ctrl.c
+++ b/openbsc/src/osmo-bsc/osmo_bsc_ctrl.c
@@ -67,7 +67,7 @@ static void generate_location_state_trap(struct gsm_bts *bts, struct bsc_msc_con
}
cmd->id = "0";
- cmd->variable = talloc_asprintf(cmd, "net.bts.%i.location-state", bts->nr);
+ cmd->variable = talloc_asprintf(cmd, "bts.%i.location-state", bts->nr);
/* Prepare the location reply */
cmd->node = bts;
@@ -299,7 +299,7 @@ int bsc_ctrl_cmds_install()
rc = ctrl_cmd_install(CTRL_NODE_BTS, &cmd_bts_loc);
if (rc)
goto end;
- rc = ctrl_cmd_install(CTRL_NODE_NET, &cmd_net_rf_lock);
+ rc = ctrl_cmd_install(CTRL_NODE_ROOT, &cmd_net_rf_lock);
end:
return rc;
}