fixeria has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-bts/+/26048 )


Change subject: measurement: pass *mr to repeated_dl_facch_active_decision()
......................................................................

measurement: pass *mr to repeated_dl_facch_active_decision()

Change-Id: Idbf5f95d632aa2270c49b351ad5561ca2182bf9a
---
M src/common/measurement.c
1 file changed, 3 insertions(+), 10 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/48/26048/1

diff --git a/src/common/measurement.c b/src/common/measurement.c
index 124e6e5..b25104e 100644
--- a/src/common/measurement.c
+++ b/src/common/measurement.c
@@ -795,9 +795,8 @@
 /* Decide if repeated FACCH should be applied or not. If RXQUAL level, that the
  * MS reports is high enough, FACCH repetition is not needed. */
 static void repeated_dl_facch_active_decision(struct gsm_lchan *lchan,
-                                             const struct gsm48_hdr *gh)
+                                             const struct gsm48_meas_res 
*meas_res)
 {
-       const struct gsm48_meas_res *meas_res;
        uint8_t upper;
        uint8_t lower;
        uint8_t rxqual;
@@ -828,14 +827,8 @@
        }

        /* Parse MS measurement results */
-       if (gh == NULL)
+       if (meas_res == NULL)
                goto out;
-       /* Check if this is a Measurement Report */
-       if (gh->proto_discr != GSM48_PDISC_RR)
-               goto out;
-       if (gh->msg_type != GSM48_MT_RR_MEAS_REP)
-               goto out;
-       meas_res = (const struct gsm48_meas_res *) gh->data;
        if (meas_res->meas_valid != 0) /* 0 = valid */
                goto out;

@@ -967,7 +960,7 @@
        if (gh)
                lchan_bs_pwr_ctrl(lchan, gh);

-       repeated_dl_facch_active_decision(lchan, gh);
+       repeated_dl_facch_active_decision(lchan, mr);

        /* Reset state for next iteration */
        lchan->tch.dtx.dl_active = false;

--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/26048
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Idbf5f95d632aa2270c49b351ad5561ca2182bf9a
Gerrit-Change-Number: 26048
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <[email protected]>
Gerrit-MessageType: newchange

Reply via email to