tree bd06f0fdc6f13a40e4ee05f60a104425a438d3eb
parent 4d3d2436cdf3296ddc110be4022dc2ec13d3eb86
author Stefan Sperling <ssperling@sysmocom.de> 1523517516 +0200
committer Stefan Sperling <ssperling@sysmocom.de> 1523623377 +0200

fix handling of state changes in acc ramping

Take both the operative and administrative states into account
when deciding whether to start ACC ramping, and examine old/new
state values to avoid triggering ramping for a no-op state change.

This requires a fix to gsm_trx_lock_rf(): This function overwrote
the old administrative state of a trx before enqueuing a state
change request towards the BTS.
The BTS will confirm this request with an ACK, at which time a
signal is generated which the ACC ramp code listens to. We must
not overwrite the old state value until the signal has been handled,
otherwise the signal handler cannot tell what the old state was.

Tested with a virtphy setup, nanobts, and osmo-bts.

Change-Id: Ib3291439655598fb5ddc891a3e4cc35b0bad250f
Related: OS#2591