Hi Neels,
On 8/2/21 12:06 AM, Neels Hofmeyr wrote:
Just now I came across this code, with an early return in case of BCCH. But right below that, BCCH is still mentioned in the comment, and there are more IEs being parsed.
So I get the impression the early return may prevent that OSMO_MIN limiting for BCCH, and that may not be intended?
in rsl_rx_bs_pwr_ctrl(), we actually handle these different cases:
a) power reduction on *inactive* logical channels of the BCCH carrier, b) power reduction on *active* logical channels of the BCCH carrier, c) power reduction on *active* logical channels of a non-BCCH carrier.
Case a) is handled in bts_set_c0_pwr_red(). This function iterates over all timeslots and sets power reduction values depending on their pchan combinations.
Case b) actually depends on a), because ts->c0_power_red_db is set in bts_set_c0_pwr_red(). The idea is that b) is not allowed unless the BTS is operating in the power saving mode.
So, this is why the OSMO_MIN limiting is not relevant to a).
Case c) is not related to the question brought in this thread.
Regards, Vadim.