2022-06-15 12:42:07

by Eugene Shalygin

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

From: Michael Carns <[email protected]>

This fixes a regression where coma lead to concatenating board names
and broke module loading for C8H.

Fixes: 5b4285c57b6f ("hwmon: (asus-ec-sensors) fix Formula VIII definition")

Signed-off-by: Michael Carns <[email protected]>
Signed-off-by: Eugene Shalygin <[email protected]>
---
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)",
},
--
2.35.1

2022-06-15 15:42:27

by Guenter Roeck

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

On Wed, Jun 15, 2022 at 02:25:44PM +0200, Eugene Shalygin wrote:
> From: Michael Carns <[email protected]>
>
> This fixes a regression where coma lead to concatenating board names
> and broke module loading for C8H.
>
> Fixes: 5b4285c57b6f ("hwmon: (asus-ec-sensors) fix Formula VIII definition")
>
> Signed-off-by: Michael Carns <[email protected]>
> Signed-off-by: Eugene Shalygin <[email protected]>

Applied.

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)",
> },