ipc: Consistent coding style and major cleanup

Signed-off-by: Paul Kocialkowski <contact@paulk.fr>

2 files changed, 64 insertions, 94 deletions

diff --git a/samsung-ipc/devices/i9300/i9300_ipc.c b/samsung-ipc/devices/i9300/i9300.c
index b7364df..a3c22bd 100644
--- a/samsung-ipc/devices/i9300/i9300_ipc.c
+++ b/samsung-ipc/devices/i9300/i9300.c
@@ -1,7 +1,7 @@
 /*
  * This file is part of libsamsung-ipc.
  *
- * Copyright (C) 2013 Paul Kocialkowski <contact@paulk.fr>
+ * Copyright (C) 2013-2014 Paul Kocialkowski <contact@paulk.fr>
  *
  * libsamsung-ipc is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -29,22 +29,21 @@
 #include "xmm6260.h"
 #include "xmm6260_hsic.h"
 #include "xmm6260_sec_modem.h"
-#include "i9300_ipc.h"
+#include "i9300.h"
 
-int i9300_ipc_bootstrap(struct ipc_client *client)
+int i9300_boot(struct ipc_client *client)
 {
     void *modem_image_data = NULL;
     int modem_image_fd = -1;
     int modem_boot_fd = -1;
     int modem_link_fd = -1;
-
     unsigned char *p;
     int rc;
 
     if (client == NULL)
         return -1;
 
-    ipc_client_log(client, "Starting i9300 modem bootstrap");
+    ipc_client_log(client, "Starting i9300 modem boot");
 
     modem_image_fd = open(I9300_MODEM_IMAGE_DEVICE, O_RDONLY);
     if (modem_image_fd < 0) {
@@ -224,106 +223,88 @@ complete:
     return rc;
 }
 
-
-int i9300_ipc_fmt_send(struct ipc_client *client, struct ipc_message_info *request)
-{
-    return xmm6260_sec_modem_ipc_fmt_send(client, request);
-}
-
-int i9300_ipc_fmt_recv(struct ipc_client *client, struct ipc_message_info *response)
-{
-    return xmm6260_sec_modem_ipc_fmt_recv(client, response);
-}
-
-int i9300_ipc_rfs_send(struct ipc_client *client, struct ipc_message_info *request)
-{
-    return xmm6260_sec_modem_ipc_rfs_send(client, request);
-}
-
-int i9300_ipc_rfs_recv(struct ipc_client *client, struct ipc_message_info *response)
-{
-    return xmm6260_sec_modem_ipc_rfs_recv(client, response);
-}
-
-int i9300_ipc_open(void *data, int type)
+int i9300_open(void *data, int type)
 {
-    struct i9300_ipc_transport_data *transport_data;
+    struct i9300_transport_data *transport_data;
 
     if (data == NULL)
         return -1;
 
-    transport_data = (struct i9300_ipc_transport_data *) data;
+    transport_data = (struct i9300_transport_data *) data;
 
-    transport_data->fd = xmm6260_sec_modem_ipc_open(type);
+    transport_data->fd = xmm6260_sec_modem_open(type);
     if (transport_data->fd < 0)
         return -1;
 
     return 0;
 }
 
-int i9300_ipc_close(void *data)
+int i9300_close(void *data)
 {
-    struct i9300_ipc_transport_data *transport_data;
+    struct i9300_transport_data *transport_data;
 
     if (data == NULL)
         return -1;
 
-    transport_data = (struct i9300_ipc_transport_data *) data;
+    transport_data = (struct i9300_transport_data *) data;
 
-    xmm6260_sec_modem_ipc_close(transport_data->fd);
+    xmm6260_sec_modem_close(transport_data->fd);
     transport_data->fd = -1;
 
     return 0;
 }
 
-int i9300_ipc_read(void *data, void *buffer, unsigned int length)
+int i9300_read(void *data, void *buffer, size_t length)
 {
-    struct i9300_ipc_transport_data *transport_data;
+    struct i9300_transport_data *transport_data;
     int rc;
 
     if (data == NULL)
         return -1;
 
-    transport_data = (struct i9300_ipc_transport_data *) data;
+    transport_data = (struct i9300_transport_data *) data;
+
+    rc = xmm6260_sec_modem_read(transport_data->fd, buffer, length);
 
-    rc = xmm6260_sec_modem_ipc_read(transport_data->fd, buffer, length);
     return rc;
 }
 
-int i9300_ipc_write(void *data, void *buffer, unsigned int length)
+int i9300_write(void *data, const void *buffer, size_t length)
 {
-    struct i9300_ipc_transport_data *transport_data;
+    struct i9300_transport_data *transport_data;
     int rc;
 
     if (data == NULL)
         return -1;
 
-    transport_data = (struct i9300_ipc_transport_data *) data;
+    transport_data = (struct i9300_transport_data *) data;
+
+    rc = xmm6260_sec_modem_write(transport_data->fd, buffer, length);
 
-    rc = xmm6260_sec_modem_ipc_write(transport_data->fd, buffer, length);
     return rc;
 }
 
-int i9300_ipc_poll(void *data, struct timeval *timeout)
+int i9300_poll(void *data, struct timeval *timeout)
 {
-    struct i9300_ipc_transport_data *transport_data;
+    struct i9300_transport_data *transport_data;
     int rc;
 
     if (data == NULL)
         return -1;
 
-    transport_data = (struct i9300_ipc_transport_data *) data;
+    transport_data = (struct i9300_transport_data *) data;
+
+    rc = xmm6260_sec_modem_poll(transport_data->fd, timeout);
 
-    rc = xmm6260_sec_modem_ipc_poll(transport_data->fd, timeout);
     return rc;
 }
 
-int i9300_ipc_power_on(void *data)
+int i9300_power_on(void *data)
 {
     return 0;
 }
 
-int i9300_ipc_power_off(void *data)
+int i9300_power_off(void *data)
 {
     int fd;
     int rc;
@@ -342,18 +323,18 @@ int i9300_ipc_power_off(void *data)
     return 0;
 }
 
-int i9300_ipc_data_create(void **transport_data, void **power_data, void **gprs_data)
+int i9300_data_create(void **transport_data, void **power_data,
+    void **gprs_data)
 {
     if (transport_data == NULL)
         return -1;
 
-    *transport_data = (void *) malloc(sizeof(struct i9300_ipc_transport_data));
-    memset(*transport_data, 0, sizeof(struct i9300_ipc_transport_data));
+    *transport_data = calloc(1, sizeof(struct i9300_transport_data));
 
     return 0;
 }
 
-int i9300_ipc_data_destroy(void *transport_data, void *power_data, void *gprs_data)
+int i9300_data_destroy(void *transport_data, void *power_data, void *gprs_data)
 {
     if (transport_data == NULL)
         return -1;
@@ -363,52 +344,41 @@ int i9300_ipc_data_destroy(void *transport_data, void *power_data, void *gprs_da
     return 0;
 }
 
-char *i9300_ipc_gprs_get_iface(int cid)
-{
-    return xmm6260_sec_modem_ipc_gprs_get_iface(cid);
-}
-
-
-int i9300_ipc_gprs_get_capabilities(struct ipc_client_gprs_capabilities *capabilities)
-{
-    return xmm6260_sec_modem_ipc_gprs_get_capabilities(capabilities);
-}
-
-struct ipc_ops i9300_ipc_fmt_ops = {
-    .bootstrap = i9300_ipc_bootstrap,
-    .send = i9300_ipc_fmt_send,
-    .recv = i9300_ipc_fmt_recv,
+struct ipc_client_ops i9300_fmt_ops = {
+    .boot = i9300_boot,
+    .send = xmm6260_sec_modem_fmt_send,
+    .recv = xmm6260_sec_modem_fmt_recv,
 };
 
-struct ipc_ops i9300_ipc_rfs_ops = {
-    .bootstrap = NULL,
-    .send = i9300_ipc_rfs_send,
-    .recv = i9300_ipc_rfs_recv,
+struct ipc_client_ops i9300_rfs_ops = {
+    .boot = NULL,
+    .send = xmm6260_sec_modem_rfs_send,
+    .recv = xmm6260_sec_modem_rfs_recv,
 };
 
-struct ipc_handlers i9300_ipc_handlers = {
-    .read = i9300_ipc_read,
-    .write = i9300_ipc_write,
-    .open = i9300_ipc_open,
-    .close = i9300_ipc_close,
-    .poll = i9300_ipc_poll,
+struct ipc_client_handlers i9300_handlers = {
+    .read = i9300_read,
+    .write = i9300_write,
+    .open = i9300_open,
+    .close = i9300_close,
+    .poll = i9300_poll,
     .transport_data = NULL,
-    .power_on = i9300_ipc_power_on,
-    .power_off = i9300_ipc_power_off,
+    .power_on = i9300_power_on,
+    .power_off = i9300_power_off,
     .power_data = NULL,
     .gprs_activate = NULL,
     .gprs_deactivate = NULL,
     .gprs_data = NULL,
-    .data_create = i9300_ipc_data_create,
-    .data_destroy = i9300_ipc_data_destroy,
+    .data_create = i9300_data_create,
+    .data_destroy = i9300_data_destroy,
 };
 
-struct ipc_gprs_specs i9300_ipc_gprs_specs = {
-    .gprs_get_iface = i9300_ipc_gprs_get_iface,
-    .gprs_get_capabilities = i9300_ipc_gprs_get_capabilities,
+struct ipc_client_gprs_specs i9300_gprs_specs = {
+    .gprs_get_iface = xmm6260_sec_modem_gprs_get_iface,
+    .gprs_get_capabilities = xmm6260_sec_modem_gprs_get_capabilities,
 };
 
-struct ipc_nv_data_specs i9300_ipc_nv_data_specs = {
+struct ipc_client_nv_data_specs i9300_nv_data_specs = {
     .nv_data_path = XMM6260_NV_DATA_PATH,
     .nv_data_md5_path = XMM6260_NV_DATA_MD5_PATH,
     .nv_data_backup_path = XMM6260_NV_DATA_BACKUP_PATH,
diff --git a/samsung-ipc/devices/i9300/i9300_ipc.h b/samsung-ipc/devices/i9300/i9300.h
index 901c28d..9e63c4b 100644
--- a/samsung-ipc/devices/i9300/i9300_ipc.h
+++ b/samsung-ipc/devices/i9300/i9300.h
@@ -1,7 +1,7 @@
 /*
  * This file is part of libsamsung-ipc.
  *
- * Copyright (C) 2013 Paul Kocialkowski <contact@paulk.fr>
+ * Copyright (C) 2013-2014 Paul Kocialkowski <contact@paulk.fr>
  *
  * libsamsung-ipc is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -17,8 +17,8 @@
  * along with libsamsung-ipc.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-#ifndef __I9300_IPC_H__
-#define __I9300_IPC_H__
+#ifndef __I9300_H__
+#define __I9300_H__
 
 #define I9300_MODEM_IMAGE_SIZE                               0x1000000
 #define I9300_PSI_OFFSET                                     0x1000
@@ -34,15 +34,15 @@
 
 #define I9300_MODEM_IMAGE_DEVICE             "/dev/block/mmcblk0p7"
 
-struct i9300_ipc_transport_data {
+struct i9300_transport_data {
     int fd;
 };
 
-extern struct ipc_ops i9300_ipc_fmt_ops;
-extern struct ipc_ops i9300_ipc_rfs_ops;
-extern struct ipc_handlers i9300_ipc_handlers;
-extern struct ipc_gprs_specs i9300_ipc_gprs_specs;
-extern struct ipc_nv_data_specs i9300_ipc_nv_data_specs;
+extern struct ipc_client_ops i9300_fmt_ops;
+extern struct ipc_client_ops i9300_rfs_ops;
+extern struct ipc_client_handlers i9300_handlers;
+extern struct ipc_client_gprs_specs i9300_gprs_specs;
+extern struct ipc_client_nv_data_specs i9300_nv_data_specs;
 
 #endif