Marcel Holtmann <mar...@holtmann.org> wrote: > Hi Sjur, > >> Changes since last patch set: >> o Moved updates to plugins/ste to separate patch. >> o Updated error handling. >> o Style issues. >> >> >> Makefile.am | 4 +- >> drivers/stemodem/radio-settings.c | 223 >> +++++++++++++++++++++++++++++++++++++ >> drivers/stemodem/stemodem.c | 2 + >> drivers/stemodem/stemodem.h | 2 + >> 4 files changed, 230 insertions(+), 1 deletions(-) >> create mode 100644 drivers/stemodem/radio-settings.c > > is this identical to how MBM does it? If yes, then I prefer we add this > to drivers/mbmmodem/ and you just reference it. If not then I like to > see the MBM version of this and where it actually differs. Maybe a MBM > version with a STE quirk is better.
Actually this is not yet implemented by MBM, so I guess STE is master this time ;-) > I prefer doing this in MBM since that modem driver was just the first. > There is no other preference here ;) This actually raises one question. How should we handle version variations from the same Modem vendor. The differences will be larger as we start adding support for LTE, and support different bearers (PC cards with USB and Modems with CAIF). Should we create more entries in the vendor enum, or do you have something else in mind? Regards Sjur _______________________________________________ ofono mailing list ofono@ofono.org http://lists.ofono.org/listinfo/ofono