Re: [PATCH 1/1] hwmon: nct7802: Fix for temp6 (PECI1) processed even if PECI1 disabled

[Guenter Roeck <linux@xxxxxxxxxxxx>]

On 7/21/23 09:38, Gilles BULOZ wrote:
> Fix for temp6 (PECI1) processed even if PECI1 disabled
You should describe the problem here.

Guenter

> Signed-off-by: Gilles Buloz <gilles.buloz@xxxxxxxxxxx>
> ---
>   drivers/hwmon/nct7802.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/hwmon/nct7802.c b/drivers/hwmon/nct7802.c
> index 9339bfc..024cff1 100644
> --- a/drivers/hwmon/nct7802.c
> +++ b/drivers/hwmon/nct7802.c
> @@ -725,7 +725,7 @@ static umode_t nct7802_temp_is_visible(struct kobject *kobj,
>       if (index >= 38 && index < 46 && !(reg & 0x01))        /* PECI 0 */
>           return 0;
>
> -    if (index >= 0x46 && (!(reg & 0x02)))  /* PECI 1 */
> +    if (index >= 46 && !(reg & 0x02))            /* PECI 1 */
>           return 0;
>
>       return attr->mode;