laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-remsim/+/28564 )

Change subject: bankd: Log not just ModemToCard but also CardToModem
......................................................................

bankd: Log not just ModemToCard but also CardToModem

This way we'll get a full trace of commands and repsonses.

Right now the log only shows one direction which is a bit odd.

Related: OS#5615
Change-Id: Ia2de0b850ce78074a411cf0edebfb3ad19253d34
---
M src/bankd/bankd_main.c
1 file changed, 1 insertion(+), 0 deletions(-)

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



diff --git a/src/bankd/bankd_main.c b/src/bankd/bankd_main.c
index 8c1c9a8..b28eec9 100644
--- a/src/bankd/bankd_main.c
+++ b/src/bankd/bankd_main.c
@@ -746,6 +746,7 @@
        if (rc < 0)
                return rc;

+       LOGW(worker, "Tx RSPRO tpduCardToModem(%s)\n", 
osmo_hexdump_nospc(rx_buf, rx_buf_len));
        /* encode response PDU and send it */
        pdu_resp = rspro_gen_TpduCard2Modem(&mdm2sim->toBankSlot, 
&mdm2sim->fromClientSlot,
                                            rx_buf, rx_buf_len);

--
To view, visit https://gerrit.osmocom.org/c/osmo-remsim/+/28564
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-remsim
Gerrit-Branch: master
Gerrit-Change-Id: Ia2de0b850ce78074a411cf0edebfb3ad19253d34
Gerrit-Change-Number: 28564
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-MessageType: merged

Reply via email to