diff mbox series

[v2] hwmon: (pmbus/mpq8785) Convert comma to semicolon

Message ID 20240919064939.3282-1-shenlichuan@vivo.com (mailing list archive)
State Accepted
Headers show
Series [v2] hwmon: (pmbus/mpq8785) Convert comma to semicolon | expand

Commit Message

Shen Lichuan Sept. 19, 2024, 6:49 a.m. UTC 
To ensure code clarity and prevent potential errors, use ';' instead of
',' as a statement separator in the function mpq8785_identify.

Signed-off-by: Shen Lichuan <shenlichuan@vivo.com>
---
v2: 
* Changed subject prefix to "hwmon: (pmbus/mpq8785)".
* Modified the commit message to describe change more clearly.
v1: https://lore.kernel.org/all/faee3de3-4fd2-4b48-87af-348c8415b84c@roeck-us.net/

 drivers/hwmon/pmbus/mpq8785.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Guenter Roeck Sept. 19, 2024, 5:05 p.m. UTC | #1 
On Thu, Sep 19, 2024 at 02:49:39PM +0800, Shen Lichuan wrote:
> To ensure code clarity and prevent potential errors, use ';' instead of
> ',' as a statement separator in the function mpq8785_identify.
> 
> Signed-off-by: Shen Lichuan <shenlichuan@vivo.com>
> ---

Applied.

Thanks,
Guenter
diff mbox series

Patch

diff --git a/drivers/hwmon/pmbus/mpq8785.c b/drivers/hwmon/pmbus/mpq8785.c
index 7f87e117b49d..0d16491cd770 100644
--- a/drivers/hwmon/pmbus/mpq8785.c
+++ b/drivers/hwmon/pmbus/mpq8785.c
@@ -22,7 +22,7 @@  static int mpq8785_identify(struct i2c_client *client,
 		break;
 	case 1:
 	case 2:
-		info->format[PSC_VOLTAGE_OUT] = direct,
+		info->format[PSC_VOLTAGE_OUT] = direct;
 		info->m[PSC_VOLTAGE_OUT] = 64;
 		info->b[PSC_VOLTAGE_OUT] = 0;
 		info->R[PSC_VOLTAGE_OUT] = 1;