Discussion:
[PATCH v2 0/5] lte atom proto and authentication
Jonas Bonn
2018-10-12 08:25:19 UTC
Permalink
Wow... 45 minutes since your last submission and two minutes since I
sent my last review email.  Impressive speed! ;)  Like I said, slow down...

Since I doubt any of my comments are incorporated here, I'll just wait
for v3 before reading them again...
Second version of the lte atom, with selection of protocol and authentication.
In this version the settings are applied as soon as a property is changed.
Include a long description here of what problem you are trying to solve
and what approach you are using to solve that problem. Reading this
description should be sufficient for me to be able to understand where
the following sequence of patches is taking me.

For follow-up versions, include a little section "Changes in v2" that
tells me how v2 differs from the series v1 that I, possibly, have
already reviewed.

/Jonas
lte-api: protocol and authentication properties
lte.h: added proto and authentication handling
src/lte: added proto and authentication handling
atmodem/lte: proto and authentication handling
Gemalto contributors so far
AUTHORS | 2 +
doc/lte-api.txt | 39 +++++++
drivers/atmodem/lte.c | 92 ++++++++++++++--
include/lte.h | 5 +
src/lte.c | 236 ++++++++++++++++++++++++++++++++----------
5 files changed, 307 insertions(+), 67 deletions(-)
Loading...