fixeria submitted this change.

View Change

Approvals: fixeria: Looks good to me, approved osmith: Looks good to me, but someone else must approve Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve
s1ap_proxy: drop fetch_enb_info/1

This API is no longer used/needed, since we have the enb_registry.

Change-Id: I5a5f0dc1374f14fce14b33b3ea237eb083dd302c
Related: ada15bf ("enb_registry: new module")
---
M src/s1ap_proxy.erl
1 file changed, 0 insertions(+), 11 deletions(-)

diff --git a/src/s1ap_proxy.erl b/src/s1ap_proxy.erl
index 8d765d4..cbd9b10 100644
--- a/src/s1ap_proxy.erl
+++ b/src/s1ap_proxy.erl
@@ -44,7 +44,6 @@
process_pdu/2,
fetch_erab/2,
fetch_erab_list/1,
- fetch_enb_info/1,
shutdown/1]).

-include_lib("kernel/include/logger.hrl").
@@ -122,12 +121,6 @@
gen_server:call(Pid, ?FUNCTION_NAME).


-%% Fetch information about the eNB
--spec fetch_enb_info(pid()) -> enb_info().
-fetch_enb_info(Pid) ->
- gen_server:call(Pid, ?FUNCTION_NAME).
-
-
-spec shutdown(pid()) -> ok.
shutdown(Pid) ->
gen_server:stop(Pid).
@@ -157,10 +150,6 @@
#proxy_state{erabs = ERABs} = S) ->
{reply, dict:to_list(ERABs), S};

-handle_call(fetch_enb_info, _From,
- #proxy_state{} = S) ->
- {reply, enb_info(S), S};
-
handle_call(Info, From,
#proxy_state{} = S) ->
?LOG_ERROR("unknown ~p() from ~p: ~p", [?FUNCTION_NAME, From, Info]),

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

Gerrit-MessageType: merged
Gerrit-Project: erlang/osmo-s1gw
Gerrit-Branch: master
Gerrit-Change-Id: I5a5f0dc1374f14fce14b33b3ea237eb083dd302c
Gerrit-Change-Number: 41620
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>