#include "can-utils.hpp"
-
/********************************************************************************
*
* CanMessage method implementation
uint32_t can_message_t::get_id() const
{
- (id_ != 0) ? return id_ : return 0;
+ if (id_ != 0)
+ return id_;
+ return 0;
}
int can_message_t::get_format() const
{
- (format_ != CanMessageFormat::SIMPLE || format_ != CanMessageFormat::EXTENDED) return -1 : return format_;
+ if (format_ != CanMessageFormat::STANDARD || format_ != CanMessageFormat::EXTENDED)
+ return -1;
+ return format_;
}
-uint8_t can_message_t::get_data() const
+const uint8_t* can_message_t::get_data() const
{
- return data_;
+ return &data_;
}
-uint8_t can_message_t::get_lenght() const
+uint8_t can_message_t::get_length() const
{
- return lenght_;
+ return length_;
}
-void can_message_t::set_id(uint32_t &new_id)
+void can_message_t::set_id(const uint32_t new_id)
{
- switch(format):
- case CanMessageFormat::SIMPLE:
- id = new_id & CAN_SFF_MASK;
+ switch(format_)
+ {
+ case CanMessageFormat::STANDARD:
+ id_ = new_id & CAN_SFF_MASK;
+ break;
case CanMessageFormat::EXTENDED:
- id = new_id & CAN_EFF_MASK;
+ id_ = new_id & CAN_EFF_MASK;
+ break;
default:
ERROR(interface_, "ERROR: Can set id, not a compatible format or format not set prior to set id.");
+ break;
+ }
}
-void can_message_t::set_format(CanMessageFormat &new_format)
+void can_message_t::set_format(const CanMessageFormat new_format)
{
- if(new_format == CanMessageFormat::SIMPLE || new_format == CanMessageFormat::EXTENDED)
- format = new_format;
+ if(new_format == CanMessageFormat::STANDARD || new_format == CanMessageFormat::EXTENDED)
+ format_ = new_format;
else
ERROR(interface_, "ERROR: Can set format, wrong format chosen");
}
-void can_message_t::set_data(uint8_t &new_data)
+void can_message_t::set_data(const uint8_t new_data)
{
- ::memcpy(data_, new_data, new_data.size());
- lenght_ = new_data(size);
+ ::memcpy(&data_, &new_data, sizeof(new_data));
+ length_ = sizeof(new_data);
}
/*
- * This is the preferred way to initialize a CanMessage object
+ * @brief This is the preferred way to initialize a CanMessage object
* from a read canfd_frame message.
*
- * params: canfd_frame pointer
+ * @param: canfd_frame pointer
*/
-void can_message_t::convert_from_canfd_frame(canfd_frame &frame)
+void can_message_t::convert_from_canfd_frame(const canfd_frame& frame)
{
- lenght_ = (frame.len > CAN_MAX_DLEN) ? CAN_MAX_DLEN : frame.len;
- lenght_ = (frame.len > CANFD_MAX_DLEN) ? CANFD_MAX_DLEN : frame.len;
-
- switch (frame.can_id):
- case (frame.can_id & CAN_ERR_FLAG):
- id_ = frame.can_id & (CAN_ERR_MASK|CAN_ERR_FLAG);
- break;
- case (frame.can_id & CAN_EFF_FLAG):
- id_ = frame.can_id & CAN_EFF_MASK;
- format_ = CanMessageFormat::EXTENDED;
- break;
- default:
- format_ = CanMessageFormat::STANDARD;
- id_ = frame.can_id & CAN_SFF_MASK;
- break;
+ length_ = (frame.len > CAN_MAX_DLEN) ? (uint8_t)CAN_MAX_DLEN : frame.len;
+ length_ = (frame.len > CANFD_MAX_DLEN) ? (uint8_t)CANFD_MAX_DLEN : frame.len;
- if (sizeof(frame.data) <= data_.size())
+ if (frame.can_id & CAN_ERR_FLAG)
+ id_ = frame.can_id & (CAN_ERR_MASK|CAN_ERR_FLAG);
+ else if (frame.can_id & CAN_EFF_FLAG)
{
- ::memcpy(data_, canfd_frame.data, lenght_);
- return 0;
- } else if (sizeof(frame.data) >= CAN_MAX_DLEN)
+ id_ = frame.can_id & CAN_EFF_MASK;
+ format_ = CanMessageFormat::EXTENDED;
+ }
+ else
+ {
+ id_ = frame.can_id & CAN_SFF_MASK;
+ format_ = CanMessageFormat::STANDARD;
+ }
+
+ if (sizeof(frame.data) <= sizeof(data_))
+ ::memcpy(&data_, frame.data, length_);
+ else if (sizeof(frame.data) >= CAN_MAX_DLEN)
ERROR(interface_, "can_message_t: canfd_frame data too long to be stored into CanMessage object");
}
canfd_frame frame;
frame.can_id = get_id();
- frame.len = get_lenght();
- ::memcpy(frame.data, get_data(), lenght_);
+ frame.len = get_length();
+ ::memcpy(frame.data, get_data(), length_);
return frame;
}
{
::close(can_socket_);
can_socket_ = -1;
+ return can_socket_;
}
canfd_frame can_bus_dev_t::read(const struct afb_binding_interface* interface)
* @brief start reading threads and set flag is_running_
*
*/
-void can_bus_dev_t::start_reading()
+void can_bus_dev_t::start_reading(can_bus_t& can_bus)
{
- th_reading_ = std::thread(can_reader, this);
+ th_reading_ = std::thread(can_reader, std::ref(*this), std::ref(can_bus));
is_running_ = true;
}
return is_running_;
}
-/**
- * @brief: Get a can_message_t from can_message_q and return it
- * then point to the next can_message_t in queue.
- *
- * @return the next queue element or NULL if queue is empty.
- */
-can_message_t can_bus_dev_t::next_can_message()
-{
- if(! can_message_q_.empty())
- {
- can_message_t can_msg = can_message_q_.front();
- can_message_q_.pop();
- return can_msg;
- }
-
- has_can_message_ = false;
-}
-
-/**
- * @brief Append a new element to the can message queue and set
- * has_can_message_ boolean to true
- *
- * @params[const can_message_t& can_msg] the can_message_t to append
- *
- */
-void can_bus_dev_t::push_new_can_message(const can_message_t& can_msg)
-{
- can_message_q_.push(can_msg);
-}
-
-/**
- * @brief Flag that let you know when can message queue is exhausted
- *
- * @return[bool] has_can_message_ bool
- */
-bool can_bus_dev_t::has_can_message() const
-{
- return has_can_message_;
-}
-
/**
* @brief Send a can message from a can_message_t object.
*
*/
int can_bus_dev_t::send_can_message(can_message_t& can_msg, const struct afb_binding_interface* interface)
{
- size_t nbytes;
+ ssize_t nbytes;
canfd_frame f;
f = can_msg.convert_to_canfd_frame();
{
nbytes = ::sendto(can_socket_, &f, sizeof(struct canfd_frame), 0,
(struct sockaddr*)&txAddress_, sizeof(txAddress_));
-
if (nbytes == -1)
{
ERROR(interface, "send_can_message: Sending CAN frame failed.");
*
*********************************************************************************/
-can_bus_t::can_bus_t(const afb_binding_interface *itf, int& conf_file)
- : interface_{itf}, conf_file_{conf_file}
+can_bus_t::can_bus_t(const struct afb_binding_interface *interface, int& conf_file)
+ : interface_{interface}, conf_file_{conf_file}
{
}
/**
* @brief start threads relative to the can bus: decoding and pushing
* as the reading is handled by can_bus_dev_t object
- *
*/
void can_bus_t::start_threads()
{
- th_decoding_ = std::thread(can_decoder, this);
- th_pushing_ = std::thread(can_event_push, this);
+ th_decoding_ = std::thread(can_decode_message, std::ref(*this));
+ th_pushing_ = std::thread(can_event_push, std::ref(*this));
}
/**
{
can_bus_dev_t can_bus_device_handler(device);
can_bus_device_handler.open(interface_);
- can_bus_device_handler.start_reading();
+ can_bus_device_handler.start_reading(std::ref(*this));
i++;
}
{
std::vector<std::string> ret;
json_object *jo, *canbus;
- int n, i, ok;
+ int n, i;
FILE *fd = fdopen(conf_file_, "r");
if (fd)
else
{
n = json_object_array_length(canbus);
- ok = 0;
for (i = 0 ; i < n ; i++)
- ret.push_back(json_object_get_string(json_object_array_get_idx(canbus, i)));
+ ret.push_back(json_object_get_string(json_object_array_get_idx(canbus, i)));
}
return ret;
}
return ret;
}
+/**
+ * @brief: Get a can_message_t from can_message_q and return it
+ * then point to the next can_message_t in queue.
+ *
+ * @return the next queue element or NULL if queue is empty.
+ */
+can_message_t can_bus_t::next_can_message()
+{
+ can_message_t can_msg(interface_);
+
+ if(!can_message_q_.empty())
+ {
+ can_msg = can_message_q_.front();
+ can_message_q_.pop();
+ DEBUG(interface_, "next_can_message: Here is the next can message : id %d, length %d", can_msg.get_id(), can_msg.get_length());
+ return can_msg;
+ }
+
+ NOTICE(interface_, "next_can_message: End of can message queue");
+ has_can_message_ = false;
+ return can_msg;
+}
+
+/**
+ * @brief Append a new element to the can message queue and set
+ * has_can_message_ boolean to true
+ *
+ * @params[const can_message_t& can_msg] the can_message_t to append
+ *
+ */
+void can_bus_t::push_new_can_message(const can_message_t& can_msg)
+{
+ can_message_q_.push(can_msg);
+}
+
+/**
+ * @brief Flag that let you know when can message queue is exhausted
+ *
+ * @return[bool] has_can_message_ bool
+ */
+bool can_bus_t::has_can_message() const
+{
+ return has_can_message_;
+}
+
/**
* @brief: Get a VehicleMessage from vehicle_message_q and return it
* then point to the next VehicleMessage in queue.
*/
openxc_VehicleMessage can_bus_t::next_vehicle_message()
{
+ openxc_VehicleMessage v_msg;
+
if(! vehicle_message_q_.empty())
{
- openxc_VehicleMessage v_msg = vehicle_message_q_.front();
+ v_msg = vehicle_message_q_.front();
vehicle_message_q_.pop();
+ DEBUG(interface_, "next_vehicle_message: next vehicle message poped");
return v_msg;
}
-
+
+ NOTICE(interface_, "next_vehicle_message: End of vehicle message queue");
has_vehicle_message_ = false;
+ return v_msg;
}
/**
Code Review / apps / agl-service-can-low-level.git / blobdiff