pespin submitted this change.

View Change

Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved pespin: Looks good to me, approved
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(-)

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 change 30898. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I9fabe39575f191ce6c893a1fecaa7ac040ef0e3e
Gerrit-Change-Number: 30898
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged