[Bug 204807] Hardware monitoring sensor nct6798d doesn't work unless acpi_enforce_resources=lax is enabled

[bugzilla-daemon@bugzilla.kernel.org]

https://bugzilla.kernel.org/show_bug.cgi?id=204807

--- Comment #123 from Denis Pauk (pauk.denis@gmail.com) ---
(In reply to Kamil Pietrzak from comment #122)
> I confrm patch works on my "TUF GAMING Z490-PLUS (WI-FI)".
> 
> [1295150.017048] nct6775: Found NCT6798D or compatible chip at 0x0:0x290
....
> 
> I also noticed that some voltage values reported by nct6775 differs from the
> ones reported by Asus software on Windows.
> 
> I changed voltage scaling factors to those listed below and now voltages are
> reported like on Asus software on Windows.
> 
> /*
>  * Some of the voltage inputs have internal scaling, the tables below
>  * contain 8 (the ADC LSB in mV) * scaling factor * 100
>  */
> static const u16 scale_in[15] = {
>       888, 4000, 1600, 1600, 9600, 800, 800, 1600, 1600, 1600, 1600, 1600,
> 800,
>       800, 800
> };

It looks like need to update code with custom scale values in relation to board
name. And it can be in future patches. 

Also need to look what functionality is nondestructive and can be merged in:
* https://gitlab.com/CalcProgrammer1/OpenRGB/-/blob/master/OpenRGB.patch
*
https://github.com/zeule/asus-wmi-ec-sensors/blob/master/asus-wmi-ec-sensors.c
* https://github.com/electrified/asus-wmi-sensors/
and cover maximum boards.

OpenRGB looks as good candidate for merge, as I see it uses i2c bus instead
asuswmi, and we already have ground for custom logic, it should be possible if
we have list of boards where such access is implemented by ASUS?.

-- 
You may reply to this email to add a comment.

You are receiving this mail because:
You are watching the assignee of the bug.