pespin submitted this change.

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

nm_is_running(): Drop duplicate check

NM_AVSTATE_OK is 0xff, so we are checking twice against same value.

Change-Id: Idac56e5eca2b6f6998195ec764fac7edbc8f53e2
---
M src/osmo-bsc/gsm_data.c
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/osmo-bsc/gsm_data.c b/src/osmo-bsc/gsm_data.c
index de298a8..3438eaf 100644
--- a/src/osmo-bsc/gsm_data.c
+++ b/src/osmo-bsc/gsm_data.c
@@ -760,7 +760,7 @@
 bool nm_is_running(const struct gsm_nm_state *s) {
 	if (s->operational != NM_OPSTATE_ENABLED)
 		return false;
-	if ((s->availability != NM_AVSTATE_OK) && (s->availability != 0xff))
+	if (s->availability != NM_AVSTATE_OK)
 		return false;
 	if (s->administrative != NM_STATE_UNLOCKED)
 		return false;

3 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.

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