laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/36964?usp=email )

Change subject: ipaccess-config: rename build_physconf to build_physconf_arfcn_by_rxlev ......................................................................

ipaccess-config: rename build_physconf to build_physconf_arfcn_by_rxlev

It doesn't build up a generic physconf. It only build up an arfcn white list based on a list of rxlev

Change-Id: I6e2d27c10dc9075f06070bb784369bbd1d75f88e --- M src/ipaccess/ipaccess-config.c 1 file changed, 15 insertions(+), 3 deletions(-)

Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified

diff --git a/src/ipaccess/ipaccess-config.c b/src/ipaccess/ipaccess-config.c index 3df9f61..720837e 100644 --- a/src/ipaccess/ipaccess-config.c +++ b/src/ipaccess/ipaccess-config.c @@ -212,7 +212,7 @@

static const uint8_t phys_conf_min[] = { 0x02 };

-static uint16_t build_physconf(uint8_t *physconf_buf, const struct rxlev_stats *st) +static uint16_t build_physconf_arfcn_by_rxlev(uint8_t *physconf_buf, const struct rxlev_stats *st) { uint16_t *whitelist = (uint16_t *) (physconf_buf + 4); int num_arfcn; @@ -244,7 +244,7 @@ /* Dump RxLev results */ //rxlev_stat_dump(&trx->ipaccess.rxlev_stat); /* Create whitelist from results */ - physconf_len = build_physconf(physconf_buf, + physconf_len = build_physconf_arfcn_by_rxlev(physconf_buf, &trx->ipaccess.rxlev_stat); /* Start next test about BCCH channel usage */ ipac_nwl_test_start(trx, NM_IPACC_TESTNO_BCCH_CHAN_USAGE, @@ -254,7 +254,7 @@ /* Dump BCCH RxLev results */ //rxlev_stat_dump(&trx->ipaccess.rxlev_stat); /* Create whitelist from results */ - physconf_len = build_physconf(physconf_buf, + physconf_len = build_physconf_arfcn_by_rxlev(physconf_buf, &trx->ipaccess.rxlev_stat); /* Start next test about BCCH info */ ipac_nwl_test_start(trx, NM_IPACC_TESTNO_BCCH_INFO,