From: jupiter <jupiter....@gmail.com>

Hi Jonas,

Is any reason you need to add ofono_modem_set_string(modem->modem, "Quirk", 
"SARAR4"); to static gboolean setup_sim7x00 rather than directly in static 
gboolean setup_ubloxqmi(struct modem_info *modem) ?

Your patch:

--- a/plugins/udevng.c
+++ b/plugins/udevng.c
@@ -1366,6 +1366,11 @@ static gboolean setup_sim7x00(struct modem_info
*modem)
                ofono_modem_set_string(modem->modem, "Audio", audio);
        }
  +     if ((g_strcmp0(modem->vendor, "0xffff") == 0) &&
+               (g_strcmp0(modem->model, "0xffff") == 0)) {
+               ofono_modem_set_string(modem->modem, "Quirk", "SARAR4");
+       }
+


My patch:
--- a/plugins/udevng.c
+++ b/plugins/udevng.c
@@ -1194,6 +1194,45 @@ static gboolean setup_ublox(struct modem_info *modem)
    return TRUE;
 }

+static gboolean setup_ubloxqmi(struct modem_info *modem) {
..........
+   ofono_modem_set_string(modem->modem, "Device", qmi);
+   ofono_modem_set_string(modem->modem, "NetworkInterface", net);
+   ofono_modem_set_string(modem->modem, "Quirk", "SARAR4");
+   ofono_modem_set_driver(modem->modem, "gobi");
+
+   return TRUE;
+}

Thanks.

- jupiter


jupiter (4):
  Change 1: SARA R4 QMI support
  Change 2: Add SARA R4 vendor ID
  Change 3: Check SARA R4 vendor ID, need to make it conditional on the
    actions being attach and detach
  Change 4: Add SARA R4 vendor and enable vendor in gprs create

 drivers/atmodem/vendor.h |  1 +
 drivers/qmimodem/gprs.c  | 15 +++++++++++++++
 plugins/gobi.c           | 11 ++++++++++-
 plugins/udevng.c         | 41 ++++++++++++++++++++++++++++++++++++++++
 4 files changed, 67 insertions(+), 1 deletion(-)

-- 
2.17.1
_______________________________________________
ofono mailing list -- ofono@ofono.org
To unsubscribe send an email to ofono-le...@ofono.org

Reply via email to