<div dir="ltr">Hi,<br><br>On Sun, Jan 6, 2013 at 12:34 AM, Sylvain Munaut <<a href="mailto:246tnt@gmail.com">246tnt@gmail.com</a>> wrote:<br>><br>> Hi,<br>><br>> > This a respin of my old series of patches[1], to add decoding of SMSCB<br>
> > messages on the MS side.<br>><br>> Before I dig deeper, a quick question:<br>><br>> Why in libosmocore ?  This implementation is MS side only right ? so<br>> can't be re-used to implement smscb in OpenBSC or OsmoBTS.<br>
<br><br>Inertia would be one reason -- since my first patchset, LAPDm has moved<br>from BB into libosmocore, so I just tagged along.<br><br>On the practical side, I see two reasons now:<br><br>i) if we want to hook it into LAPDm, it needs to be where lapdm.c is<br>
<br>ii) libosmocore has a testsuite, so I could just stuff some testcases in<br>there<br><br>In the future, we can also add the BTS-side implementation.  I don't<br>think they can share much code, but it might make sense to have both in<br>
the same place.<br><br>I can spin a patch series directly on top of osmocom-bb, but the<br>testcases will probably not make it.<br><br><br>Thanks,<br>Alex<br><br><br><span class=""></span><span class=""></span><span class=""></span><span class=""></span> </div>