diff options
| author |   Romain Forlot <romain.forlot@iot.bzh> | 2019-12-10 16:53:22 +0100 |
|---|---|---|
| committer | Romain Forlot <romain.forlot@iot.bzh> | 2020-01-09 16:25:36 +0100 |
| commit | 542e3d732ec17e4c49d1059a9262f6535a7f3e62 (patch) | |
| tree | 7e1ba564433a4021eec9e88456868325eb43d130 /low-can-binding/can | |
| parent | 0e786385e547a247f96fce164d38712d0f59298e (diff) | |
message: Rework message flags
Adding bit_position and frame manipulation flags
Change-Id: Ibd727a54a33a78b49f2c319b4b70dcda21ea87cb
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
Diffstat (limited to 'low-can-binding/can')
| -rw-r--r-- | low-can-binding/can/can-decoder.cpp | 9 | ||||
| -rw-r--r-- | low-can-binding/can/message/message.hpp | 21 |
2 files changed, 17 insertions, 13 deletions
diff --git a/low-can-binding/can/can-decoder.cpp b/low-can-binding/can/can-decoder.cpp index ae10cd98..488fdfb8 100644 --- a/low-can-binding/can/can-decoder.cpp +++ b/low-can-binding/can/can-decoder.cpp @@ -106,13 +106,16 @@ float decoder_t::parse_signal_bitfield(signal_t& signal, std::shared_ptr<message uint8_t new_start_bit = 0; uint8_t new_end_bit = 0; - if(signal.get_message()->frame_layout_is_bigendian()) - { + if(signal.get_message()->get_flags() & CONTINENTAL_BIT_POSITION) + bit_position = converter_t::continental_bit_position_mess(message->get_length(), + signal.get_bit_position(), + bit_size); + if(signal.get_message()->get_flags() & BIT_POSITION_REVERSED) bit_position = converter_t::bit_position_swap(message->get_length(), signal.get_bit_position(), bit_size); + if(signal.get_message()->get_flags() & FRAME_LAYOUT_IS_BIGENDIAN) message->frame_swap(); - } data = message->get_data_vector(); converter_t::signal_to_bits_bytes(bit_position, bit_size, new_start_byte, new_end_byte, new_start_bit, new_end_bit); diff --git a/low-can-binding/can/message/message.hpp b/low-can-binding/can/message/message.hpp index cc478c33..2c27f1f3 100644 --- a/low-can-binding/can/message/message.hpp +++ b/low-can-binding/can/message/message.hpp @@ -35,16 +35,17 @@ /** * FLAGS */ - -#define INVALID_FLAG 0x0001 -#define CAN_PROTOCOL 0x0002 -#define J1939_PROTOCOL 0x0004 -#define J1939_ADDR_CLAIM_PROTOCOL 0x0008 -#define ISOTP_PROTOCOL 0x0010 -#define ISOTP_SEND 0x0020 -#define ISOTP_RECEIVE 0x0040 -#define CAN_PROTOCOL_WITH_FD_FRAME 0x0080 -#define FRAME_LAYOUT_IS_BIGENDIAN 0x0100 // LE stand for Little Endian +#define CAN_PROTOCOL 0x0001 +#define CAN_PROTOCOL_WITH_FD_FRAME 0x0002 +#define J1939_ADDR_CLAIM_PROTOCOL 0x0004 +#define J1939_PROTOCOL 0x0008 +#define ISOTP_PROTOCOL 0x0010 +#define ISOTP_SEND 0x0020 +#define ISOTP_RECEIVE 0x0040 +#define FRAME_LAYOUT_IS_BIGENDIAN 0x0080 +#define BIT_POSITION_REVERSED 0x0100 +#define CONTINENTAL_BIT_POSITION 0x0200 +#define INVALID_FLAG 0x8000 /// @class message_t /// |