8 Jun
2023
8 Jun
'23
12:01 p.m.
On 08/06/2023 11:57, AngeloGioacchino Del Regno wrote:
- int test_done_1, test_done_2; + u8 test_done_1, test_done_2; int cycle_1, cycle_2;
Shouldn't be unsigned too ?
I'm wondering if it would be better (probably in another patch) to change the data type of the other variables too, to match their use-case. (maybe it's already the case, I'm just wondering)
In theory, yes, cycle_1 and 2 should be unsigned, but the signedness of this variable is getting used in the calibration logic later, as in the for loop they are both being reinitialized to -1 ... so I couldn't just switch 'em to unsigned.
Understood, thanks for your explanation.
--
Regards,
Alexandre