libqmi-glib,helpers: move sysfs R/W operations to the internal helpers

1 files changed, 8 insertions, 74 deletions

diff --git a/src/libqmi-glib/qmi-device.c b/src/libqmi-glib/qmi-device.c
index f1dc83d..dad407a 100644
--- a/src/libqmi-glib/qmi-device.c
+++ b/src/libqmi-glib/qmi-device.c
@@ -766,72 +766,6 @@ qmi_device_get_wwan_iface (QmiDevice *self)
 /* Expected data format */
 
 static gboolean
-read_sysfs_file (const gchar  *sysfs_path,
-                 gchar        *out_value,
-                 GError      **error)
-{
-    FILE     *f;
-    gboolean  status = FALSE;
-
-    if (!(f = fopen (sysfs_path, "r"))) {
-        g_set_error (error, G_IO_ERROR, g_io_error_from_errno (errno),
-                     "Failed to open sysfs file '%s': %s",
-                     sysfs_path, g_strerror (errno));
-        goto out;
-    }
-
-    if (fread (out_value, 1, 1, f) != 1) {
-        g_set_error (error, G_IO_ERROR, g_io_error_from_errno (errno),
-                     "Failed to read from sysfs file '%s': %s",
-                     sysfs_path, g_strerror (errno));
-        goto out;
-    }
-
-    if (*out_value != 'Y' && *out_value != 'N') {
-        g_set_error (error, QMI_CORE_ERROR, QMI_CORE_ERROR_FAILED,
-                     "Unexpected sysfs file contents: %c", *out_value);
-        goto out;
-    }
-
-    status = TRUE;
-
- out:
-    if (f)
-        fclose (f);
-    return status;
-}
-
-static gboolean
-write_sysfs_file (const gchar  *sysfs_path,
-                  gchar         value,
-                  GError      **error)
-{
-    gboolean  status = FALSE;
-    FILE     *f;
-
-    if (!(f = fopen (sysfs_path, "w"))) {
-        g_set_error (error, G_IO_ERROR, g_io_error_from_errno (errno),
-                     "Failed to open sysfs file '%s' for R/W: %s",
-                     sysfs_path, g_strerror (errno));
-        goto out;
-    }
-
-    if (fwrite (&value, 1, 1, f) != 1) {
-        g_set_error (error, G_IO_ERROR, g_io_error_from_errno (errno),
-                     "Failed to write to sysfs file '%s': %s",
-                     sysfs_path, g_strerror (errno));
-        goto out;
-    }
-
-    status = TRUE;
-
- out:
-    if (f)
-        fclose (f);
-    return status;
-}
-
-static gboolean
 get_expected_data_format (QmiDevice    *self,
                           const gchar  *raw_ip_sysfs_path,
                           const gchar  *pass_through_sysfs_path,
@@ -840,13 +774,13 @@ get_expected_data_format (QmiDevice    *self,
     gchar raw_ip_value = '\0';
     gchar pass_through_value = '\0';
 
-    if (!read_sysfs_file (raw_ip_sysfs_path, &raw_ip_value, error))
+    if (!qmi_helpers_read_sysfs_file (raw_ip_sysfs_path, &raw_ip_value, error))
         return QMI_DEVICE_EXPECTED_DATA_FORMAT_UNKNOWN;
 
     if (raw_ip_value == 'N')
         return QMI_DEVICE_EXPECTED_DATA_FORMAT_802_3;
 
-    if (read_sysfs_file (pass_through_sysfs_path, &pass_through_value, NULL) && (pass_through_value == 'Y'))
+    if (qmi_helpers_read_sysfs_file (pass_through_sysfs_path, &pass_through_value, NULL) && (pass_through_value == 'Y'))
         return QMI_DEVICE_EXPECTED_DATA_FORMAT_QMAP_PASS_THROUGH;
 
     return QMI_DEVICE_EXPECTED_DATA_FORMAT_RAW_IP;
@@ -860,18 +794,18 @@ set_expected_data_format (QmiDevice                    *self,
                           GError                      **error)
 {
     if (requested == QMI_DEVICE_EXPECTED_DATA_FORMAT_802_3) {
-        write_sysfs_file (pass_through_sysfs_path, 'N', NULL);
-        return write_sysfs_file (raw_ip_sysfs_path, 'N', error);
+        qmi_helpers_write_sysfs_file (pass_through_sysfs_path, 'N', NULL);
+        return qmi_helpers_write_sysfs_file (raw_ip_sysfs_path, 'N', error);
     }
 
     if (requested == QMI_DEVICE_EXPECTED_DATA_FORMAT_RAW_IP) {
-        write_sysfs_file (pass_through_sysfs_path, 'N', NULL);
-        return write_sysfs_file (raw_ip_sysfs_path, 'Y', error);
+        qmi_helpers_write_sysfs_file (pass_through_sysfs_path, 'N', NULL);
+        return qmi_helpers_write_sysfs_file (raw_ip_sysfs_path, 'Y', error);
     }
 
     if (requested == QMI_DEVICE_EXPECTED_DATA_FORMAT_QMAP_PASS_THROUGH) {
-        return (write_sysfs_file (raw_ip_sysfs_path, 'Y', error) &&
-                write_sysfs_file (pass_through_sysfs_path, 'Y', error));
+        return (qmi_helpers_write_sysfs_file (raw_ip_sysfs_path, 'Y', error) &&
+                qmi_helpers_write_sysfs_file (pass_through_sysfs_path, 'Y', error));
     }
 
     g_assert_not_reached ();