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 91381d323566987f97861c620747956e3e3e49c4..e465108d9998a1563a646ddb3fef6309a98337b4 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));