auxdisplay: lcd2s: Use array size explicitly in lcd2s_gotoxy()
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Tue, 8 Mar 2022 15:11:19 +0000 (17:11 +0200)
committerMiguel Ojeda <ojeda@kernel.org>
Fri, 18 Mar 2022 19:31:14 +0000 (20:31 +0100)
Currently the reading of the onstack array is confusing since two
out of three members are of different types. Let it be more clear
by explicitly set the array size, so everybody will understand that
parameters are cast to the type of the array.

While at it, add a missed space.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Miguel Ojeda <ojeda@kernel.org>
drivers/auxdisplay/lcd2s.c

index 91381d3..e465108 100644 (file)
@@ -106,7 +106,7 @@ static int lcd2s_print(struct charlcd *lcd, int c)
 static int lcd2s_gotoxy(struct charlcd *lcd, unsigned int x, unsigned int y)
 {
        struct lcd2s_data *lcd2s = lcd->drvdata;
-       u8 buf[] = { LCD2S_CMD_CUR_POS, y + 1, x + 1};
+       u8 buf[3] = { LCD2S_CMD_CUR_POS, y + 1, x + 1 };
 
        lcd2s_i2c_master_send(lcd2s->i2c, buf, sizeof(buf));