[PATCH]: Implement SMS decoding
Gianni Tedesco
gianni at scaramanga.co.uk
Thu May 12 08:58:40 CEST 2011
On Thu, 2011-05-12 at 08:48 +0200, Harald Welte wrote:
> Hi Gianni,
>
> On Thu, May 12, 2011 at 03:27:15AM +0100, Gianni Tedesco wrote:
>
> > Attached patch implements SMS decoding.
>
> are you aware that we already have 7bit decoding routines in libosmogsm,
> as well as other SMS parsing code in OpenBSC? I would prefer if we have
> one implementation that is in the shared library and re-used by all the
> various Osmocom projects.
>
> Maybe the existing routines are not suitable right-away, then they
> should be adapted. But two implementations for the same task doesn't
> sound like a good solution to me.
No, I wasn't aware of that, I'll check it out.
I am planning to progress this work by sending back the CP-ACK, adding
an API so that the SMS can be "delivered" via a function pointer
somwhere and provide another API to send the RP-ACK +
SMS-DELIVER-REPORT. I think that should complete it? But I'm only just
getting familiar with the specs.
Will re-submit when I make the relevant changes.
Thanks for the feedback
Gianni
More information about the baseband-devel
mailing list