From d6d322103eaf1d2d5a1a94a0720a01a692f6a134 Mon Sep 17 00:00:00 2001 From: Romain Forlot Date: Thu, 16 Feb 2017 16:09:08 +0000 Subject: [PATCH] Changing CanBus_c to can_bus_t class name Change-Id: Ibf277e36187b78853718f76552ff730d0474a3e6 Signed-off-by: Romain Forlot --- src/can-signals.cpp | 2 +- src/can_event_push.cpp | 2 +- src/can_reader.cpp | 39 ++++----------------------------------- src/low-can-binding.cpp | 13 +++++++++++-- src/obd2.cpp | 15 +++++++++++---- src/obd2.h | 34 ++++++++++++++++++++++++++++------ 6 files changed, 56 insertions(+), 49 deletions(-) diff --git a/src/can-signals.cpp b/src/can-signals.cpp index 8e3a1e69..b20c540d 100644 --- a/src/can-signals.cpp +++ b/src/can-signals.cpp @@ -29,7 +29,7 @@ std::map ::iterator subscribed_signals_i; /* Find one or many signals based on its name or id * passed through openxc_DynamicField. */ -void find_signals(openxc_DynamicField *key, std:vector *signals) +void find_can_signals(openxc_DynamicField *key, std:vector *signals) { int n_signals, i; diff --git a/src/can_event_push.cpp b/src/can_event_push.cpp index db14a448..ccfe6b19 100644 --- a/src/can_event_push.cpp +++ b/src/can_event_push.cpp @@ -52,7 +52,7 @@ void jsonify_DynamicField(openxc_DynamicField *field, json_object *value) else if(field->has_string_value) json_object_object_add(value, "value", json_object_new_string(field->string_value)); -return value; + return value; } /* Extract the simple message value from an openxc_VehicleMessage diff --git a/src/can_reader.cpp b/src/can_reader.cpp index 6403cfa7..80889372 100644 --- a/src/can_reader.cpp +++ b/src/can_reader.cpp @@ -23,44 +23,13 @@ #include "can-utils.h" -void can_reader(CanBus_c *can_bus)) +void can_reader(CanBus_t &can_bus) { - ssize_t nbytes; - int maxdlen; CanMessage_c can_message; - canfd_frame canfd_frame; - /* Test that socket is really opened */ - if ( can_bus->socket < 0) + while(can_bus.is_running()) { - ERROR(interface, "read_can: Socket unavailable"); - return -1; - } - - while(true) - { - nbytes = read(can_bus->socket, &canfd_frame, CANFD_MTU); - - switch(nbytes) - { - case CANFD_MTU: - DEBUG(interface, "read_can: Got an CAN FD frame with length %d", canfd_frame.len); - maxdlen = CANFD_MAX_DLEN; - break; - case CAN_MTU: - DEBUG(interface, "read_can: Got a legacy CAN frame with length %d", canfd_frame.len); - maxdlen = CAN_MAX_DLEN; - break; - default: - if (errno == ENETDOWN) - ERROR(interface, "read_can: %s interface down", device); - - ERROR(interface, "read_can: Error reading CAN bus"); - return -2; - } - - can_message.convert_from_canfd_frame(canfd_frame); - - can_message_q.push(can_message); + can_message.convert_from_canfd_frame(canbus.read()); + can_bus.insert_new_can_message(can_message); } } \ No newline at end of file diff --git a/src/low-can-binding.cpp b/src/low-can-binding.cpp index af86786d..18f5a75e 100644 --- a/src/low-can-binding.cpp +++ b/src/low-can-binding.cpp @@ -40,7 +40,6 @@ #include #include -#include "ll-can-binding.h" #include "obd2.h" /* @@ -177,7 +176,10 @@ static int subscribe_unsubscribe_name(struct afb_req request, int subscribe, con if (0 == strcmp(name, "*")) return subscribe_unsubscribe_all(request, subscribe); - find_signals(name, sig); + if(obd2_handler_c.is_obd2_signal(name)) + + else + find_can_signals(name, sig); if (sig == NULL) { return 0; } @@ -246,6 +248,13 @@ const struct afb_binding *afbBindingV1Register (const struct afb_binding_interfa return &binding_desc; } +/** + * @brief Initialize the binding. + * + * @param[in] service Structure which represent the Application Framework Binder. + * + * @return Exit code, zero if success. + */ int afbBindingV1ServiceInit(struct afb_service service) { std::ifstream fd_conf; diff --git a/src/obd2.cpp b/src/obd2.cpp index 7db5d97d..ae2db98a 100644 --- a/src/obd2.cpp +++ b/src/obd2.cpp @@ -28,7 +28,7 @@ void shims_timer() /* * Will scan for supported Obd2 pids */ -Obd2Handler::Obd2Handler(afb_binding_interface *itf, CanBus_c cb) +obd2_handler_c::obd2_handler_c(afb_binding_interface *itf, CanBus_c cb) { CanBus_c can_bus = cb; DiagnosticShims shims = diagnostic_init_shims(shims_logger, can_bus.send_can_message, NULL); @@ -41,19 +41,26 @@ Obd2Handler::Obd2Handler(afb_binding_interface *itf, CanBus_c cb) } } -Obd2Handler::add_request(int pid) +void obd2_handler_c::add_request(int pid) { DiagnosticRequest request = { arbitration_id: OBD2_FUNCTIONAL_BROADCAST_ID, mode: 0x1, has_pid: true, pid: pid}; } -Obd2Handler::is_obd2_request(DiagnosticRequest* request) +bool obd2_handler_c::is_obd2_request(DiagnosticRequest* request) { return request->mode == 0x1 && request->has_pid && request->pid < 0xff; } -Obd2Handler::decode_obd2_response(DiagnosticResponse* responce) +bool obd2_handler_c::is_obd2_signal(const char *name) +{ + if(fnmatch("obd2.*", name, NULL) == 0) + return true; + return false; +} + +bool obd2_handler_c::decode_obd2_response(DiagnosticResponse* responce) { return diagnostic_decode_obd2_pid(response); } diff --git a/src/obd2.h b/src/obd2.h index 61cb70c3..cd362c79 100644 --- a/src/obd2.h +++ b/src/obd2.h @@ -50,7 +50,6 @@ const char *UNIT_NAMES[10] = { "NM" }; - /* * A representation of an OBD-II PID. * @@ -63,7 +62,6 @@ const char *UNIT_NAMES[10] = { * when automatic, recurring OBD-II requests are enabled. * supported - is it supported by the vehicle. Initialized after scan * event - application framework event handler. - * */ typedef struct _Obd2Pid { uint8_t pid; @@ -116,7 +114,7 @@ float handleObd2Pid(const DiagnosticResponse* response, float parsedPayload); * Object to handle obd2 session with pre-scan of supported pid * then request them regularly */ -class Obd2Handler_c { +class obd2_handler_c { private: public: @@ -145,7 +143,31 @@ class Obd2Handler_c { { pid: 0x63, name: "obd2.engine.torque", min: 0, max: 65535, unit: NM, frequency: 1, supported: false }, }; - Obd2Handler_c(); - bool isObd2Request(request); -} + /* Public: Check if a request is an OBD-II PID request. + * + * Returns true if the request is a mode 1 request and it has a 1 byte PID. + */ + void find_obd2_pid(const char *name, std::Vector *pids); + + /* Public: Check if a request is an OBD-II PID request. + * + * Returns true if the request is a mode 1 request and it has a 1 byte PID. + */ + bool is_obd2_request(DiagnosticRequest *request); + + /* + * Public: Check if requested signal name is an obd2 pid + * + * Returns true if name began with ob2.* else false. + */ + bool is_obd2_signal(const char *name); + + /* + * Public: pass response to UDS-C library function + * diagnostic_decode_obd2_pid() + * + * Return: float number representing the requested value. + */ + bool decode_obd2_response(DiagnosticResponse* responce); +} \ No newline at end of file -- 2.16.6