<div dir="ltr">Hi all!<br><br>I would like to inform, that this task is almost finished! Just tested<br>my changes, and build was successful without any warnings. But I still<br>have a couple of unresolved issues:<br><br>I have some problems with understanding how to add a new item into the<br>convolutional codes generator (utils/conv_gen.py), so temporary I just<br>used a single file from OsmoBTS (gsm0503_conv.c) to check, if all the<br>rest forks fine. If someone (Max?) can help me to move missed tables<br>(mostly EGPRS specific, see 73cb583e5147a267a370c576e8ac77507de6d0d7),<br>I will be grateful, and the task will be completely finished soon.<br><br>Also, there are several external dependences from libosmocodec, such as:<br><br> - gsm620_unvoiced_bitorder<br> - gsm620_voiced_bitorder<br> - gsm660_bitorder<br> - gsm610_bitorder<br><br>They are some uint16_t arrays, related to GSM HR, FR and EFR. It's sadly,<br>that they cannot be included using exactly corresponding header files.<br>I think there are three possible ways to solve this problem:<br><br> 1. Link gsm620.c, gsm660.c and gsm610.c to libosmogsm sources.<br>     (I am not sure, that it is a good way...)<br> 2. Copy required arrays to libosmogsm (also unlikely).<br> 3. Add <a href="http://libosmocodec.la">libosmocodec.la</a> as dependence (I suggest exactly this solution):<br>     - libgsmint_la_LIBADD = ../<a href="http://libosmocore.la">libosmocore.la</a><br>     + libgsmint_la_LIBADD = ../<a href="http://libosmocore.la">libosmocore.la</a> ../codec/<a href="http://libosmocodec.la">libosmocodec.la</a><br><br>All other problems was solved.<br><br></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>With best regards,<br></div><div>Vadim Yanitskiy.<br></div></div></div></div></div></div>
<br><div class="gmail_quote">2016-08-04 19:51 GMT+06:00 Vadim Yanitskiy <span dir="ltr"><<a href="mailto:axilirator@gmail.com" target="_blank">axilirator@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Max,<br><br>Thanks for your reply!<br><br>> I think extending utils/conv_gen.py is the way to go because it will<br>> allow us to have concise description of the convolutional codes in one<br>> place and as close to the description in standards as possible.<br><br>Ok, I will keep your opinion in my mind.<br><br>> Could you specify which files exactly are you referring to? Also you can<br>> use 'git blame' to clarify authorship.<br><br>No problem, they are:<br><br> - gsm0503_conv.c<br> - gsm0503_interleaving.c<br> - gsm0503_mapping.c<br> - gsm0503_parity.c<br> - gsm0503_tables.c<br><br>Almost all the gsm0503_*, excluding the 'gsm0503_coding.*'.<br>Thank you for this hint, this command is very usable! :)<br><br>> I don't think library functions should do any sort of logging by itself<br>> (unless it's a logging functions of course :)<br>> Instead they should return clearly distinguishable values and let caller<br>> do the logging as they see fit.<br><br>I am agree with you. It's time to use return codes.<br><br>Have a nice day!<span class=""><br><div><br><br clear="all"><div><div data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>With best regards,<br></div><div>Vadim Yanitskiy.<br></div></div></div></div></div></div>
</div></span></div>
</blockquote></div><br></div>