Attention is currently required from: osmith, laforge, pespin. fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/16145 )
Change subject: logging: get rid of logging category DSUM ......................................................................
Patch Set 3:
(5 comments)
This change is ready for review.
Patchset:
PS3:
I wonder if we can completely remove that category.
Agree, removed in the new patchset.
File src/common/bts.c:
https://gerrit.osmocom.org/c/osmo-bts/+/16145/comment/d885e0fb_6beebce5 PS1, Line 309: LOGP(DBTS, LOGL_INFO, "Main link established, sending Status'.\n");
Could be oml
Done
https://gerrit.osmocom.org/c/osmo-bts/+/16145/comment/2fc0d35d_ecc7dbf1 PS1, Line 345: LOGP(DBTS, LOGL_INFO, "RSL link (TRX %02x) state changed to %s, sending Status'.\n",
Could be rsl
Done
https://gerrit.osmocom.org/c/osmo-bts/+/16145/comment/1bd5d075_c0616413 PS1, Line 369: LOGP(DBTS, LOGL_INFO, "TRX(%d): Setting available = %d\n",
OML too
Done
https://gerrit.osmocom.org/c/osmo-bts/+/16145/comment/c3bc06ef_8b6ba0cc PS1, Line 635: LOGP(DBTS, LOGL_ERROR,
RR?
Done
-
fixeria