d/changelog: release version 0.4.2-1

Update to upstream version '0.4.2'
with Debian dir ad2c13f51c

README.md

@@ -15,7 +15,7 @@ It implements the following features:
 
 `eg25-manager` requires the following development libraries:
 - libglib2.0-dev
-- libgpiod-dev (>= 2.0)
+- libgpiod-dev
 - libmm-glib-dev
 
 ## Building

data/eg25-manager.service.in

@@ -4,7 +4,7 @@ Before=ModemManager.service
 
 [Service]
 Type=simple
-ExecStart=@bindir@/eg25-manager
+ExecStart=@bindir@/eg25manager
 Restart=on-failure
 ProtectControlGroups=true
 ProtectHome=true

data/pine64,pinephone-1.0.toml

@@ -1,5 +1,4 @@
 [manager]
-monitor_udev = true
 need_libusb = true
 usb_vid = 0x2c7c
 usb_pid = 0x0125

data/pine64,pinephone-1.1.toml

@@ -1,5 +1,4 @@
 [manager]
-monitor_udev = true
 need_libusb = true
 usb_vid = 0x2c7c
 usb_pid = 0x0125

data/pine64,pinephone-1.2.toml

@@ -1,7 +1,4 @@
 [manager]
-monitor_udev = true
-usb_vid = 0x2c7c
-usb_pid = 0x0125
 # Delay between setting GPIO and PWRKEY sequence, set in microseconds
 poweron_delay = 100000
 

data/pine64,pinephone-pro.toml

@@ -1,7 +1,4 @@
 [manager]
-monitor_udev = false
-usb_vid = 0x2c7c
-usb_pid = 0x0125
 # Delay between setting GPIO and PWRKEY sequence, set in microseconds
 poweron_delay = 100000
 
