Patch Set 2: (1 comment)
> Is this really necessary on sysmoBTS only? What about other BTS > models? I'll check and provide separate patches for each of them. https://gerrit.osmocom.org/#/c/3056/1/src/osmo-bts-sysmo/l1_if.c File src/osmo-bts-sysmo/l1_if.c: Line 969: } > I think comment right next to the check explaining why we do it that way is It makes sense. I'm still working on this so I may even send a new version with more changes on it (like handling packets considered low quality in l1if_tch_rx in a different way, see https://osmocom.org/issues/2335#note-4 for more context. -- To view, visit https://gerrit.osmocom.org/3056 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: If5df8940fab833eb4e3ed851880b66987d356031 Gerrit-PatchSet: 2 Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-Owner: Pau Espin Pedrol <pes...@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