laforge submitted this change.

View Change


Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved
DAHDI: Log hexdump of TX at level DEBUG, not ERROR

This fixes Change-Id I447a2360757fed97ed50f9db1e2efbf2f90e46a0 where
log messages received more context. However, by accident, one log
statement got elevated from DEBUG to ERROR level. Let's revert that
accident.

Change-Id: I29aa82c8586d846a861e62d90d376ff9cfaa9654
---
M src/input/dahdi.c
1 file changed, 15 insertions(+), 1 deletion(-)

diff --git a/src/input/dahdi.c b/src/input/dahdi.c
index efed38d..b9a3fcf 100644
--- a/src/input/dahdi.c
+++ b/src/input/dahdi.c
@@ -259,7 +259,7 @@
return 0;
}

- LOGPITS(e1i_ts, DLMI, LOGL_ERROR, "TX: %s\n", osmo_hexdump(msg->data, msg->len));
+ LOGPITS(e1i_ts, DLMI, LOGL_DEBUG, "TX: %s\n", osmo_hexdump(msg->data, msg->len));
lapd_transmit(e1i_ts->lapd, sign_link->tei,
sign_link->sapi, msg);


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

Gerrit-Project: libosmo-abis
Gerrit-Branch: master
Gerrit-Change-Id: I29aa82c8586d846a861e62d90d376ff9cfaa9654
Gerrit-Change-Number: 35494
Gerrit-PatchSet: 2
Gerrit-Owner: keith <keith@rhizomatica.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-MessageType: merged