staging: fbtft: Consider type of init sequence values in fbtft_init_display()
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Wed, 16 Mar 2022 16:53:51 +0000 (18:53 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 18 Mar 2022 12:42:48 +0000 (13:42 +0100)
In the fbtft_init_display() the init sequence is printed for
the debug purposes. Unfortunately the current code doesn't take
into account that values in the buffer are of the s16 type.

Consider that and replace the printing code with fbtft_par_dbg_hex()
call.

Fixes: b888897014a9 ("staging/fbtft: Remove all strcpy() uses")
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/r/20220316165351.58107-2-andriy.shevchenko@linux.intel.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/fbtft/fbtft-core.c

index 6dc77895a87cf6637a0f74768069b18423a28945..3b182115bd6e7f71ba2afed108e13b4d08b4ee1e 100644 (file)
@@ -1034,10 +1034,9 @@ int fbtft_init_display(struct fbtft_par *par)
                        for (j = 0; par->init_sequence[i + 1 + j] >= 0; j++)
                                ;
 
-                       fbtft_par_dbg(DEBUG_INIT_DISPLAY, par,
-                                     "init: write(0x%02X) %*ph\n",
-                                     par->init_sequence[i], j,
-                                     &par->init_sequence[i + 1]);
+                       fbtft_par_dbg_hex(DEBUG_INIT_DISPLAY, par, par->info->device,
+                                         s16, &par->init_sequence[i + 1], j,
+                                         "init: write(0x%02X)", par->init_sequence[i]);
 
                        /* Write */
                        j = 0;