pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmocom-bb/+/34129 )
Change subject: l1ctl: Fix fill ph_data_param fn field ......................................................................
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);