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); -- To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/34129 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmocom-bb Gerrit-Branch: master Gerrit-Change-Id: Id96015c8b419932abb8095c6cb85aceef34e366f Gerrit-Change-Number: 34129 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: newchange