fixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/34756?usp=email )

Change subject: firmware/layer1: emit TRAFFIC.ind even if B_BFI is set ......................................................................

firmware/layer1: emit TRAFFIC.ind even if B_BFI is set

Even if the DSP marks a traffic frame as bad (B_BFI), we still want to deliver something to the upper layers, just like we do for FACCH.

Change-Id: I559793a3506089b1c1758ee7022cceb7753afb30 Related: OS#4396 --- M src/target/firmware/layer1/prim_tch.c 1 file changed, 15 insertions(+), 2 deletions(-)

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

diff --git a/src/target/firmware/layer1/prim_tch.c b/src/target/firmware/layer1/prim_tch.c index fb7ebf4..c6fbddd 100644 --- a/src/target/firmware/layer1/prim_tch.c +++ b/src/target/firmware/layer1/prim_tch.c @@ -313,8 +313,6 @@ goto skip_rx_traffic; if (~traffic_buf[0] & (1 << B_BLUD)) goto skip_rx_traffic; - if (~traffic_buf[0] & (1 << B_BFI)) - goto skip_rx_traffic;

/* Allocate msgb */ /* FIXME: we actually want all allocation out of L1S! */ @@ -341,6 +339,8 @@ else dl->num_biterr = num_biterr;

+ dl->fire_crc = ((traffic_buf[0] & 0xffff) & ((1 << B_FIRE1) | (1 << B_FIRE0))) >> B_FIRE0; + /* Update rx level for pm report */ pu_update_rx_level(dl->rx_level);