Re: [PATCH v2] hwmon: (nct6775) Add support for 18 IN readings for nct6799

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jul 18, 2023 at 07:29:12PM +0100, Ahmad Khalifa wrote:
> * Add additional VIN/IN_MIN/IN_MAX register values
> * Separate ALARM/BEEP bits for nct6799
> * Separate scaling factors for nct6799
> 
> Registers/alarms match for NCT6796D-S and NCT6799D-R
> Tested on NCT6799D-R for new IN/MIN/MAX and ALARMS
> 
> Signed-off-by: Ahmad Khalifa <ahmad@xxxxxxxxxx>
> ---
> 
> Notes:
>     v2 changes:
>     * Rebase after increased BITS with same registers
>     * Assign missing ALARM_BITS and add BEEP_BITS
>     * Add separate scaling_in and pass it through data as they were
>       missing in v1 and the factors differ with other chips

The scaling change is really a bug fix; it should be a separate patch
marked as Fixes:. It also affects NCT6798D.

Please split the patch into two so I can apply the bug fix for v6.5.

Thanks,
Guenter



[Index of Archives]     [LM Sensors]     [Linux Sound]     [ALSA Users]     [ALSA Devel]     [Linux Audio Users]     [Linux Media]     [Kernel]     [Gimp]     [Yosemite News]     [Linux Media]

  Powered by Linux