Am 11.02.23 um 19:54 schrieb Aleksa Savic:
On 2023-02-11 19:08:27 GMT+01:00, Guenter Roeck wrote:
aquaero is already supported, and the checksum is so far generated
and sent. Is it ignored ? Also, is it guaranteed that _all_ aquero devices
don't need it ?
Reading its sensors is currently supported, not writing to it (before these
patches).
The checksum is ignored and not needed for either aquaero 5 (which Leonard has)
nor 6 (which I have).
If it is not needed and ignored, does it really add value to selectively drop it ?
I think we can indeed remove that check.
I don't think that check can be removed as the checksum is not appended
to the control report but is in the last two bytes. So using the
checksum on Aquaero will overwrite the data at that location. It is
currently unknown what these two bytes do so I do not want to overwrite
them.
Thanks,
Aleksa
Either case, this change is not mentioned in the commit log, and it
violates the "one logical change per patch" rule. Please split it into
a separate patch and explain why the change is needed.
Another change to separate is the introduction of ctrl_report_id
and the secondary_ctrl_report variables, which is also done silently
and not explained. That should also be a separate patch to simplify
review.
I will separate the changes into more commits for the next version.
Regards,
Leonard
Thanks,
Guenter
