Change in osmo-trx[master]: uhd: Improve some logging lines printing UHD pretty-print output

[laforge]

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

Change subject: uhd: Improve some logging lines printing UHD pretty-print output
......................................................................

uhd: Improve some logging lines printing UHD pretty-print output

Change-Id: If5aba28aaf8a3312d89b3e963184f9f20966d199
---
M Transceiver52M/device/uhd/UHDDevice.cpp
1 file changed, 6 insertions(+), 3 deletions(-)

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



diff --git a/Transceiver52M/device/uhd/UHDDevice.cpp 
b/Transceiver52M/device/uhd/UHDDevice.cpp
index 4a6c233..8bab882 100644
--- a/Transceiver52M/device/uhd/UHDDevice.cpp
+++ b/Transceiver52M/device/uhd/UHDDevice.cpp
@@ -566,7 +566,7 @@
        init_gains();

        // Print configuration
-       LOGC(DDEV, INFO) << "\n" << usrp_dev->get_pp_string();
+       LOGC(DDEV, INFO) << "Device configuration: " << 
usrp_dev->get_pp_string();

        if (iface == MULTI_ARFCN)
                return MULTI_ARFCN;
@@ -919,15 +919,18 @@
        std::vector<double> freqs;
        uhd::tune_result_t tres;
        uhd::tune_request_t treq = select_freq(freq, chan, tx);
+       std::string str_dir;

        if (tx) {
                tres = usrp_dev->set_tx_freq(treq, chan);
                tx_freqs[chan] = usrp_dev->get_tx_freq(chan);
+               str_dir = "Tx";
        } else {
                tres = usrp_dev->set_rx_freq(treq, chan);
                rx_freqs[chan] = usrp_dev->get_rx_freq(chan);
+               str_dir = "Rx";
        }
-       LOGC(DDEV, INFO) << "\n" << tres.to_pp_string() << std::endl;
+       LOGCHAN(chan, DDEV, INFO) << "set_freq(" << freq << ", " << str_dir << 
"): " << tres.to_pp_string() << std::endl;

        if ((chans == 1) || ((chans == 2) && dev_type == UMTRX))
                return true;
@@ -947,7 +950,7 @@
                        rx_freqs[!chan] = usrp_dev->get_rx_freq(!chan);

                }
-               LOGC(DDEV, INFO) << "\n" << tres.to_pp_string() << std::endl;
+               LOGCHAN(chan, DDEV, INFO) << "set_freq(" << freq << ", " << 
str_dir << "): " << tres.to_pp_string() << std::endl;
        }

        return true;

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

Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: If5aba28aaf8a3312d89b3e963184f9f20966d199
Gerrit-Change-Number: 16666
Gerrit-PatchSet: 4
Gerrit-Owner: pespin <pes...@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <axilira...@gmail.com>
Gerrit-Reviewer: laforge <lafo...@osmocom.org>
Gerrit-MessageType: merged