On 2/21/24 14:12, Ahmad Khalifa wrote:
On 21/02/2024 21:59, Guenter Roeck wrote:
Tiny note, i is in the range of 0..7 on nct6798/nct6799 if I
follow correctly? Still 8 > 2, well caught.
Sorry, I don't understand your comment. Yes, i is in the range of 0..7 on
nct6798/nct6799, which triggers the failure if i >= 2 because the code uses
the NCT6779_REG_TEMP_CONFIG array to identify configuration registers,
and that array only has two entries.
Ok, I think I was following correctly, just thrown off by the comment
on the bug report thread about i being 0..11:
The range of "i" is 0..11, and the size of the reg_temp_config[] array is 2. Oops.
Ah, ok. I should have said "the possible range..."
Guenter