@@ -38,7 +35,7 @@ configure = [
 # Print software version
     { cmd = "QGMR" },
 # Configure audio
-    { cmd = "QDAI", expect = "3,0,0,4,0,1,1,1" },
+    { cmd = "QDAI", expect = "3,0,0,4,0,0,1,1" },
 # RI signaling using physical RI pin
     { cmd = "QCFG", subcmd = "risignaltype", expect = "\"physical\"" },
 # Enable VoLTE support

debian/changelog

@@ -0,0 +1,112 @@
+eg25-manager (0.4.2-1) unstable; urgency=medium
+
+  * d/gbp.conf: update for current Mobian workflow
+  * New upstream version 0.4.2
+
+ -- Arnaud Ferraris <arnaud.ferraris@collabora.com>  Wed, 08 Dec 2021 18:24:38 +0100
+
+eg25-manager (0.4.1-1) unstable; urgency=medium
+
+  * New upstream version 0.4.1
+  * debian: drop distro-specific systemd service.
+  * d/copyright: add missing entries
+
+ -- Arnaud Ferraris <arnaud.ferraris@collabora.com>  Fri, 08 Oct 2021 11:02:27 +0200
+
+eg25-manager (0.4.0-1) unstable; urgency=medium
+
+  [ undef ]
+  * d/service: Use systemd to sandbox eg25-manager.
+  * d/salsa-ci: Add Mobian's CI
+
+  [ Arnaud Ferraris ]
+  * New upstream version 0.4.0
+  * d/eg25-manager.service: be less restrictive.
+  * d/control: add libcurl as build dependency
+
+ -- Arnaud Ferraris <arnaud.ferraris@gmail.com>  Wed, 01 Sep 2021 00:44:04 +0200
+
+eg25-manager (0.3.0-1) unstable; urgency=medium
+
+  * New upstream version 0.3.0
+
+ -- Arnaud Ferraris <arnaud.ferraris@collabora.com>  Fri, 28 May 2021 13:58:33 +0200
+
+eg25-manager (0.2.1-1) unstable; urgency=medium
+
+  * New upstream version 0.2.1
+
+ -- Arnaud Ferraris <arnaud.ferraris@gmail.com>  Sun, 21 Feb 2021 16:41:31 +0100
+
+eg25-manager (0.2.0-1) unstable; urgency=medium
+
+  * New upstream version 0.2.0
+  * d/gbp.conf: enable multimaint-merge
+  * d/eg25-manager.service: remove deprecated -g option
+
+ -- Arnaud Ferraris <arnaud.ferraris@gmail.com>  Sat, 20 Feb 2021 22:26:19 +0100
+
+eg25-manager (0.1.2-1) unstable; urgency=medium
+
+  * New upstream release 0.1.2
+  * d/eg25-manager.service: enable GNSS management
+
+ -- Arnaud Ferraris <arnaud.ferraris@gmail.com>  Thu, 14 Jan 2021 00:09:23 +0100
+
+eg25-manager (0.1.1-1) unstable; urgency=medium
+
+  * d/control: build-depend on gudev
+  * New upstream release 0.1.1
+
+ -- Arnaud Ferraris <arnaud.ferraris@gmail.com>  Fri, 18 Dec 2020 01:42:06 +0100
+
+eg25-manager (0.1.0-1) unstable; urgency=medium
+
+  * New upstream release 0.1.0
+
+ -- Arnaud Ferraris <arnaud.ferraris@gmail.com>  Mon, 14 Dec 2020 16:45:20 +0100
+
+eg25-manager (0.0.6-1) unstable; urgency=medium
+
+  * New upstream release 0.0.6
+  * d/patches: drop upstreamed patches
+
+ -- Arnaud Ferraris <arnaud.ferraris@gmail.com>  Fri, 11 Dec 2020 15:11:26 +0100
+
+eg25-manager (0.0.5-2) unstable; urgency=medium
+
+  * d/patches: fix crash on modem recovery
+
+ -- Arnaud Ferraris <arnaud.ferraris@gmail.com>  Fri, 11 Dec 2020 14:33:14 +0100
+
+eg25-manager (0.0.5-1) unstable; urgency=medium
+
+  * New upstream release 0.0.5
+
+ -- Arnaud Ferraris <arnaud.ferraris@gmail.com>  Fri, 11 Dec 2020 13:38:41 +0100
+
+eg25-manager (0.0.4-1) unstable; urgency=medium
+
+  * d/control: build only on arm64.
+  * d/service: restart daemon on failure
+  * d/control: build-depend on libusb-1.0
+
+ -- Arnaud Ferraris <arnaud.ferraris@gmail.com>  Fri, 11 Dec 2020 12:51:41 +0100
+
+eg25-manager (0.0.3-1) unstable; urgency=medium
+
+  * New upstream release 0.0.3
+
+ -- Arnaud Ferraris <arnaud.ferraris@gmail.com>  Thu, 10 Dec 2020 21:34:26 +0100
+
+eg25-manager (0.0.2-1) unstable; urgency=medium
+
+  * New upstream release 0.0.2
+
+ -- Arnaud Ferraris <arnaud.ferraris@gmail.com>  Thu, 10 Dec 2020 19:50:50 +0100
+
+eg25-manager (0.0.1-1) unstable; urgency=medium
+
+  * Initial Debian packaging
+
+ -- Arnaud Ferraris <arnaud.ferraris@gmail.com>  Thu, 10 Dec 2020 15:19:15 +0100

debian/control

@@ -0,0 +1,28 @@
+Source: eg25-manager
+Section: libs
+Priority: optional
+Maintainer: Arnaud Ferraris <arnaud.ferraris@gmail.com>
+Standards-Version: 4.5.1
+Rules-Requires-Root: no
+Build-Depends: debhelper-compat (= 13),
+               libcurl-dev,
+               libglib2.0-dev,
+               libgpiod-dev,
+               libgudev-1.0-dev,
+               libmm-glib-dev,
+               libusb-1.0-0-dev,
+               meson,
+Homepage: https://gitlab.com/mobian1/devices/eg25-manager
+Vcs-Git: https://gitlab.com/mobian1/devices/eg25-manager.git
+Vcs-Browser: https://gitlab.com/mobian1/devices/eg25-manager
+
+Package: eg25-manager
+Architecture: arm64
+Depends: ${misc:Depends},
+         ${shlibs:Depends},
+         modemmanager,
+Conflicts: pinephone-modem-scripts,
+Replaces: pinephone-modem-scripts,
+Description: Manager daemon for the Quectel EG25 mobile broadband modem
+ A set of scripts for the PinePhone modem, allowing to power on/off the modem,
+ and configure its audio interface to work properly with the A64 audio codec.

debian/copyright

@@ -0,0 +1,67 @@
+Format: https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/
+Upstream-Name: eg25-manager
+Upstream-Contact: Arnaud Ferraris <arnaud.ferraris@gmail.com>
+Source: https://gitlab.com/mobian1/devices/eg25-manager
+
+Files: *
+Copyright: 2020 Arnaud Ferraris <arnaud.ferraris@gmail.com>
+License: GPL-3.0-or-later
+
+Files: src/gnss.*
+Copyright: 2021 Dylan Van Assche <me@dylanvanassche.be>
+License: GPL-3.0-or-later
+
+Files: src/mm-iface.c
+Copyright: 2019 Purism SPC
+           2020 Arnaud Ferraris <arnaud.ferraris@gmail.com>
+License: GPL-3.0-or-later
+
+Files: src/ofono-iface.c
+Copyright: 2020 Oliver Smith <ollieparanoid@postmarketos.org>
+           2021 Bhushan Shah <bshah@kde.org>
+License: GPL-3.0-or-later
+
+Files: src/suspend.c
+Copyright: 2012 Red Hat, Inc
+           2020 Arnaud Ferraris <arnaud.ferraris@gmail.com>
+License: GPL-3.0-or-later
+
+Files: src/toml.*
+Copyright: 2017-2019 CK Tan
+License: MIT
+
+License: GPL-3.0-or-later
+ This program is free software: you can redistribute it and/or modify
+ it under the terms of the GNU General Public License as published by
+ the Free Software Foundation, either version 3 of the License, or
+ (at your option) any later version.
+ .
+ This program is distributed in the hope that it will be useful,
+ but WITHOUT ANY WARRANTY; without even the implied warranty of
+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ GNU General Public License for more details.
+ .
+ You should have received a copy of the GNU General Public License
+ along with this program.  If not, see <https://www.gnu.org/licenses/>.
+ .
+ On Debian systems, the full text of the GNU General Public License
+ version 3 can be found in the file `/usr/share/common-licenses/GPL-3'.
+
+License: MIT
+ Permission is hereby granted, free of charge, to any person obtaining a copy
+ of this software and associated documentation files (the "Software"), to deal
+ in the Software without restriction, including without limitation the rights
+ to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+ copies of the Software, and to permit persons to whom the Software is
+ furnished to do so, subject to the following conditions:
+ .
+ The above copyright notice and this permission notice shall be included in all
+ copies or substantial portions of the Software.
+ .
+ THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+ AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+ SOFTWARE.

debian/gbp.conf

@@ -0,0 +1,10 @@
+[DEFAULT]
+debian-branch = mobian
+debian-tag = mobian/%(version)s
+upstream-branch = upstream/latest
+upstream-tag = %(version)s
+pristine-tar = True
+multimaint-merge = True
+
+[tag]
+sign-tags = True

debian/rules

@@ -0,0 +1,6 @@
+#!/usr/bin/make -f
+
+export DEB_BUILD_MAINT_OPTIONS = hardening=+all
+
+%:
+	dh $@

debian/salsa-ci.yml

@@ -0,0 +1,2 @@
+include:
+    - https://gitlab.com/mobian1/packaging-tools/-/raw/master/mobian-ci.yml

debian/source/format

@@ -0,0 +1 @@
+3.0 (quilt)

debian/watch

@@ -0,0 +1,5 @@
+
+
+version=4
+opts=filenamemangle=s/.*\/archive\/(\d\S+)\/eg25-manager.*\.tar\.gz/eg25-manager-$1\.tar\.gz/g \
+   https://gitlab.com/mobian1/eg25-manager/tags?sort=updated_desc .*/archive/(\d\S+)/.*\.tar\.gz.*

doc/eg25-manager.5.scd

@@ -1,106 +0,0 @@
-eg25-manager(5)
-
-# NAME
-eg25-manager configuration file format
-
-# SYNOPSIS
-eg25-manager uses toml formatted files for configuration.
-
-Configurations are loaded from:
-	- *@eg25_confdir@/<compatible>.toml*: User-provided overrides (optional)
-	- *@eg25_datadir@/<compatible>.toml*: Default configuration (required)
-
-# SECTION: manager
-General settings for eg25-manager.
-
-*poweron_delay* int (microseconds)
-	Delay between de-asserting RESET and starting the PWRKEY sequence.
-
-# SECTION: suspend
-Settings for how to handle suspend on the system where eg25-manager is running.
-
-*boot_timeout* int (seconds)
-	Prevent the system from suspending for boot_timeout seconds to allow the
-	modem to fully boot.
-
-	Default: 120 if unset or zero.
-
-*recovery_timeout* int (seconds)
-	Amount of time to wait for the modem to reappear after suspend. If the
-	timeout is reached the modem's USB connection will be reset.
-
-	Default: 9 if unset or zero.
-
-# SECTION: at
-AT commands to send when different events happen, and where to send them to.
-
-Each command has 4 possible elements:
-	- *cmd*: the AT command itself, which will be translated to "AT+`cmd`"
-	- *subcmd*: the subcommand in case a single AT command can be used to
-	  change multiple parameters, such as QCFG
-	- *value*: the command's argument(s), usually used to set the value of a
-	  specific parameter
-	- *expect*: the expected return value; the command is first executed
-	  without any value in order to query the current state. This state is
-	  then compared to the *expect* string; if they don't match, the command
-	  is then executed with value *expect* in order to set the parameter to
-	  the expected value
-A command can have *expect* OR *value* configured, but it shouldn't have both
-
-*NOTE:* If a command sequence is configured in an override file, the default
-commands won't be loaded from the system configuration. The default commands
-should be copied into the override file when changing them.
-
-*uart* string
-	The serial port to use for sending AT commands to the modem.
-
-*configure* List of commands
-	AT commands to send to the modem when it is first started.
-
-*suspend* List of commands
-	AT commands to send to the modem before the system suspends.
-
-*resume* List of commands
-	AT commands to send to the modem after the system resumes from suspend.
-
-*reset* List of commands
-	AT commands to send to the modem if resetting the usb port fails.
-
-# SECTION: gnss
-Settings for uploading AGPS assistance data to the modem.
-
-*enabled* boolean
-	Enable or disable uploading AGPS data to the modem
-
-*url* string
-	The directory on the server that contains the assistance files
-
-	Example: https://xtrapath4.izatcloud.net
-
-*file* string
-	The name of the assistance file on the server.
-
-	Example: xtra2.bin
-
-# SECTION: gpio
-The *gpio* section defines the GPIO pins to use for different modem functions.
-These settings should only be changed when porting eg25-manager to a new device;
-for this reason they aren't documented here.
-
-# EXAMPLES
-Print the firmware version every time the phone wakes from suspend:
-```
-[at]
-resume = [
-    { cmd = "QGMR" },
-]
-```
-
-Disable uploading AGPS data to the modem:
-```
-[gnss]
-enabled = false
-```
-
-# SEE AlSO
-*eg25-manager*(8)

doc/eg25-manager.8.scd

@@ -1,37 +0,0 @@
-eg25-manager(8)
-
-# NAME
-eg25-manager - a daemon for managing the Quectel EG25 modem found on the
-Pine64 PinePhone.
-
-# SYNOPSIS
-*eg25-manager* [-v] [-c config_file]
-
-# OPTIONS
-*-v*
-	Show the version number and quit.
-*-c*
-	User configuration file, defaults to the device configuration file in
-	/etc/eg25-manager.
-
-# FILES
-Configurations are loaded from:
-	- *@eg25_confdir@/<compatible>.toml*: User-provided overrides (optional)
-	- *@eg25_datadir@/<compatible>.toml*: Default configuration (required)
-
-eg25-manager will search these folders for files named after the value of the
-compatible device-tree property (with the .toml file extension) and use the
-first matching file in each directory. If no matching default configuration is
-found, eg25-manager will exit with an error message.
-
-Values from the user-provided overrides will take priority over values stored in
-the default configuration. Only changed values must be stored as user overrides,
-so eg25-manager can fall back to the default configuration as often as possible.
-
-The file names eg25-manager will check can be listed using:
-```
-xargs -0 printf '%s.toml\\n' < /proc/device-tree/compatible
-```
-
-# SEE ALSO
-*eg25-manager*(5) *ModemManager*(8) *ofono*(8)

doc/meson.build

@@ -1,33 +0,0 @@
-#
-# Copyright (C) 2020 Arnaud Ferraris <arnaud.ferraris@gmail.com>
-#
-# SPDX-License-Identifier: GPL-3.0-or-later
-#
-
-scdoc = dependency('scdoc', native: true, required: false)
-if scdoc.found()
-  scdoc_prog = find_program(scdoc.get_variable('scdoc'), native: true)
-
-  foreach section: [5, 8]
-    name = 'eg25-manager'
-    out = '@0@.@1@'.format(name, section)
-
-    preprocessed = configure_file(
-      input: '@0@.scd'.format(out),
-      output: '@BASENAME@.preprocessed',
-      configuration: {
-        'eg25_confdir': eg25_confdir,
-        'eg25_datadir': eg25_datadir,
-        }
-      )
-
-    custom_target(
-      out,
-      output: out,
-      input: preprocessed,
-      command: ['sh', '-c', '@0@ < @INPUT@'.format(scdoc_prog.full_path())],
-      capture: true,
-      install: true,
-      install_dir: join_paths(get_option('mandir'), 'man@0@'.format(section)))
-  endforeach
-endif

meson.build

@@ -8,9 +8,9 @@
 project (
   'eg25-manager',
   'c',
-  version : '0.5.1',
+  version : '0.4.2',
   license : 'GPLv3+',
-  meson_version : '>= 0.58.0',
+  meson_version : '>= 0.50.0',
   default_options :
     [
       'warning_level=1',
@@ -59,13 +59,12 @@ mgr_deps = [
     dependency('glib-2.0'),
     dependency('gio-unix-2.0'),
     dependency('gudev-1.0'),
-    dependency('libgpiod', version: '>= 2.0'),
+    dependency('libgpiod'),
     dependency('libusb-1.0'),
     dependency('libcurl'),
     mmglib_dep,
 ]
 
 subdir('data')
-subdir('doc')
 subdir('src')
 subdir('udev')

src/at.c

@@ -73,10 +73,10 @@ gboolean at_send_command(struct EG25Manager *manager)
     int ret, len = 0, pos = 0;
 
     if (at_cmd) {
-        /* Wake up the modem from soft sleep before sending an AT command */
-        gpio_sequence_wake(manager);
+    	/* Wake up the modem from soft sleep before sending an AT command */
+    	gpio_sequence_wake(manager);
 
-        /* Send AT command */
+	/* Send AT command */
         if (at_cmd->subcmd == NULL && at_cmd->value == NULL && at_cmd->expected == NULL)
             len = snprintf(command, sizeof(command), "AT+%s\r\n", at_cmd->cmd);
         else if (at_cmd->subcmd == NULL && at_cmd->value == NULL)
@@ -126,8 +126,8 @@ gboolean at_send_command(struct EG25Manager *manager)
 
         g_message("Successfully sent command: %s", g_strstrip(command));
     } else {
-        /* Allow the modem to enter soft sleep again when we sent the AT command*/
-        gpio_sequence_sleep(manager);
+    	/* Allow the modem to enter soft sleep again when we sent the AT command*/
+    	gpio_sequence_sleep(manager);
 
         if (manager->modem_state < EG25_STATE_CONFIGURED) {
             if (manager->modem_iface == MODEM_IFACE_MODEMMANAGER) {
@@ -146,6 +146,8 @@ gboolean at_send_command(struct EG25Manager *manager)
             }
         } else if (manager->modem_state == EG25_STATE_SUSPENDING) {
             modem_suspend_post(manager);
+        } else if (manager->modem_state == EG25_STATE_RESETTING) {
+            manager->modem_state = EG25_STATE_POWERED;
         }
     }
 

src/gnss.c

@@ -327,7 +327,7 @@ static void init_assistance_data_upload(struct EG25Manager *manager)
 static void upload_assistance_data(struct EG25Manager *manager)
 {
     gint error;
-    off_t written_total = 0;
+    glong written_total = 0;
     gint ret;
     struct stat sb;
 

src/gpio.c

@@ -8,8 +8,6 @@
 #include "gpio.h"
 
 #include <unistd.h>
-#include <dirent.h>
-#include <sys/stat.h>
 
 /* Those defines are used for legacy config files only */
 #define GPIO_CHIP1_LABEL "1c20800.pinctrl"
@@ -42,77 +40,21 @@ static char *gpio_in_names[] = {
     "status",
 };
 
-enum gpiod_line_value gpio_line_get_value(struct EG25Manager *manager, int line) {
-    enum gpiod_line_value value;
-    unsigned int offset;
-
-    gpiod_line_request_get_requested_offsets(manager->gpio_in[line], &offset, 1);
-    value = gpiod_line_request_get_value(manager->gpio_in[line], offset);
-
-    if (value == GPIOD_LINE_VALUE_ERROR) {
-            g_warning("gpio: couldn't get value on line %d", line);
-    }
-
-    return value;
-}
-
-int gpio_line_set_value(struct EG25Manager *manager, int line, enum gpiod_line_value value) {
-    unsigned int offset;
-    int ret;
-
-    gpiod_line_request_get_requested_offsets(manager->gpio_out[line], &offset, 1);
-    ret = gpiod_line_request_set_value(manager->gpio_out[line], offset, value);
-    if (ret) {
-        g_warning("gpio: couldn't set value %d on line %d", value, line);
-        return -1;
-    }
-    else {
-        manager->gpio_out_value[line] = value;
-        return 0;
-    }
-}
-
 int gpio_sequence_poweron(struct EG25Manager *manager)
 {
-    /*
-     * Force the modem to poweroff using the RESET_N pin before attempting to
-     * boot in case the it got into a bad state.
-     *
-     * If the modem was on, this will cause it to start booting, so press the
-     * power button while in reset to avoid a (probably only theoretical) race
-     * condition where it starts booting after reset, and then powers off from
-     * the power key.
-     */
-    gpio_line_set_value(manager, GPIO_OUT_RESET, GPIOD_LINE_VALUE_ACTIVE);
-    gpio_line_set_value(manager, GPIO_OUT_PWRKEY, GPIOD_LINE_VALUE_ACTIVE);
-
-    /*
-     * The datasheet says to pull the pin low for between 150 and 460 ms. usleep
-     * should always sleep for at least the specified amount of time, so use
-     * 200ms because it's closer to the bottom of that range.
-     */
-    usleep(200000);
-
-    gpio_line_set_value(manager, GPIO_OUT_RESET, GPIOD_LINE_VALUE_INACTIVE);
-
-    /*
-     * The modem has finished it's reset, now we wait to allow it a chance to
-     * react to the power key
-     */
+    gpiod_line_set_value(manager->gpio_out[GPIO_OUT_PWRKEY], 1);
     sleep(1);
+    gpiod_line_set_value(manager->gpio_out[GPIO_OUT_PWRKEY], 0);
 
-    gpio_line_set_value(manager, GPIO_OUT_PWRKEY, GPIOD_LINE_VALUE_INACTIVE);
-
-    g_message("Executed power-on sequence");
+    g_message("Executed power-on/off sequence");
 
     return 0;
 }
 
 int gpio_sequence_shutdown(struct EG25Manager *manager)
 {
-    gpio_line_set_value(manager, GPIO_OUT_PWRKEY, GPIOD_LINE_VALUE_ACTIVE);
-    sleep(1);
-    gpio_line_set_value(manager, GPIO_OUT_PWRKEY, GPIOD_LINE_VALUE_INACTIVE);
+    gpiod_line_set_value(manager->gpio_out[GPIO_OUT_DISABLE], 1);
+    gpio_sequence_poweron(manager);
 
     g_message("Executed power-off sequence");
 
@@ -121,7 +63,7 @@ int gpio_sequence_shutdown(struct EG25Manager *manager)
 
 int gpio_sequence_suspend(struct EG25Manager *manager)
 {
-    gpio_line_set_value(manager, GPIO_OUT_APREADY, GPIOD_LINE_VALUE_ACTIVE);
+    gpiod_line_set_value(manager->gpio_out[GPIO_OUT_APREADY], 1);
 
     g_message("Executed suspend sequence");
 
@@ -130,7 +72,7 @@ int gpio_sequence_suspend(struct EG25Manager *manager)
 
 int gpio_sequence_resume(struct EG25Manager *manager)
 {
-    gpio_line_set_value(manager, GPIO_OUT_APREADY, GPIOD_LINE_VALUE_INACTIVE);
+    gpiod_line_set_value(manager->gpio_out[GPIO_OUT_APREADY], 0);
 
     g_message("Executed resume sequence");
 
@@ -139,8 +81,8 @@ int gpio_sequence_resume(struct EG25Manager *manager)
 
 int gpio_sequence_wake(struct EG25Manager *manager)
 {
-    if (manager->gpio_out_value[GPIO_OUT_DTR]) {
-        gpio_line_set_value(manager, GPIO_OUT_DTR, GPIOD_LINE_VALUE_INACTIVE);
+    if (gpiod_line_get_value(manager->gpio_out[GPIO_OUT_DTR])) {
+        gpiod_line_set_value(manager->gpio_out[GPIO_OUT_DTR], 0);
 
         /* Give the modem 200ms to wake from soft sleep */
         usleep(200000);
@@ -153,150 +95,42 @@ int gpio_sequence_wake(struct EG25Manager *manager)
 
 int gpio_sequence_sleep(struct EG25Manager *manager)
 {
-    gpio_line_set_value(manager, GPIO_OUT_DTR, GPIOD_LINE_VALUE_ACTIVE);
+    gpiod_line_set_value(manager->gpio_out[GPIO_OUT_DTR], 1);
     g_message("Executed soft sleep sequence");
 
     return 0;
 }
 
-struct gpiod_line_request *gpio_request_line(struct EG25Manager *manager, int chip, unsigned int line, enum gpiod_line_direction direction) {
-    struct gpiod_line_request *request = NULL;
-    struct gpiod_line_settings *settings;
-    struct gpiod_line_config *line_cfg;
-    struct gpiod_request_config *req_cfg;
-    int ret;
+struct gpiod_line *gpio_get_output_line(struct EG25Manager *manager, int chip, int line)
+{
+    struct gpiod_line *gpio_line;
 
-    settings = gpiod_line_settings_new();
-    if (!settings)
+    gpio_line = gpiod_chip_get_line(manager->gpiochip[chip], line);
+    if (!gpio_line)
         return NULL;
 
-    gpiod_line_settings_set_direction(settings, direction);
-
-    line_cfg = gpiod_line_config_new();
-    if (!line_cfg)
-        goto free_settings;
-
-    ret = gpiod_line_config_add_line_settings(line_cfg, &line, 1, settings);
-    if (ret)
-        goto free_line_config;
-
-    req_cfg = gpiod_request_config_new();
-    if (!req_cfg)
-        goto free_line_config;
-
-    gpiod_request_config_set_consumer(req_cfg, "eg25-manager");
-
-    request = gpiod_chip_request_lines(manager->gpiochip[chip], req_cfg, line_cfg);
-
-    gpiod_request_config_free(req_cfg);
-
-free_line_config:
-    gpiod_line_config_free(line_cfg);
-
-free_settings:
-    gpiod_line_settings_free(settings);
-
-    return request;
-}
-
-static int gpio_chip_dir_filter(const struct dirent *entry)
-{
-    struct stat sb;
-    int ret = 0;
-    char *path;
-
-    if (asprintf(&path, "/dev/%s", entry->d_name) < 0)
-        return 0;
-
-    if ((lstat(path, &sb) == 0) && (!S_ISLNK(sb.st_mode)) &&
-        gpiod_is_gpiochip_device(path))
-        ret = 1;
-
-    free(path);
-
-    return ret;
-}
-
-int gpio_all_chip_paths(char ***paths_ptr)
-{
-    int i, j, num_chips, ret = 0;
-    struct dirent **entries;
-    char **paths;
-
-    num_chips = scandir("/dev/", &entries, gpio_chip_dir_filter, alphasort);
-    if (num_chips < 0)
-        g_error("gpio: unable to scan /dev: %s", g_strerror(errno));
-
-    paths = calloc(num_chips, sizeof(*paths));
-    if (paths == NULL)
-        g_error("gpio: out of memory");
-
-    for (i = 0; i < num_chips; i++) {
-        if (asprintf(&paths[i], "/dev/%s", entries[i]->d_name) < 0) {
-            for (j = 0; j < i; j++)
-                free(paths[j]);
-
-            free(paths);
-            return 0;
-        }
+    if (gpiod_line_request_output(gpio_line, "eg25manager", 0) < 0) {
+        gpiod_line_release(gpio_line);
+        return NULL;
     }
 
-    *paths_ptr = paths;
-    ret = num_chips;
-
-    for (i = 0; i < num_chips; i++)
-        free(entries[i]);
-
-    free(entries);
-    return ret;
+    return gpio_line;
 }
 
-struct gpiod_chip *gpio_chip_open_by_label(const char *label)
+struct gpiod_line *gpio_get_input_line(struct EG25Manager *manager, int chip, int line)
 {
-    int num_chips, i;
-    char **paths;
-    const char *clabel;
-    struct gpiod_chip *chip;
-    struct gpiod_chip_info *cinfo;
+    struct gpiod_line *gpio_line;
 
-    num_chips = gpio_all_chip_paths(&paths);
+    gpio_line = gpiod_chip_get_line(manager->gpiochip[chip], line);
+    if (!gpio_line)
+        return NULL;
 
-    for (i = 0; i < num_chips; i++) {
-        chip = gpiod_chip_open(paths[i]);
-        if (!chip)
-            continue;
-
-        cinfo = gpiod_chip_get_info(chip);
-        if (!cinfo)
-            goto clean_chip_open;
-
-        clabel = gpiod_chip_info_get_label(cinfo);
-
-        if (strcmp(label, clabel) == 0) {
-            return chip;
-        }
-
-clean_chip_open:
-        gpiod_chip_close(chip);
+    if (gpiod_line_request_input(gpio_line, "eg25manager") < 0) {
+        gpiod_line_release(gpio_line);
+        return NULL;
     }
 
-    return NULL;
-}
-
-unsigned int gpio_chip_num_lines(struct EG25Manager *manager, unsigned int chip_num) {
-    struct gpiod_chip *chip = manager->gpiochip[chip_num];
-    struct gpiod_chip_info *info;
-    unsigned int num_lines;
-
-    info = gpiod_chip_get_info(chip);
-    if (!info)
-        g_error("gpio: failed to read info: %s", strerror(errno));
-
-    num_lines = gpiod_chip_info_get_num_lines(info);
-
-    gpiod_chip_info_free(info);
-
-    return num_lines;
+    return gpio_line;
 }
 
 int gpio_init(struct EG25Manager *manager, toml_table_t *config[])
@@ -335,7 +169,7 @@ int gpio_init(struct EG25Manager *manager, toml_table_t *config[])
             toml_datum_t data = toml_string_at(chipslist, i);
             if (!data.ok)
                 continue;
-            manager->gpiochip[i] = gpio_chip_open_by_label(data.u.s);
+            manager->gpiochip[i] = gpiod_chip_open_by_label(data.u.s);
             if (!manager->gpiochip[i])
                 g_error("Unable to find GPIO chip '%s'", data.u.s);
         }
@@ -351,10 +185,10 @@ int gpio_init(struct EG25Manager *manager, toml_table_t *config[])
                 g_error("Wrong chip ID for output GPIO '%s'", gpio_out_names[i]);
 
             line = toml_int_in(table, "line");
-            if (!line.ok || line.u.i < 0 || line.u.i > gpio_chip_num_lines(manager, chip.u.i))
+            if (!line.ok || line.u.i < 0 || line.u.i > gpiod_chip_num_lines(manager->gpiochip[chip.u.i]))
                 g_error("Wrong line ID for output GPIO '%s'", gpio_out_names[i]);
 
-            manager->gpio_out[i] = gpio_request_line(manager, chip.u.i, line.u.i, GPIOD_LINE_DIRECTION_OUTPUT);
+            manager->gpio_out[i] = gpio_get_output_line(manager, chip.u.i, line.u.i);
             if (!manager->gpio_out[i])
                 g_error("Unable to get output GPIO %d", i);
         }
@@ -362,23 +196,18 @@ int gpio_init(struct EG25Manager *manager, toml_table_t *config[])
         for (i = 0; i < GPIO_IN_COUNT; i++) {
             toml_table_t *table;
             toml_datum_t chip, line;
-
-            if (!config_get_table(gpio_config, gpio_in_names[i], &table)) {
-                // BH edition don't have the STATUS line connected, ignore it
-                if (manager->use_libusb && g_strcmp0(gpio_in_names[i], "status") == 0)
-                    continue;
+            if (!config_get_table(gpio_config, gpio_in_names[i], &table))
                 g_error("Unable to get config for input GPIO '%s'", gpio_in_names[i]);
-            }
 
             chip = toml_int_in(table, "chip");
             if (!chip.ok || chip.u.i < 0 || chip.u.i > 2)
                 g_error("Wrong chip ID for input GPIO '%s'", gpio_in_names[i]);
 
             line = toml_int_in(table, "line");
-            if (!line.ok || line.u.i < 0 || line.u.i > gpio_chip_num_lines(manager, chip.u.i))
+            if (!line.ok || line.u.i < 0 || line.u.i > gpiod_chip_num_lines(manager->gpiochip[chip.u.i]))
                 g_error("Wrong line ID for input GPIO '%s'", gpio_in_names[i]);
 
-            manager->gpio_in[i] = gpio_request_line(manager, chip.u.i, line.u.i, GPIOD_LINE_DIRECTION_INPUT);
+            manager->gpio_in[i] = gpio_get_input_line(manager, chip.u.i, line.u.i);
             if (!manager->gpio_in[i])
                 g_error("Unable to get input GPIO %d", i);
         }
@@ -386,11 +215,11 @@ int gpio_init(struct EG25Manager *manager, toml_table_t *config[])
         guint offset, chipidx, gpio_idx;
 
         /* Legacy config file, only used on the OG PinePhone */
-        manager->gpiochip[0] = gpio_chip_open_by_label(GPIO_CHIP1_LABEL);
+        manager->gpiochip[0] = gpiod_chip_open_by_label(GPIO_CHIP1_LABEL);
         if (!manager->gpiochip[0])
             g_error("Unable to open GPIO chip " GPIO_CHIP1_LABEL);
 
-        manager->gpiochip[1] = gpio_chip_open_by_label(GPIO_CHIP2_LABEL);
+        manager->gpiochip[1] = gpiod_chip_open_by_label(GPIO_CHIP2_LABEL);
         if (!manager->gpiochip[1])
             g_error("Unable to open GPIO chip " GPIO_CHIP2_LABEL);
 
@@ -406,7 +235,7 @@ int gpio_init(struct EG25Manager *manager, toml_table_t *config[])
                 chipidx = 1;
             }
 
-            manager->gpio_out[i] = gpio_request_line(manager, chipidx, offset, GPIOD_LINE_DIRECTION_OUTPUT);
+            manager->gpio_out[i] = gpio_get_input_line(manager, chipidx, offset);
             if (!manager->gpio_out[i])
                 g_error("Unable to get output GPIO %d", i);
         }
@@ -423,7 +252,7 @@ int gpio_init(struct EG25Manager *manager, toml_table_t *config[])
                 chipidx = 1;
             }
 
-            manager->gpio_in[i] = gpio_request_line(manager, chipidx, offset, GPIOD_LINE_DIRECTION_INPUT);
+            manager->gpio_in[i] = gpio_get_input_line(manager, chipidx, offset);
             if (!manager->gpio_in[i])
                 g_error("Unable to get input GPIO %d", i);
         }
@@ -432,18 +261,16 @@ int gpio_init(struct EG25Manager *manager, toml_table_t *config[])
     return 0;
 }
 
-void gpio_force_off(struct EG25Manager *manager)
-{
-    if (manager->gpio_out[GPIO_OUT_RESET]) {
-        g_message("Setting the reset pin to ensure the modem stays off");
-        gpio_line_set_value(manager, GPIO_OUT_RESET, GPIOD_LINE_VALUE_ACTIVE);
-    }
-}
-
-gboolean gpio_check_poweroff(struct EG25Manager *manager)
+gboolean gpio_check_poweroff(struct EG25Manager *manager, gboolean keep_down)
 {
     if (manager->gpio_in[GPIO_IN_STATUS] &&
-        gpio_line_get_value(manager, GPIO_IN_STATUS) == GPIOD_LINE_VALUE_ACTIVE) {
+        gpiod_line_get_value(manager->gpio_in[GPIO_IN_STATUS]) == 1) {
+
+        if (keep_down && manager->gpio_out[GPIO_OUT_RESET]) {
+            // Asserting RESET line to prevent modem from rebooting
+            gpiod_line_set_value(manager->gpio_out[GPIO_OUT_RESET], 1);
+        }
+
         return TRUE;
     }
 
@@ -456,12 +283,12 @@ void gpio_destroy(struct EG25Manager *manager)
 
     for (i = 0; i < GPIO_OUT_COUNT; i++) {
         if (manager->gpio_out[i])
-            gpiod_line_request_release(manager->gpio_out[i]);
+            gpiod_line_release(manager->gpio_out[i]);
     }
 
     for (i = 0; i < GPIO_IN_COUNT; i++) {
         if (manager->gpio_in[i])
-            gpiod_line_request_release(manager->gpio_in[i]);
+            gpiod_line_release(manager->gpio_in[i]);
     }
 
     if (manager->gpiochip[0])

src/gpio.h

@@ -18,5 +18,4 @@ int gpio_sequence_resume(struct EG25Manager *state);
 int gpio_sequence_wake(struct EG25Manager *state);
 int gpio_sequence_sleep(struct EG25Manager *state);
 
-void gpio_force_off(struct EG25Manager *manager);
-gboolean gpio_check_poweroff(struct EG25Manager *manager);
+gboolean gpio_check_poweroff(struct EG25Manager *manager, gboolean keep_down);

src/manager.c

@@ -38,9 +38,6 @@
 #define EG25_VERSION "0.0.0"
 #endif
 
-#define EG25_DEFAULT_VENDOR_ID  0x2c7c
-#define EG25_DEFAULT_PRODUCT_ID 0x0125
-
 #define POWERON_DELAY_US 100000UL
 
 static gboolean quit_app(struct EG25Manager *manager)
@@ -62,17 +59,13 @@ static gboolean quit_app(struct EG25Manager *manager)
         gpio_sequence_shutdown(manager);
         manager->modem_state = EG25_STATE_FINISHING;
         for (i = 0; i < 30; i++) {
-            if (gpio_check_poweroff(manager)) {
-                g_message("Modem successfully powered down");
+            if (gpio_check_poweroff(manager, TRUE))
                 break;
-            }
             sleep(1);
         }
     }
-
-    gpio_force_off(manager);
-
     g_message("Modem down, quitting...");
+
     g_main_loop_quit(manager->loop);
 
     return FALSE;
@@ -102,7 +95,7 @@ static gboolean modem_start(struct EG25Manager *manager)
 
         libusb_free_device_list(devices, 1);
         libusb_exit(ctx);
-    } else if (!gpio_check_poweroff(manager)) {
+    } else if (!gpio_check_poweroff(manager, FALSE)) {
         g_message("STATUS is low, modem already powered");
         should_boot = FALSE;
     }
@@ -145,38 +138,10 @@ void modem_configure(struct EG25Manager *manager)
     at_sequence_configure(manager);
 }
 
-static gboolean modem_gpio_reset_done(struct EG25Manager *manager)
-{
-    gpio_sequence_poweron(manager);
-    manager->modem_state = EG25_STATE_POWERED;
-    manager->complete_reset_timer = 0;
-
-    return G_SOURCE_REMOVE;
-}
-
-static gboolean modem_at_reset_done(struct EG25Manager* manager)
-{
-    /*
-     * If the modem was successfully rebooted, then we should have received
-     * "RDY" by now which will transition the state to started.
-     */
-    if (manager->modem_state != EG25_STATE_RESETTING) {
-        manager->complete_reset_timer = 0;
-        return G_SOURCE_REMOVE;
-    }
-
-    g_message("AT reset failed, falling back to GPIO reset");
-
-    gpio_sequence_shutdown(manager);
-    manager->complete_reset_timer = g_timeout_add_seconds(30, G_SOURCE_FUNC(modem_gpio_reset_done), manager);
-
-    return G_SOURCE_REMOVE;
-}
-
-static gboolean modem_rebind_done(struct EG25Manager* manager)
+static gboolean modem_reset_done(struct EG25Manager* manager)
 {
     manager->modem_state = EG25_STATE_RESUMING;
-    manager->complete_reset_timer = 0;
+    manager->reset_timer = 0;
     return FALSE;
 }
 
@@ -184,18 +149,7 @@ gboolean modem_reset(struct EG25Manager *manager)
 {
     int fd, ret, len;
 
-    /* reset sequence started, cannot be canceled anymore */
-    if (manager->schedule_reset_timer) {
-        g_source_remove(manager->schedule_reset_timer);
-        manager->schedule_reset_timer = 0;
-    }
-
-    if (manager->modem_recovery_timer) {
-        g_source_remove(manager->modem_recovery_timer);
-        manager->modem_recovery_timer = 0;
-    }
-
-    if (manager->complete_reset_timer) {
+    if (manager->reset_timer) {
         g_message("modem_reset: timer already setup, skipping...");
         return G_SOURCE_REMOVE;
     }
@@ -210,6 +164,11 @@ gboolean modem_reset(struct EG25Manager *manager)
         return G_SOURCE_REMOVE;
     }
 
+    if (manager->modem_recovery_timer) {
+        g_source_remove(manager->modem_recovery_timer);
+        manager->modem_recovery_timer = 0;
+    }
+
     if (!manager->modem_usb_id) {
         g_warning("Empty modem USB ID");
         goto error;
@@ -226,26 +185,24 @@ gboolean modem_reset(struct EG25Manager *manager)
         g_warning("Unable to open /sys/bus/usb/drivers/usb/unbind");
         goto error;
     }
-
     ret = write(fd, manager->modem_usb_id, len);
-    close(fd);
     if (ret < len) {
         g_warning("Couldn't unbind modem: wrote %d/%d bytes", ret, len);
         goto error;
     }
+    close(fd);
 
     fd = open("/sys/bus/usb/drivers/usb/bind", O_WRONLY);
     if (fd < 0) {
-        g_warning("Unable to open /sys/bus/usb/drivers/usb/bind");
+        g_warning("Unable to open /sys/bus/usb/drivers/usb/unbind");
         goto error;
     }
-
     ret = write(fd, manager->modem_usb_id, len);
-    close(fd);
     if (ret < len) {
         g_warning("Couldn't bind modem: wrote %d/%d bytes", ret, len);
         goto error;
     }
+    close(fd);
 
     g_message("Successfully reset modem's USB connection");
 
@@ -253,7 +210,7 @@ gboolean modem_reset(struct EG25Manager *manager)
      * 3s is long enough to make sure the modem has been bound back and
      * short enough to ensure it hasn't been acquired by ModemManager
      */
-    manager->complete_reset_timer = g_timeout_add_seconds(3, G_SOURCE_FUNC(modem_rebind_done), manager);
+    manager->reset_timer = g_timeout_add_seconds(3, G_SOURCE_FUNC(modem_reset_done), manager);
 
     return G_SOURCE_REMOVE;
 
@@ -271,7 +228,7 @@ error:
     at_sequence_reset(manager);
 
     // Setup timer for making sure we don't queue other reset commands
-    manager->complete_reset_timer = g_timeout_add_seconds(45, G_SOURCE_FUNC(modem_at_reset_done), manager);
+    manager->reset_timer = g_timeout_add_seconds(30, G_SOURCE_FUNC(modem_reset_done), manager);
 
     return G_SOURCE_REMOVE;
 }
@@ -357,7 +314,6 @@ int main(int argc, char *argv[])
     struct EG25Manager manager;
     gchar *config_file = NULL;
     gboolean show_version = FALSE;
-    gboolean monitor_udev = TRUE;
     toml_table_t *toml_config[EG25_CONFIG_COUNT];
     toml_table_t *manager_config[EG25_CONFIG_COUNT];
     const GOptionEntry options[] = {
@@ -407,13 +363,10 @@ int main(int argc, char *argv[])
     if (!manager_config[EG25_CONFIG_SYS])
         g_error("Default config file lacks the 'manager' section!");
 
-    config_get_bool(manager_config, "monitor_udev", &monitor_udev);
     config_get_bool(manager_config, "need_libusb", &manager.use_libusb);
+    config_get_uint(manager_config, "usb_vid", &manager.usb_vid);
+    config_get_uint(manager_config, "usb_pid", &manager.usb_pid);
     config_get_uint(manager_config, "poweron_delay", &manager.poweron_delay);
-    if (!config_get_uint(manager_config, "usb_vid", &manager.usb_vid))
-        manager.usb_vid = EG25_DEFAULT_VENDOR_ID;
-    if (!config_get_uint(manager_config, "usb_pid", &manager.usb_pid))
-        manager.usb_pid = EG25_DEFAULT_PRODUCT_ID;
 
     at_init(&manager, toml_config);
     gpio_init(&manager, toml_config);
@@ -422,8 +375,7 @@ int main(int argc, char *argv[])
 #endif
     ofono_iface_init(&manager, toml_config);
     suspend_init(&manager, toml_config);
-    if (monitor_udev)
-        udev_init(&manager, toml_config);
+    udev_init(&manager, toml_config);
     gnss_init(&manager, toml_config);
 
     for (int i = 0; i < EG25_CONFIG_COUNT; i++) {

src/manager.h

@@ -53,7 +53,6 @@ enum EG25State {
     EG25_STATE_SUSPENDING, // System is going into suspend
     EG25_STATE_RESUMING, // System is being resumed, waiting for modem to come back
     EG25_STATE_RESETTING, // Something went wrong, we're restarting the modem
-    EG25_STATE_UPDATING, // Modem is present but being updated
     EG25_STATE_FINISHING
 };
 
@@ -71,8 +70,7 @@ enum EG25Config {
 
 struct EG25Manager {
     GMainLoop *loop;
-    guint complete_reset_timer;
-    guint schedule_reset_timer;
+    guint reset_timer;
     gboolean use_libusb;
     guint usb_vid;
     guint usb_pid;
@@ -116,9 +114,8 @@ struct EG25Manager {
     GUdevClient *udev;
 
     struct gpiod_chip *gpiochip[2];
-    struct gpiod_line_request *gpio_out[5];
-    guint gpio_out_value[5];
-    struct gpiod_line_request *gpio_in[2];
+    struct gpiod_line *gpio_out[5];
+    struct gpiod_line *gpio_in[2];
 };
 
 void modem_configure(struct EG25Manager *data);

src/meson.build

@@ -25,7 +25,7 @@ if mmglib_dep.found()
 endif
 
 executable (
-    'eg25-manager',
+    'eg25manager',
     src,
     dependencies : mgr_deps,
     link_with: gdbofono_lib,

src/suspend.c

@@ -18,24 +18,10 @@
 #define SD_PATH         "/org/freedesktop/login1"
 #define SD_INTERFACE    "org.freedesktop.login1.Manager"
 
-static void resume_ok(struct EG25Manager *manager)
-{
-    manager->modem_state = EG25_STATE_CONFIGURED;
-    modem_resume_post(manager);
-}
-
 static gboolean check_modem_resume(struct EG25Manager *manager)
 {
-    manager->modem_recovery_timer = 0;
-
-#ifdef HAVE_MMGLIB
-    if (manager->mm_modem) {
-        resume_ok(manager);
-        return FALSE;
-    }
-#endif
-
     g_message("Modem wasn't probed in time, restart it!");
+    manager->modem_recovery_timer = 0;
     modem_reset(manager);
 
     return FALSE;
@@ -198,7 +184,8 @@ static void signal_cb(GDBusProxy *proxy,
              * If modem is managed by ofono, we also do resume sequence immediately
              * as ofono handles resuming from sleep itself.
              */
-            resume_ok(manager);
+            manager->modem_state = EG25_STATE_CONFIGURED;
+            modem_resume_post(manager);
         } else {
             manager->modem_state = EG25_STATE_RESUMING;
             manager->modem_recovery_timer = g_timeout_add_seconds(manager->modem_recovery_timeout,

src/udev.c

@@ -11,47 +11,18 @@
 static void udev_event_cb(GUdevClient *client, gchar *action, GUdevDevice *device, gpointer data)
 {
     struct EG25Manager *manager = data;
-    const gchar *prop;
-    long vid = 0, pid = 0;
-
-    /*
-     * Act only if the device is the one identified as a modem by MM/ofono
-     */
-    if (!manager->modem_usb_id ||
-        strcmp(g_udev_device_get_name(device), manager->modem_usb_id) != 0) {
-        return;
-    }
-
-    prop = g_udev_device_get_property(device, "ID_VENDOR_ID");
-    if (prop)
-        vid = strtol(prop, NULL, 16);
-
-    prop = g_udev_device_get_property(device, "ID_MODEL_ID");
-    if (prop)
-        pid = strtol(prop, NULL, 16);
-
-    if (strcmp(action, "bind") == 0 && vid != manager->usb_vid && pid != manager->usb_pid) {
-        /*
-         * Modem is probably executing a FW upgrade, make sure we don't interrupt it
-         */
-        if (manager->schedule_reset_timer != 0) {
-            g_message("Modem re-appeared with different VID/PID, cancel reset.");
-            g_source_remove(manager->schedule_reset_timer);
-            manager->schedule_reset_timer = 0;
-        }
-        manager->modem_state = EG25_STATE_UPDATING;
-    }
 
     if (strcmp(action, "unbind") != 0 ||
-        manager->modem_state == EG25_STATE_UPDATING ||
         manager->modem_state == EG25_STATE_RESETTING ||
-        manager->complete_reset_timer != 0 ||
-        manager->schedule_reset_timer != 0) {
+        !manager->modem_usb_id) {
         return;
-    }
+        }
 
-    g_message("Lost modem, resetting...");
-    manager->schedule_reset_timer = g_timeout_add_seconds(3, G_SOURCE_FUNC(modem_reset), manager);
+    if (strcmp(g_udev_device_get_name(device), manager->modem_usb_id) == 0 &&
+        manager->reset_timer == 0) {
+        g_message("Lost modem, resetting...");
+        g_timeout_add_seconds(2, G_SOURCE_FUNC(modem_reset), manager);
+    }
 }
 
 void udev_init (struct EG25Manager *manager, toml_table_t *config[])

udev/80-modem-eg25.rules

@@ -1,18 +1,4 @@
-ACTION=="add", SUBSYSTEM=="usb", DRIVERS=="usb", ENV{DEVTYPE}=="usb_device", GOTO="eg25_start"
-GOTO="eg25_end"
-
-# Default attributes values
-LABEL="eg25_start"
-ATTRS{idVendor}=="2c7c", ATTRS{idProduct}=="0125", ATTR{power/control}="auto"
-ATTRS{idVendor}=="2c7c", ATTRS{idProduct}=="0125", ATTR{power/autosuspend_delay_ms}="3000"
-ATTRS{idVendor}=="2c7c", ATTRS{idProduct}=="0125", ATTR{power/wakeup}="enabled"
-ATTRS{idVendor}=="2c7c", ATTRS{idProduct}=="0125", ATTR{power/persist}="0"
-
-# power/control needs to be "on" for the community-maintained firmware
-ATTRS{serial}=="community_fw", ATTRS{idVendor}=="2c7c", ATTRS{idProduct}=="0125", ATTR{power/control}="on"
-
-# Special trick for the PinePhone Pro: set power/persist to 1 *only* with the community FW
-# We can identify the PPP by looking for the string "pinephone-pro" in the device tree "compatible" property
-ATTRS{serial}=="community_fw", ATTRS{idVendor}=="2c7c", ATTRS{idProduct}=="0125", PROGRAM=="/bin/grep pine64,pinephone-pro /proc/device-tree/compatible", ATTR{power/persist}="1"
-
-LABEL="eg25_end"
+ACTION=="add", SUBSYSTEM=="usb", DRIVERS=="usb", ENV{DEVTYPE}=="usb_device", ATTRS{idVendor}=="2c7c", ATTRS{idProduct}=="0125", ATTR{power/control}="auto"
+ACTION=="add", SUBSYSTEM=="usb", DRIVERS=="usb", ENV{DEVTYPE}=="usb_device", ATTRS{idVendor}=="2c7c", ATTRS{idProduct}=="0125", ATTR{power/autosuspend_delay_ms}="3000"
+ACTION=="add", SUBSYSTEM=="usb", DRIVERS=="usb", ENV{DEVTYPE}=="usb_device", ATTRS{idVendor}=="2c7c", ATTRS{idProduct}=="0125", ATTR{power/wakeup}="enabled"
+ACTION=="add", SUBSYSTEM=="usb", DRIVERS=="usb", ENV{DEVTYPE}=="usb_device", ATTRS{idVendor}=="2c7c", ATTRS{idProduct}=="0125", ATTR{power/persist}="0"