fixeria submitted this change.

Approvals:
  laforge: Looks good to me, but someone else must approve
  fixeria: Looks good to me, approved
  jolly: Looks good to me, but someone else must approve
  Jenkins Builder: Verified

assignment_fsm: ipacc_chan_mode_supported(): log unsupported chan_mode

Change-Id: I1718994e25ff1349618bdb4660afbe735c5c3977
Related: aa959d74f ("assignment_fsm: check ipaccess channel mode support")
---
M src/osmo-bsc/assignment_fsm.c
1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/osmo-bsc/assignment_fsm.c b/src/osmo-bsc/assignment_fsm.c
index 54a57aa..cae0425 100644
--- a/src/osmo-bsc/assignment_fsm.c
+++ b/src/osmo-bsc/assignment_fsm.c
@@ -442,7 +442,13 @@
 		return true; /* unknown channel mode */
 	}
 
-	return (feat->val & flag) != 0;
+	if (feat->val & flag)
+		return true;
+
+	LOG_ASSIGNMENT(lchan->conn, LOGL_NOTICE,
+		       "BTS does not support channel mode '%s'\n",
+		       get_value_string(abis_nm_ipacc_chanm_desc, flag));
+	return false;
 }
 
 static bool lchan_type_compat_with_mode(const struct gsm_lchan *lchan,

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