diff options
author | 2017-03-13 15:03:00 +0100 | |
---|---|---|
committer | 2017-03-16 17:15:55 +0100 | |
commit | 542518e3f4390fcb0d8fb17bcbb14c7eb4335a61 (patch) | |
tree | 340eda25b52c5f6e4b47dae3aace322c57359e55 /src/diagnostic/diagnostic-message.cpp | |
parent | f0d7a6523955ee94a32ec4b62e2a207b23f62316 (diff) |
Move member from obd2_signals_t class to
active_diagnostic_request_t (adr)class. It's more logical
to have them here as call about incoming diagnostic CAN
messages are made against adr class not obd2_signal_t that
it's a subset of diagnostic messages.
It's made to handle obd2 requests as well as normal UDS
diagnostic request for constructor proprietary diagnostic
signals.
Change-Id: I96c714640d6ecfa25dfc9942974057c1797eb0e1
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
Diffstat (limited to 'src/diagnostic/diagnostic-message.cpp')
-rw-r--r-- | src/diagnostic/diagnostic-message.cpp | 72 |
1 files changed, 2 insertions, 70 deletions
diff --git a/src/diagnostic/diagnostic-message.cpp b/src/diagnostic/diagnostic-message.cpp index 8b2273e..5a8f9e5 100644 --- a/src/diagnostic/diagnostic-message.cpp +++ b/src/diagnostic/diagnostic-message.cpp @@ -33,8 +33,6 @@ const char *UNIT_NAMES[10] = { "NM" }; -std::string obd2_signal_t::prefix_ = "diagnostic_messages."; - obd2_signal_t::obd2_signal_t(uint8_t pid, const char* generic_name, const int min, const int max, enum UNIT unit, int frequency, bool supported) : pid_{pid}, generic_name_{generic_name}, min_{min}, max_{max}, unit_{unit}, frequency_{frequency}, supported_{supported} { @@ -52,12 +50,7 @@ const std::string& obd2_signal_t::get_generic_name() const const std::string obd2_signal_t::get_name() const { - return prefix_ + "." + generic_name_; -} - -const std::string& obd2_signal_t::get_prefix() -{ - return prefix_; + return active_diagnostic_request_t::get_prefix() + "." + generic_name_; } int obd2_signal_t::get_frequency() const @@ -65,11 +58,6 @@ int obd2_signal_t::get_frequency() const return frequency_; } -void obd2_signal_t::set_prefix(const std::string& val) -{ - prefix_ = val; -} - /** * @brief Build a DiagnosticRequest struct to be passed * to diagnostic manager instance. @@ -87,73 +75,17 @@ const DiagnosticRequest obd2_signal_t::build_diagnostic_request() /*DiagnosticRequestType: */DiagnosticRequestType::DIAGNOSTIC_REQUEST_TYPE_PID }; } -bool obd2_signal_t::is_obd2_response(can_message_t can_message) -{ - /* - if(can_message.get_id() >= 0x7E8 && can_message.get_id() <= 0x7EF) - { - openxc_VehicleMessage message = {0}; - message.has_type = true; - message.type = openxc_VehicleMessage_Type_DIAGNOSTIC; - message.has_diagnostic_response = true; - message.diagnostic_response = {0}; - message.diagnostic_response.has_bus = true; - message.diagnostic_response.bus = bus->address; - message.diagnostic_response.has_message_id = true; - //7DF should respond with a random message id between 7e8 and 7ef - //7E0 through 7E7 should respond with a id that is 8 higher (7E0->7E8) - if(can_message.get_id() == 0x7DF) - { - message.diagnostic_response.message_id = rand()%(0x7EF-0x7E8 + 1) + 0x7E8; - } - else if(commandRequest->message_id >= 0x7E0 && commandRequest->message_id <= 0x7E7) - { - message.diagnostic_response.message_id = commandRequest->message_id + 8; - } - message.diagnostic_response.has_mode = true; - message.diagnostic_response.mode = commandRequest->mode; - if(commandRequest->has_pid) - { - message.diagnostic_response.has_pid = true; - message.diagnostic_response.pid = commandRequest->pid; - } - message.diagnostic_response.has_value = true; - message.diagnostic_response.value = rand() % 100; - pipeline::publish(&message, &getConfiguration()->pipeline); - } - else //If it's outside the range, the command_request will return false - { - debug("Sent message ID is outside the valid range for emulator (7DF to 7E7)"); - status=false; - } - return false; - */ - return false; -} - /** * @brief Check if a request is an OBD-II PID request. * * @return true if the request is a mode 1 request and it has a 1 byte PID. */ -bool obd2_signal_t::is_obd2_request(DiagnosticRequest* request) +bool obd2_signal_t::is_obd2_request(const DiagnosticRequest* request) { return request->mode == 0x1 && request->has_pid && request->pid < 0xff; } /** -* @brief Check if requested signal name is an obd2 pid -* -* @return true if name began with obd2 else false. -*/ -bool obd2_signal_t::is_obd2_signal(const std::string& name) -{ - if(name.find_first_of(prefix_.c_str(), 0, prefix_.size())) - return true; - return false; -} - -/** * @brief Decode the payload of an OBD-II PID. * * This function matches the type signature for a DiagnosticResponseDecoder, so |