laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/31912 ) Change subject: rsl: remove redundant gsm_lchan_name() in rsl_tx_rf_rel_ack() ...................................................................... rsl: remove redundant gsm_lchan_name() in rsl_tx_rf_rel_ack() Change-Id: I69dc11c66a774ddb7364d6ecd9bdb2e94e6cad66 --- M src/common/rsl.c 1 file changed, 10 insertions(+), 1 deletion(-) Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve laforge: Looks good to me, approved diff --git a/src/common/rsl.c b/src/common/rsl.c index 2e6e512..c7061b2 100644 --- a/src/common/rsl.c +++ b/src/common/rsl.c @@ -1295,7 +1295,7 @@ } if (!send_rel_ack) { - LOGPLCHAN(lchan, DRSL, LOGL_NOTICE, "%s not sending REL ACK\n", gsm_lchan_name(lchan)); + LOGPLCHAN(lchan, DRSL, LOGL_NOTICE, "not sending REL ACK\n"); return 0; } -- To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/31912 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-Change-Id: I69dc11c66a774ddb7364d6ecd9bdb2e94e6cad66 Gerrit-Change-Number: 31912 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge@osmocom.org> Gerrit-Reviewer: pespin <pespin@sysmocom.de> Gerrit-MessageType: merged