Wrong start bit for auth info signal

[Patryk-97]

Since auth info/freshness signal is encoded in Motorola byte order format then start bit should indicate to most significant bit. But in the code start bit points to least significant bit:

canmatrix/src/canmatrix/formats/arxml.py

Line 1643 in 9a70a85

start_bit = int(ipdu_length, 0)*8 + int(freshness_tx_length, 0) + int(auth_tx_length, 0) - 8,

canmatrix/src/canmatrix/formats/arxml.py

Line 1634 in 9a70a85

start_bit = int(ipdu_length, 0)*8 + int(freshness_tx_length, 0) - 8,

@xRowe are you sure it's correct?

[xRowe]

Thanks for your feedback

I validate on CANoe, it should be OK now
Please Try xRowe:Fix_#833

[Patryk-97]

Hello @xRowe & @ebroecker. Thanks for answer, but still it's not what I meant. Freshness and auth info signals are encoded in Motorola byte order format. So start bit of those signals indicates to most significant bit. So start bit of freshness signal should be:
start_bit = int(ipdu_length, 0)*8 + 7 (assuming it exists)

And start bit of auth info signal should be:
start_bit = int(ipdu_length, 0)*8 + int(freshness_tx_length, 0) + 7

Reference to AUTOSAR system template: https://www.autosar.org/fileadmin/standards/R22-11/CP/AUTOSAR_TPS_SystemTemplate.pdf

[xRowe]

For the start-bit of Motorola format in canmatrix,
I think you can refer to
#829
#821

Hope it is the answer.

[ebroecker]

is this still an issue we have to do something?