Re: [PATCH v2 1/3] hwmon: (nct6775) Add additional ASUS motherboards

From: Oleksandr Natalenko <oleksandr@natalenko.name>
To: Denis Pauk <pauk.denis@gmail.com>
Cc: andy.shevchenko@gmail.com, pauk.denis@gmail.com,
	matt-testalltheway <sefoci9222@rerunway.com>,
	Kamil Dudka <kdudka@redhat.com>,
	Robert Swiecki <robert@swiecki.net>,
	Kamil Pietrzak <kpietrzak@disroot.org>, Igor <igor@svelig.com>,
	Jean Delvare <jdelvare@suse.com>,
	Guenter Roeck <linux@roeck-us.net>,
	Eugene Shalygin <eugene.shalygin@gmail.com>,
	linux-kernel@vger.kernel.org, linux-hwmon@vger.kernel.org
Subject: Re: [PATCH v2 1/3] hwmon: (nct6775) Add additional ASUS motherboards
Date: Thu, 07 Oct 2021 12:35:19 +0200	[thread overview]
Message-ID: <9337468.ToWr0jQUh6@natalenko.name> (raw)
In-Reply-To: <20211006222502.645003-2-pauk.denis@gmail.com>

Hello.

On čtvrtek 7. října 2021 0:24:59 CEST Denis Pauk wrote:
> Add support:
> * PRIME B360-PLUS
> * PRIME X570-PRO
> * ROG CROSSHAIR VIII FORMULA
> * ROG STRIX B550-I GAMING
> * ROG STRIX X570-F GAMING
> * ROG STRIX Z390-E GAMING
> * TUF GAMING B550-PRO
> * TUF GAMING Z490-PLUS
> * TUF GAMING Z490-PLUS (WI-FI)
> 
> BugLink: https://bugzilla.kernel.org/show_bug.cgi?id=204807
> Signed-off-by: Denis Pauk <pauk.denis@gmail.com>
> Tested-by: matt-testalltheway <sefoci9222@rerunway.com>
> Tested-by: Kamil Dudka <kdudka@redhat.com>
> Tested-by: Robert Swiecki <robert@swiecki.net>
> Tested-by: Kamil Pietrzak <kpietrzak@disroot.org>
> Tested-by: Igor <igor@svelig.com>
> Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
> ---
> Changes in v2:
>   - Without changes.
> ---
>  drivers/hwmon/nct6775.c | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/drivers/hwmon/nct6775.c b/drivers/hwmon/nct6775.c
> index aa58ead0ad43..8eaf86ea2433 100644
> --- a/drivers/hwmon/nct6775.c
> +++ b/drivers/hwmon/nct6775.c
> @@ -4986,20 +4986,29 @@ static int __init nct6775_find(int sioaddr, struct
> nct6775_sio_data *sio_data) static struct platform_device *pdev[2];
> 
>  static const char * const asus_wmi_boards[] = {
> +	"PRIME B360-PLUS",
>  	"PRIME B460-PLUS",
> +	"PRIME X570-PRO",
>  	"ROG CROSSHAIR VIII DARK HERO",
> +	"ROG CROSSHAIR VIII FORMULA",
>  	"ROG CROSSHAIR VIII HERO",
>  	"ROG CROSSHAIR VIII IMPACT",
>  	"ROG STRIX B550-E GAMING",
>  	"ROG STRIX B550-F GAMING",
>  	"ROG STRIX B550-F GAMING (WI-FI)",
> +	"ROG STRIX B550-I GAMING",
> +	"ROG STRIX X570-F GAMING",
> +	"ROG STRIX Z390-E GAMING",
>  	"ROG STRIX Z490-I GAMING",
>  	"TUF GAMING B550M-PLUS",
>  	"TUF GAMING B550M-PLUS (WI-FI)",
>  	"TUF GAMING B550-PLUS",
> +	"TUF GAMING B550-PRO",
>  	"TUF GAMING X570-PLUS",
>  	"TUF GAMING X570-PLUS (WI-FI)",
>  	"TUF GAMING X570-PRO (WI-FI)",
> +	"TUF GAMING Z490-PLUS",
> +	"TUF GAMING Z490-PLUS (WI-FI)",
>  };
> 
>  static int __init sensors_nct6775_init(void)

Please also add ASUS Pro WS X570-ACE as shown here [1].

Thanks.

[1] https://lore.kernel.org/lkml/20211003133344.9036-2-oleksandr@natalenko.name/

-- 
Oleksandr Natalenko (post-factum)