diff options
| author | Pau Espin Pedrol <pespin@sysmocom.de> | 2021-07-15 13:28:38 +0200 |
|---|---|---|
| committer | pespin <pespin@sysmocom.de> | 2021-07-20 10:07:03 +0000 |
| commit | ce869c6baa227ccc0756b921eba0c332c563b923 (patch) | |
| tree | ac03de436e737c56710c4115105503f2dfb320ec /src/osmo-bsc/abis_rsl.c | |
| parent | 2956b52c5b3603f5eaf5f5168a1244cf8b287e15 (diff) | |
_select_sdcch_for_call: Avoid 2nd lchan lookup when finally selecting it
We already looked it up, it's not necessary to look it up again by
calling lchan_select_by_type(). Let's instead call
lchan_select_set_type() directly on the lchan pointer.
Related: SYS#5309
Change-Id: I1054c18f58c9e249f263e3e97a365a1fd8b03a93
Diffstat (limited to 'src/osmo-bsc/abis_rsl.c')
| -rw-r--r-- | src/osmo-bsc/abis_rsl.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/osmo-bsc/abis_rsl.c b/src/osmo-bsc/abis_rsl.c index 9b6429d5d..c84aeda2b 100644 --- a/src/osmo-bsc/abis_rsl.c +++ b/src/osmo-bsc/abis_rsl.c @@ -1964,8 +1964,7 @@ struct gsm_lchan *_select_sdcch_for_call(struct gsm_bts *bts, const struct chan_ } select_lchan: - /* FIXME: we already have lchan, simply do lchan->type = GSM_LCHAN_SDCCH? Split lchan_select_by_type in 2 functions? */ - lchan = lchan_select_by_type(bts, GSM_LCHAN_SDCCH); + lchan_select_set_type(lchan, GSM_LCHAN_SDCCH); return lchan; } |