[PATCH] SAP client - revisited

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/baseband-devel@lists.osmocom.org/.

Nico Golde osmocom at ngolde.de
Fri Apr 24 14:41:53 UTC 2015


Hi,
* Tomcsányi, Domonkos <domi at tomcsanyi.net> [2015-04-23 18:12]:
[...] 
> It seems like there was no movement in this topic in the last couple of years, so I decided to go ahead and integrate Nico's SAP client into the current master branch and created a patch from it.

Cool, great work!

> One thing however that I think is strange, and worth mentioning: I'm not 
> sure why Nico decided to implement the switch between phone and SAP-client 
> inside of l1ctl.c, for me it would feel better to do it in sim.c (since 
> sim.c deals with SIM activities, l1ctl should deal only with L1 stuff...also 
> the current SAP client calls back to sim.c, but receives data from l1ctl - 
> little bit confusing), but I left it as is because of not knowing exactly 
> the thoughts behind it.

I don't remember details to be honest as this is too long ago, but I think 
there was no real design decision behind this other than that the code that is 
sending traffic to the SIM was already in l1ctl and I just added the switch...

Cheers,
Nico
-- 
Nico Golde - XMPP: nion at jabber.ccc.de - GPG: 0xA0A0AAAA
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.osmocom.org/pipermail/baseband-devel/attachments/20150424/dbf2efe2/attachment.bin>


More information about the baseband-devel mailing list