Change in osmocom-bb[master]: trxcon: use osmo_{store, load}32be() to pack / unpack TDMA fn

[fixeria]

fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmocom-bb/+/18823 )


Change subject: trxcon: use osmo_{store,load}32be() to pack / unpack TDMA fn
......................................................................

trxcon: use osmo_{store,load}32be() to pack / unpack TDMA fn

Change-Id: I9eff9b8e4b8ce9e0563a1ec3c485ab8b0f306491
---
M src/host/trxcon/trx_if.c
1 file changed, 2 insertions(+), 5 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/23/18823/1

diff --git a/src/host/trxcon/trx_if.c b/src/host/trxcon/trx_if.c
index 91f84ec..7a736a8 100644
--- a/src/host/trxcon/trx_if.c
+++ b/src/host/trxcon/trx_if.c
@@ -592,7 +592,7 @@
 	}
 
 	tn = buf[0];
-	fn = (buf[1] << 24) | (buf[2] << 16) | (buf[3] << 8) | buf[4];
+	fn = osmo_load32be(buf + 1);
 	rssi = -(int8_t) buf[5];
 	toa256 = ((int16_t) (buf[6] << 8) | buf[7]);
 
@@ -650,10 +650,7 @@
 	LOGP(DTRXD, LOGL_DEBUG, "TX burst tn=%u fn=%u pwr=%u\n", tn, fn, pwr);
 
 	buf[0] = tn;
-	buf[1] = (fn >> 24) & 0xff;
-	buf[2] = (fn >> 16) & 0xff;
-	buf[3] = (fn >>  8) & 0xff;
-	buf[4] = (fn >>  0) & 0xff;
+	osmo_store32be(fn, buf + 1);
 	buf[5] = pwr;
 
 	/* Copy ubits {0,1} */

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

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I9eff9b8e4b8ce9e0563a1ec3c485ab8b0f306491
Gerrit-Change-Number: 18823
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy at sysmocom.de>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20200614/b2fa1b6c/attachment.htm>