diff options
author | Paul Kocialkowski <contact@paulk.fr> | 2014-02-15 20:16:19 +0100 |
---|---|---|
committer | Paul Kocialkowski <contact@paulk.fr> | 2014-02-15 20:58:20 +0100 |
commit | 005645221f3d4dd432970f8f28d9092d66d61213 (patch) | |
tree | 6ff93168ae04da425313ada33631fee507cfd7da /samsung-ipc/devices/i9300 | |
parent | ec82c3a5350b8747e1c0a7a0393ab0374dd9fbd5 (diff) | |
download | hardware_replicant_libsamsung-ipc-005645221f3d4dd432970f8f28d9092d66d61213.tar.gz hardware_replicant_libsamsung-ipc-005645221f3d4dd432970f8f28d9092d66d61213.tar.bz2 hardware_replicant_libsamsung-ipc-005645221f3d4dd432970f8f28d9092d66d61213.zip |
ipc: Consistent coding style and major cleanup
Signed-off-by: Paul Kocialkowski <contact@paulk.fr>
Diffstat (limited to 'samsung-ipc/devices/i9300')
-rw-r--r-- | samsung-ipc/devices/i9300/i9300.c (renamed from samsung-ipc/devices/i9300/i9300_ipc.c) | 140 | ||||
-rw-r--r-- | samsung-ipc/devices/i9300/i9300.h (renamed from samsung-ipc/devices/i9300/i9300_ipc.h) | 18 |
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 |