Patch Set 1: (1 comment)
https://gerrit.osmocom.org/#/c/3275/1/src/osmo-bts-litecell15/l1_if.c File src/osmo-bts-litecell15/l1_if.c: Line 1508: LOGP(DL1C, LOGL_DEBUG, "BTS hw support band %s\n", gsm_band_name(fl1h->hw_info.band_support)); If it's not called that often, it's probably a good idea to use INFO level. Also print then the other options you are fetching here. Maybe create a function to print the fl1h structure? -- To view, visit https://gerrit.osmocom.org/3275 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I73bccd81ec56845ba11b939937b295eb0f51b4a8 Gerrit-PatchSet: 1 Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-Owner: Max <[email protected]> Gerrit-Reviewer: Harald Welte <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: Pau Espin Pedrol <[email protected]> Gerrit-HasComments: Yes
