pespin has uploaded this change for review.
l1ctl: Fix fill ph_data_param fn field
This commit fixes recent previous commit filling in the fn field. The
dl->frame_nr is network order, and we want to pass a host order integer
in the primitive. Use the tm.fn which already includes the proper value
calculated from dl->frame_nr.
Fixes: d524c17d907b39189c1c7b465b8f40db2d79f55b
Related: OS#3626
Change-Id: Id96015c8b419932abb8095c6cb85aceef34e366f
---
M src/host/layer23/src/common/l1ctl.c
1 file changed, 17 insertions(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/29/34129/1
diff --git a/src/host/layer23/src/common/l1ctl.c b/src/host/layer23/src/common/l1ctl.c
index 08fb646..b2ad505 100644
--- a/src/host/layer23/src/common/l1ctl.c
+++ b/src/host/layer23/src/common/l1ctl.c
@@ -317,7 +317,7 @@
PRIM_OP_INDICATION, msg);
pp.u.data.chan_nr = dl->chan_nr;
pp.u.data.link_id = dl->link_id;
- pp.u.data.fn = dl->frame_nr;
+ pp.u.data.fn = tm.fn;
/* send it up into LAPDm */
return lapdm_phsap_up(&pp.oph, le);
To view, visit change 34129. To unsubscribe, or for help writing mail filters, visit settings.