From 0ff6413b6dc42849f87ce8caa17a91f44ede2f93 Mon Sep 17 00:00:00 2001 From: Romain Forlot Date: Wed, 4 Dec 2019 17:26:58 +0100 Subject: decoder: rework how to swap frame layout. This also change the bit_position to retrieve the bit word starting from the left or the right of the frame depending on the endianness of the frame layout. Change-Id: I28658e9d46bd35d8ecabeece317331832229384a Signed-off-by: Romain Forlot --- low-can-binding/utils/converter.cpp | 20 ++++++++++++-------- low-can-binding/utils/converter.hpp | 3 ++- 2 files changed, 14 insertions(+), 9 deletions(-) (limited to 'low-can-binding/utils') diff --git a/low-can-binding/utils/converter.cpp b/low-can-binding/utils/converter.cpp index 0a09125f..6392fbdf 100644 --- a/low-can-binding/utils/converter.cpp +++ b/low-can-binding/utils/converter.cpp @@ -65,20 +65,24 @@ void converter_t::signal_to_bits_bytes(unsigned int bit_position, unsigned int b * @param bit_size Size of the data. * @return uint32_t New bit position. */ -uint32_t converter_t::bit_position_swap(uint32_t bit_position,uint32_t bit_size) +uint32_t converter_t::bit_position_swap(unsigned int msg_length, unsigned int bit_position, unsigned int bit_size) { - uint32_t start_byte_position = (uint32_t)(bit_position/8); - uint32_t bit_size_rest = bit_size; + return msg_length - bit_position - bit_size; + /* + unsigned int start_byte_position = (unsigned int)(bit_position/8); + unsigned int bit_size_rest = bit_size; - if((int)(bit_size-(8 + start_byte_position*8-bit_position%8))>0) + if((int)(bit_size-(8 + start_byte_position * 8 - bit_position % 8)) > 0) { AFB_ERROR("Error: bit_position and bit_size getting out of range"); return bit_position; } - if(bit_size<=8 && ((bit_position+bit_size)%8==bit_size || (bit_position+bit_size)%8==0)) + if(bit_size <= 8 && + ((bit_position+bit_size) % 8 == bit_size || + (bit_position+bit_size)%8==0)) { - return (uint32_t)(start_byte_position*8 + (8-bit_size)); + return (unsigned int)(start_byte_position*8 + (8-bit_size)); } else { @@ -88,7 +92,7 @@ uint32_t converter_t::bit_position_swap(uint32_t bit_position,uint32_t bit_size) start_byte_position--; bit_position = start_byte_position*8; } while (bit_size_rest>8); - return (uint32_t)(start_byte_position*8 + (8-bit_size_rest)); + return (unsigned int)(start_byte_position*8 + (8-bit_size_rest)); } - + */ } diff --git a/low-can-binding/utils/converter.hpp b/low-can-binding/utils/converter.hpp index 32c898f1..23f8ae77 100644 --- a/low-can-binding/utils/converter.hpp +++ b/low-can-binding/utils/converter.hpp @@ -29,6 +29,7 @@ class converter_t int &new_end_byte, uint8_t &new_start_bit, uint8_t &new_end_bit); - static uint32_t bit_position_swap(unsigned int bit_position, + static uint32_t bit_position_swap(unsigned int msg_length, + unsigned int bit_position, unsigned int bit_size); }; -- cgit 1.2.3-korg