pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmocom-bb/+/30898 )

Change subject: modem: fix unusual argument order: *ms always first
......................................................................

modem: fix unusual argument order: *ms always first

Change-Id: I9fabe39575f191ce6c893a1fecaa7ac040ef0e3e
Related: SYS#5500
---
M src/host/layer23/src/modem/app_modem.c
1 file changed, 6 insertions(+), 6 deletions(-)

Approvals:
  Jenkins Builder: Verified
  laforge: Looks good to me, approved
  pespin: Looks good to me, approved



diff --git a/src/host/layer23/src/modem/app_modem.c 
b/src/host/layer23/src/modem/app_modem.c
index d643991..c135ff3 100644
--- a/src/host/layer23/src/modem/app_modem.c
+++ b/src/host/layer23/src/modem/app_modem.c
@@ -141,7 +141,7 @@
        return 0;
 }

-static int modem_rx_bcch(struct msgb *msg, struct osmocom_ms *ms)
+static int modem_rx_bcch(struct osmocom_ms *ms, struct msgb *msg)
 {
        const struct gsm48_system_information_type_header *si_hdr = 
msgb_l3(msg);
        const uint8_t si_type = si_hdr->system_information;
@@ -163,7 +163,7 @@
        };
 }

-static int modem_rx_imm_ass(struct msgb *msg, struct osmocom_ms *ms)
+static int modem_rx_imm_ass(struct osmocom_ms *ms, struct msgb *msg)
 {
        const struct gsm48_imm_ass *ia = msgb_l3(msg);
        uint8_t ch_type, ch_subch, ch_ts;
@@ -241,7 +241,7 @@
        return false;
 }

-static int modem_rx_ccch(struct msgb *msg, struct osmocom_ms *ms)
+static int modem_rx_ccch(struct osmocom_ms *ms, struct msgb *msg)
 {
        const struct gsm48_system_information_type_header *sih = msgb_l3(msg);

@@ -263,7 +263,7 @@

        switch (sih->system_information) {
        case GSM48_MT_RR_IMM_ASS:
-               return modem_rx_imm_ass(msg, ms);
+               return modem_rx_imm_ass(ms, msg);
        default:
                return 0;
        }
@@ -291,9 +291,9 @@

        switch (rllh->chan_nr) {
        case RSL_CHAN_PCH_AGCH:
-               return modem_rx_ccch(msg, ms);
+               return modem_rx_ccch(ms, msg);
        case RSL_CHAN_BCCH:
-               return modem_rx_bcch(msg, ms);
+               return modem_rx_bcch(ms, msg);
        default:
                return 0;
        }

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

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I9fabe39575f191ce6c893a1fecaa7ac040ef0e3e
Gerrit-Change-Number: 30898
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-MessageType: merged

Reply via email to