Re: [PATCH v4 2/2] drm/tiny: add driver for Apple Touch Bars in x86 Macs

[kernel test robot <lkp@intel.com>]

Hi Aditya,

kernel test robot noticed the following build warnings:

[auto build test WARNING on linus/master]
[also build test WARNING on v6.14-rc4 next-20250227]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Aditya-Garg/drm-format-helper-Add-conversion-from-XRGB8888-to-BGR888/20250224-214352
base:   linus/master
patch link:    https://lore.kernel.org/r/844C1D39-4891-4DC2-8458-F46FA1B59FA0%40live.com
patch subject: [PATCH v4 2/2] drm/tiny: add driver for Apple Touch Bars in x86 Macs
config: mips-allyesconfig (https://download.01.org/0day-ci/archive/20250228/202502280740.4ffoKiLx-lkp@intel.com/config)
compiler: mips-linux-gcc (GCC) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250228/202502280740.4ffoKiLx-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202502280740.4ffoKiLx-lkp@intel.com/

All warnings (new ones prefixed by >>):

   In file included from drivers/gpu/drm/tiny/appletbdrm.c:13:
   drivers/gpu/drm/tiny/appletbdrm.c: In function 'appletbdrm_send_request':
>> include/drm/drm_print.h:589:54: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'size_t' {aka 'unsigned int'} [-Wformat=]
     589 |         dev_##level##type((drm) ? (drm)->dev : NULL, "[drm] " fmt, ##__VA_ARGS__)
         |                                                      ^~~~~~~~
   include/linux/dev_printk.h:110:30: note: in definition of macro 'dev_printk_index_wrap'
     110 |                 _p_func(dev, fmt, ##__VA_ARGS__);                       \
         |                              ^~~
   include/linux/dev_printk.h:154:56: note: in expansion of macro 'dev_fmt'
     154 |         dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
         |                                                        ^~~~~~~
   include/drm/drm_print.h:589:9: note: in expansion of macro 'dev_err'
     589 |         dev_##level##type((drm) ? (drm)->dev : NULL, "[drm] " fmt, ##__VA_ARGS__)
         |         ^~~~
   include/drm/drm_print.h:602:9: note: in expansion of macro '__drm_printk'
     602 |         __drm_printk((drm), err,, "*ERROR* " fmt, ##__VA_ARGS__)
         |         ^~~~~~~~~~~~
   drivers/gpu/drm/tiny/appletbdrm.c:173:17: note: in expansion of macro 'drm_err'
     173 |                 drm_err(drm, "Actual size (%d) doesn't match expected size (%lu)\n",
         |                 ^~~~~~~
   drivers/gpu/drm/tiny/appletbdrm.c: In function 'appletbdrm_read_response':
>> include/drm/drm_print.h:589:54: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'size_t' {aka 'unsigned int'} [-Wformat=]
     589 |         dev_##level##type((drm) ? (drm)->dev : NULL, "[drm] " fmt, ##__VA_ARGS__)
         |                                                      ^~~~~~~~
   include/linux/dev_printk.h:110:30: note: in definition of macro 'dev_printk_index_wrap'
     110 |                 _p_func(dev, fmt, ##__VA_ARGS__);                       \
         |                              ^~~
   include/linux/dev_printk.h:154:56: note: in expansion of macro 'dev_fmt'
     154 |         dev_printk_index_wrap(_dev_err, KERN_ERR, dev, dev_fmt(fmt), ##__VA_ARGS__)
         |                                                        ^~~~~~~
   include/drm/drm_print.h:589:9: note: in expansion of macro 'dev_err'
     589 |         dev_##level##type((drm) ? (drm)->dev : NULL, "[drm] " fmt, ##__VA_ARGS__)
         |         ^~~~
   include/drm/drm_print.h:602:9: note: in expansion of macro '__drm_printk'
     602 |         __drm_printk((drm), err,, "*ERROR* " fmt, ##__VA_ARGS__)
         |         ^~~~~~~~~~~~
   drivers/gpu/drm/tiny/appletbdrm.c:214:17: note: in expansion of macro 'drm_err'
     214 |                 drm_err(drm, "Actual size (%d) doesn't match expected size (%lu)\n",
         |                 ^~~~~~~
   drivers/gpu/drm/tiny/appletbdrm.c: In function 'appletbdrm_primary_plane_duplicate_state':
   drivers/gpu/drm/tiny/appletbdrm.c:524:40: warning: variable 'old_appletbdrm_state' set but not used [-Wunused-but-set-variable]
     524 |         struct appletbdrm_plane_state *old_appletbdrm_state;
         |                                        ^~~~~~~~~~~~~~~~~~~~


vim +589 include/drm/drm_print.h

e820f52577b14c6 Jim Cromie            2022-09-11  549  
02c9656b2f0d699 Haneen Mohammed       2017-10-17  550  /**
b52817e9de06a3a Mauro Carvalho Chehab 2020-10-27  551   * DRM_DEV_DEBUG() - Debug output for generic drm code
02c9656b2f0d699 Haneen Mohammed       2017-10-17  552   *
306589856399e18 Douglas Anderson      2021-09-21  553   * NOTE: this is deprecated in favor of drm_dbg_core().
306589856399e18 Douglas Anderson      2021-09-21  554   *
091756bbb1a9613 Haneen Mohammed       2017-10-17  555   * @dev: device pointer
091756bbb1a9613 Haneen Mohammed       2017-10-17  556   * @fmt: printf() like format string.
02c9656b2f0d699 Haneen Mohammed       2017-10-17  557   */
db87086492581c8 Joe Perches           2018-03-16  558  #define DRM_DEV_DEBUG(dev, fmt, ...)					\
db87086492581c8 Joe Perches           2018-03-16  559  	drm_dev_dbg(dev, DRM_UT_CORE, fmt, ##__VA_ARGS__)
b52817e9de06a3a Mauro Carvalho Chehab 2020-10-27  560  /**
b52817e9de06a3a Mauro Carvalho Chehab 2020-10-27  561   * DRM_DEV_DEBUG_DRIVER() - Debug output for vendor specific part of the driver
b52817e9de06a3a Mauro Carvalho Chehab 2020-10-27  562   *
306589856399e18 Douglas Anderson      2021-09-21  563   * NOTE: this is deprecated in favor of drm_dbg() or dev_dbg().
306589856399e18 Douglas Anderson      2021-09-21  564   *
b52817e9de06a3a Mauro Carvalho Chehab 2020-10-27  565   * @dev: device pointer
b52817e9de06a3a Mauro Carvalho Chehab 2020-10-27  566   * @fmt: printf() like format string.
b52817e9de06a3a Mauro Carvalho Chehab 2020-10-27  567   */
db87086492581c8 Joe Perches           2018-03-16  568  #define DRM_DEV_DEBUG_DRIVER(dev, fmt, ...)				\
db87086492581c8 Joe Perches           2018-03-16  569  	drm_dev_dbg(dev, DRM_UT_DRIVER,	fmt, ##__VA_ARGS__)
b52817e9de06a3a Mauro Carvalho Chehab 2020-10-27  570  /**
b52817e9de06a3a Mauro Carvalho Chehab 2020-10-27  571   * DRM_DEV_DEBUG_KMS() - Debug output for modesetting code
b52817e9de06a3a Mauro Carvalho Chehab 2020-10-27  572   *
306589856399e18 Douglas Anderson      2021-09-21  573   * NOTE: this is deprecated in favor of drm_dbg_kms().
306589856399e18 Douglas Anderson      2021-09-21  574   *
b52817e9de06a3a Mauro Carvalho Chehab 2020-10-27  575   * @dev: device pointer
b52817e9de06a3a Mauro Carvalho Chehab 2020-10-27  576   * @fmt: printf() like format string.
b52817e9de06a3a Mauro Carvalho Chehab 2020-10-27  577   */
db87086492581c8 Joe Perches           2018-03-16  578  #define DRM_DEV_DEBUG_KMS(dev, fmt, ...)				\
db87086492581c8 Joe Perches           2018-03-16  579  	drm_dev_dbg(dev, DRM_UT_KMS, fmt, ##__VA_ARGS__)
a18b21929453af7 Lyude Paul            2018-07-16  580  
fb6c7ab8718eb25 Jani Nikula           2019-12-10  581  /*
fb6c7ab8718eb25 Jani Nikula           2019-12-10  582   * struct drm_device based logging
fb6c7ab8718eb25 Jani Nikula           2019-12-10  583   *
fb6c7ab8718eb25 Jani Nikula           2019-12-10  584   * Prefer drm_device based logging over device or prink based logging.
fb6c7ab8718eb25 Jani Nikula           2019-12-10  585   */
fb6c7ab8718eb25 Jani Nikula           2019-12-10  586  
fb6c7ab8718eb25 Jani Nikula           2019-12-10  587  /* Helper for struct drm_device based logging. */
fb6c7ab8718eb25 Jani Nikula           2019-12-10  588  #define __drm_printk(drm, level, type, fmt, ...)			\
e04d24c4e8062b5 Luben Tuikov          2023-11-16 @589  	dev_##level##type((drm) ? (drm)->dev : NULL, "[drm] " fmt, ##__VA_ARGS__)
fb6c7ab8718eb25 Jani Nikula           2019-12-10  590  
fb6c7ab8718eb25 Jani Nikula           2019-12-10  591  

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki