Re: [PATCH] hwmon: (asus-ec-sensors) add missing comma in board name list.

[Guenter Roeck]

On 6/15/22 03:45, Eugene Shalygin wrote:
> From: Michael Carns <mike@xxxxxxxxx>
>
> This fixes a regression introduced by commit 5b4285c57b6f
> ("hwmon: (asus-ec-sensors) fix Formula VIII definition")
> where coma lead to concatenating board names and broke module
> loading for C8H.
>
> Signed-off-by: Michael Carns <mike@xxxxxxxxx>
> Signed-off-by: Eugene Shalygin <eugene.shalygin@xxxxxxxxx>

We have a Fixes: tag for such purposes. Please use it.

Thanks,
Guenter

> ---
>   drivers/hwmon/asus-ec-sensors.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/hwmon/asus-ec-sensors.c b/drivers/hwmon/asus-ec-sensors.c
> index 886a71b4c64e..19d3ca71b360 100644
> --- a/drivers/hwmon/asus-ec-sensors.c
> +++ b/drivers/hwmon/asus-ec-sensors.c
> @@ -268,7 +268,7 @@ static const struct ec_board_info board_info[] = {
>   	},
>   	{
>   		.board_names = {
> -			"ROG CROSSHAIR VIII FORMULA"
> +			"ROG CROSSHAIR VIII FORMULA",
>   			"ROG CROSSHAIR VIII HERO",
>   			"ROG CROSSHAIR VIII HERO (WI-FI)",
>   		},