From 6728ba8c4aec286f57cf3b0ce01b966dae544bbc Mon Sep 17 00:00:00 2001 From: Tobias Jahnke Date: Wed, 2 Aug 2017 09:04:07 +0200 Subject: cleanup of API i2cwrite --- ucs2-afb/ucs_binding.c | 99 ++++++++++++++++++++++---------------------------- 1 file changed, 43 insertions(+), 56 deletions(-) (limited to 'ucs2-afb/ucs_binding.c') diff --git a/ucs2-afb/ucs_binding.c b/ucs2-afb/ucs_binding.c index dffacd3..fe6e033 100644 --- a/ucs2-afb/ucs_binding.c +++ b/ucs2-afb/ucs_binding.c @@ -23,6 +23,7 @@ #define BUFFER_FRAME_COUNT 10 /* max frames in buffer */ #define WAIT_TIMER_US 1000000 /* default waiting timer 1s */ +#define I2C_MAX_DATA_SZ 32 /* max. number of bytes to be written to i2c */ #include #include @@ -39,8 +40,6 @@ #include "ucs_binding.h" #include "ucs_interface.h" - - #define MAX_FILENAME_LEN (100) #define RX_BUFFER (64) @@ -65,8 +64,6 @@ typedef struct { static ucsContextT *ucsContextS; -STATIC void ucs2_write_i2c_response(void *result_ptr, void *request_ptr); - PUBLIC void UcsXml_CB_OnError(const char format[], uint16_t vargsCnt, ...) { /*AFB_DEBUG (afbIface, format, args); */ va_list args; @@ -539,21 +536,35 @@ PUBLIC void ucs2_monitor (struct afb_req request) { afb_req_success(request,NULL,"UNICENS-to_be_done"); } -#define MUTE_VALUE 0x03FFU -#define MUTE_VALUE_HB 0x03U -#define MUTE_VALUE_LB 0xFFU - -#define CONTROL_MASTER 0x07U -#define CONTROL_CH_1 0x08U -#define CONTROL_CH_2 0x09U - -#define UCSB_I2C_MAX_PAYLOAD 32 +STATIC void ucs2_writei2c_CB(void *result_ptr, void *request_ptr) { + + if (request_ptr){ + afb_req *req = (afb_req *)request_ptr; + Ucs_I2c_ResultCode_t *res = (Ucs_I2c_ResultCode_t *)result_ptr; + + if (!res) { + afb_req_fail(*req, "processing","busy or lost initialization"); + } + else if (*res != UCS_I2C_RES_SUCCESS){ + afb_req_fail_f(*req, "error-result", "result code: %d", *res); + } + else { + afb_req_success(*req, NULL, "success"); + } + + afb_req_unref(*req); + free(request_ptr); + } + else { + AFB_NOTICE("write_i2c: ambiguous response data"); + } +} -PUBLIC void ucs2_write_i2c (struct afb_req request) { +PUBLIC void ucs2_writei2c(struct afb_req request) { struct json_object *j_obj; - static uint8_t tx_payload[UCSB_I2C_MAX_PAYLOAD]; - uint8_t tx_payload_sz = 0; + static uint8_t i2c_data[I2C_MAX_DATA_SZ]; + uint8_t i2c_data_sz = 0; uint16_t node_addr = 0; struct afb_req *async_req_ptr = NULL; @@ -569,7 +580,7 @@ PUBLIC void ucs2_write_i2c (struct afb_req request) { goto OnErrorExit; }; - node_addr = (uint16_t)json_object_get_int(json_object_object_get(j_obj, "node_address")); + node_addr = (uint16_t)json_object_get_int(json_object_object_get(j_obj, "node")); AFB_NOTICE("node_address: 0x%02X", node_addr); if (node_addr == 0) { @@ -577,14 +588,14 @@ PUBLIC void ucs2_write_i2c (struct afb_req request) { goto OnErrorExit; } - if (json_object_get_type(json_object_object_get(j_obj, "i2c_data"))==json_type_array) { - int size = json_object_array_length(json_object_object_get(j_obj, "i2c_data")); - if ((size > 0) && (size <= UCSB_I2C_MAX_PAYLOAD)) { + if (json_object_get_type(json_object_object_get(j_obj, "data"))==json_type_array) { + int size = json_object_array_length(json_object_object_get(j_obj, "data")); + if ((size > 0) && (size <= I2C_MAX_DATA_SZ)) { int32_t i; int32_t val; struct json_object *j_elem; - struct json_object *j_arr = json_object_object_get(j_obj, "i2c_data"); + struct json_object *j_arr = json_object_object_get(j_obj, "data"); for (i = 0; i < size; i++) { @@ -595,15 +606,15 @@ PUBLIC void ucs2_write_i2c (struct afb_req request) { i = 0; break; } - tx_payload[i] = (uint8_t)json_object_get_int(j_elem); + i2c_data[i] = (uint8_t)json_object_get_int(j_elem); } - tx_payload_sz = (uint8_t)i; + i2c_data_sz = (uint8_t)i; } } - if (tx_payload_sz == 0) { - AFB_NOTICE("i2c_data: invalid or not found"); + if (i2c_data_sz == 0) { + AFB_NOTICE("data: invalid or not found"); afb_req_fail_f(request, "query-params","params wrong or missing"); goto OnErrorExit; } @@ -611,22 +622,22 @@ PUBLIC void ucs2_write_i2c (struct afb_req request) { async_req_ptr = malloc(sizeof(afb_req)); *async_req_ptr = request; - if (UCSI_I2CWrite( &ucsContextS->ucsiData, /*UCSI_Data_t *pPriv*/ - node_addr, /*uint16_t targetAddress*/ - false, /*bool isBurst*/ + if (UCSI_I2CWrite( &ucsContextS->ucsiData, /* UCSI_Data_t *pPriv*/ + node_addr, /* uint16_t targetAddress*/ + false, /* bool isBurst*/ 0u, /* block count */ 0x2Au, /* i2c slave address */ 0x03E8u, /* timeout 1000 milliseconds */ - tx_payload_sz, /* uint8_t dataLen */ - &tx_payload[0], /* uint8_t *pData */ - &ucs2_write_i2c_response, /* callback*/ + i2c_data_sz, /* uint8_t dataLen */ + &i2c_data[0], /* uint8_t *pData */ + &ucs2_writei2c_CB, /* callback*/ (void*)async_req_ptr /* callback argument */ )) { /* asynchronous command is running */ afb_req_addref(request); } else { - AFB_NOTICE("i2c_data: scheduling command failed"); + AFB_NOTICE("i2c write: scheduling command failed"); afb_req_fail_f(request, "query-command-queue","command queue overload"); free(async_req_ptr); async_req_ptr = NULL; @@ -636,27 +647,3 @@ PUBLIC void ucs2_write_i2c (struct afb_req request) { OnErrorExit: return; } - -STATIC void ucs2_write_i2c_response(void *result_ptr, void *request_ptr) { - - if (request_ptr){ - afb_req *req = (afb_req *)request_ptr; - Ucs_I2c_ResultCode_t *res = (Ucs_I2c_ResultCode_t *)result_ptr; - - if (!res) { - afb_req_fail(*req, "processing","busy or lost initialization"); - } - else if (*res != UCS_I2C_RES_SUCCESS){ - afb_req_fail_f(*req, "error-result", "result code: %d", *res); - } - else { - afb_req_success(*req, NULL, "success"); - } - - afb_req_unref(*req); - free(request_ptr); - } - else { - AFB_NOTICE("write_i2c: ambiguous response data"); - } -} -- cgit 1.2.3-korg