Hi Thomas,

I love your patch! Perhaps something to improve:

[auto build test WARNING on drm-misc/drm-misc-next]
[also build test WARNING on next-20220819]
[cannot apply to linus/master v6.0-rc1]
[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/Thomas-Zimmermann/drm-probe-helper-modes-Helpers-for-single-mode-displays/20220816-215250
base:   git://anongit.freedesktop.org/drm/drm-misc drm-misc-next
reproduce:
        # 
https://github.com/intel-lab-lkp/linux/commit/25770aac56aeff6f55419ea4316406ddbb29385f
        git remote add linux-review https://github.com/intel-lab-lkp/linux
        git fetch --no-tags linux-review 
Thomas-Zimmermann/drm-probe-helper-modes-Helpers-for-single-mode-displays/20220816-215250
        git checkout 25770aac56aeff6f55419ea4316406ddbb29385f
        make menuconfig
        # enable CONFIG_COMPILE_TEST, CONFIG_WARN_MISSING_DOCUMENTS, 
CONFIG_WARN_ABI_ERRORS
        make htmldocs

If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <l...@intel.com>

All warnings (new ones prefixed by >>):

>> ./include/drm/drm_modes.h:168: warning: expecting prototype for 
>> DRM_SIMPLE_MODE_INIT(). Prototype was for DRM_MODE_INIT() instead

vim +168 ./include/drm/drm_modes.h

   133  
   134  #define DRM_MODE(nm, t, c, hd, hss, hse, ht, hsk, vd, vss, vse, vt, vs, 
f) \
   135          .name = nm, .status = 0, .type = (t), .clock = (c), \
   136          .hdisplay = (hd), .hsync_start = (hss), .hsync_end = (hse), \
   137          .htotal = (ht), .hskew = (hsk), .vdisplay = (vd), \
   138          .vsync_start = (vss), .vsync_end = (vse), .vtotal = (vt), \
   139          .vscan = (vs), .flags = (f)
   140  
   141  /**
   142   * DRM_MODE_RES_MM - Calculates the display size from resolution and DPI
   143   * @res: The resolution in pixel
   144   * @dpi: The number of dots per inch
   145   */
   146  #define DRM_MODE_RES_MM(res, dpi)       \
   147          (((res) * 254ul) / ((dpi) * 10ul))
   148  
   149  #define __DRM_MODE_INIT(pix, hd, vd, hd_mm, vd_mm) \
   150          .type = DRM_MODE_TYPE_DRIVER, .clock = (pix), \
   151          .hdisplay = (hd), .hsync_start = (hd), .hsync_end = (hd), \
   152          .htotal = (hd), .vdisplay = (vd), .vsync_start = (vd), \
   153          .vsync_end = (vd), .vtotal = (vd), .width_mm = (hd_mm), \
   154          .height_mm = (vd_mm)
   155  
   156  /**
   157   * DRM_SIMPLE_MODE_INIT - Initialize display mode
   158   * @hz: Vertical refresh rate in Hertz
   159   * @hd: Horizontal resolution, width
   160   * @vd: Vertical resolution, height
   161   * @hd_mm: Display width in millimeters
   162   * @vd_mm: Display height in millimeters
   163   *
   164   * This macro initializes a &drm_display_mode that contains information 
about
   165   * refresh rate, resolution and physical size.
   166   */
   167  #define DRM_MODE_INIT(hz, hd, vd, hd_mm, vd_mm) \
 > 168          __DRM_MODE_INIT((hd) * (vd) * (hz) / 1000 /* kHz */, hd, vd, 
 > hd_mm, vd_mm)
   169  

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp

Reply via email to