Re: cinterion radio bands

2020-05-22 Thread Aleksander Morgado
Hey, > Therefore I thing there must be something wrong with my memory > management in the new code, but I don't see where, nor Aleksander's > review highlighted some problems in that sense. > Once I review the new changes you do, I'll take a look with memory management in mind to see where the

cinterion radio bands

2020-05-22 Thread Giacinto Cifelli
Dear Aleksander, all, I am starting a new thread because I think the previous one was filtered as spam, maybe when I added the log? I collected some more modems and did more tests, and with an ALS3 (supported maybe as PLS8), which uses the single AT^SCFG: "Radio/Band" value for all technologies,

Re: Cinterion radio bands

2020-05-20 Thread Giacinto Cifelli
Hey Aleksander, thank you for your answer. On Wed, May 20, 2020 at 11:37 AM Aleksander Morgado wrote: > > Hey Giacinto, > > > I have finally submitted a merge request for the band reading and > > setting for Cinterion Modems. > > > > Not sure whether is it good quality, thou. I would appreciate

Re: Cinterion radio bands

2020-05-20 Thread Aleksander Morgado
Hey Giacinto, > I have finally submitted a merge request for the band reading and > setting for Cinterion Modems. > > Not sure whether is it good quality, thou. I would appreciate a review > and comments. > Starting for example from the naming of the udev variable, and then > the use of other

Re: Cinterion radio bands

2020-05-19 Thread Giacinto Cifelli
Hey Aleksander, all, I have finally submitted a merge request for the band reading and setting for Cinterion Modems. Not sure whether is it good quality, thou. I would appreciate a review and comments. Starting for example from the naming of the udev variable, and then the use of other variables

Re: Cinterion radio bands

2020-05-18 Thread Aleksander Morgado
Hey, > > > > > > > > > I am trying to update the code for the Radio/Bands of > > > > > > > > > Cinterion > > > > > > > > > Today in the code there is a parser for this format (only for > > > > > > > > > the test part): > > > > > > > > > AT^SCFG=? > > > > > > > > > ^SCFG:

Re: Cinterion radio bands

2020-05-18 Thread Giacinto Cifelli
Hey Aleksander, On Mon, May 18, 2020 at 1:11 PM Aleksander Morgado wrote: > > Hey, > > > > > > > > > I am trying to update the code for the Radio/Bands of Cinterion > > > > > > > > Today in the code there is a parser for this format (only for > > > > > > > > the test part): > > > > > > > >

Re: Cinterion radio bands

2020-05-18 Thread Aleksander Morgado
Hey, > > > > > > > I am trying to update the code for the Radio/Bands of Cinterion > > > > > > > Today in the code there is a parser for this format (only for the > > > > > > > test part): > > > > > > > AT^SCFG=? > > > > > > > ^SCFG: "Radio/Band",("1-479","0-1") > > > > > > > > > > > > >

Re: Cinterion radio bands

2020-05-18 Thread Giacinto Cifelli
Hi Aleksander, On Mon, May 18, 2020 at 9:04 AM Aleksander Morgado wrote: > > Hey, > > > On Fri, May 15, 2020 at 9:02 PM Aleksander Morgado > > wrote: > > > > > > Hey, > > > > > > I am trying to update the code for the Radio/Bands of Cinterion > > > > > > Today in the code there is a parser for

Re: Cinterion radio bands

2020-05-18 Thread Aleksander Morgado
Hey, > On Fri, May 15, 2020 at 9:02 PM Aleksander Morgado > wrote: > > > > Hey, > > > > > I am trying to update the code for the Radio/Bands of Cinterion > > > > > Today in the code there is a parser for this format (only for the > > > > > test part): > > > > > AT^SCFG=? > > > > >

Re: Cinterion radio bands

2020-05-17 Thread Giacinto Cifelli
Hey Aleksander, On Fri, May 15, 2020 at 9:02 PM Aleksander Morgado wrote: > > Hey, > > > > I am trying to update the code for the Radio/Bands of Cinterion > > > > Today in the code there is a parser for this format (only for the test > > > > part): > > > > AT^SCFG=? > > > > ^SCFG:

Re: Cinterion radio bands

2020-05-15 Thread Aleksander Morgado
Hey, > > > I am trying to update the code for the Radio/Bands of Cinterion > > > modules, and it seems a nasty job, because of the way the indicators > > > evolved. > > > Today in the code there is a parser for this format (only for the test > > > part): > > > AT^SCFG=? > > > ... > > >

Re: Cinterion radio bands

2020-05-15 Thread Giacinto Cifelli
Hey Aleksander, thank you for your hints. On Fri, May 15, 2020 at 1:49 PM Aleksander Morgado wrote: > > Hey! > > > I am trying to update the code for the Radio/Bands of Cinterion > > modules, and it seems a nasty job, because of the way the indicators > > evolved. > > Today in the code there is

Re: Cinterion radio bands

2020-05-15 Thread Aleksander Morgado
Hey! > I am trying to update the code for the Radio/Bands of Cinterion > modules, and it seems a nasty job, because of the way the indicators > evolved. > Today in the code there is a parser for this format (only for the test part): > AT^SCFG=? > ... > ^SCFG:

Cinterion radio bands

2020-05-15 Thread Giacinto Cifelli
Dear all, I am trying to update the code for the Radio/Bands of Cinterion modules, and it seems a nasty job, because of the way the indicators evolved. Today in the code there is a parser for this format (only for the test part): AT^SCFG=? ... ^SCFG: "Radio/Band",("1-479","0-1")