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)); > dsp and fpga version are already printed by other code. Or what do you have Does it make sense to print BAND in the same place as the others then? As all this info seems to come from the same place, it would make sense to print it together, but otherwise it's fine. Change it to INFO then :) -- 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 <msur...@sysmocom.de> Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: Max <msur...@sysmocom.de> Gerrit-Reviewer: Pau Espin Pedrol <pes...@sysmocom.de> Gerrit-HasComments: Yes
