Re: [PATCH v2 3/4] drm/plane: add drmm_universal_plane_alloc()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Philipp,

I love your patch! Perhaps something to improve:

[auto build test WARNING on drm-intel/for-linux-next]
[also build test WARNING on drm-tip/drm-tip linus/master v5.9-rc2 next-20200827]
[cannot apply to drm/drm-next]
[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]

url:    https://github.com/0day-ci/linux/commits/Philipp-Zabel/drm-add-drmm_encoder_alloc/20200828-000957
base:   git://anongit.freedesktop.org/drm-intel for-linux-next
config: arm-randconfig-r003-20200827 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project c10e63677f5d20f18010f8f68c631ddc97546f7d)
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # install arm cross compiling tool for clang build
        # apt-get install binutils-arm-linux-gnueabi
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@xxxxxxxxx>

All warnings (new ones prefixed by >>):

>> drivers/gpu/drm/drm_plane.c:301:6: warning: variable 'ap' is used uninitialized whenever 'if' condition is false [-Wsometimes-uninitialized]
           if (name)
               ^~~~
   drivers/gpu/drm/drm_plane.c:305:19: note: uninitialized use occurs here
                                            type, name, ap);
                                                        ^~
   drivers/gpu/drm/drm_plane.c:301:2: note: remove the 'if' if its condition is always true
           if (name)
           ^~~~~~~~~
   drivers/gpu/drm/drm_plane.c:298:2: note: variable 'ap' is declared here
           va_list ap;
           ^
   drivers/gpu/drm/drm_plane.c:344:6: warning: variable 'ap' is used uninitialized whenever 'if' condition is false [-Wsometimes-uninitialized]
           if (name)
               ^~~~
   drivers/gpu/drm/drm_plane.c:348:19: note: uninitialized use occurs here
                                            type, name, ap);
                                                        ^~
   drivers/gpu/drm/drm_plane.c:344:2: note: remove the 'if' if its condition is always true
           if (name)
           ^~~~~~~~~
   drivers/gpu/drm/drm_plane.c:332:2: note: variable 'ap' is declared here
           va_list ap;
           ^
   2 warnings generated.

# https://github.com/0day-ci/linux/commit/5f2373dfa20624f32ff28097eb734511ed8ca13e
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Philipp-Zabel/drm-add-drmm_encoder_alloc/20200828-000957
git checkout 5f2373dfa20624f32ff28097eb734511ed8ca13e
vim +301 drivers/gpu/drm/drm_plane.c

   271	
   272	/**
   273	 * drm_universal_plane_init - Initialize a new universal plane object
   274	 * @dev: DRM device
   275	 * @plane: plane object to init
   276	 * @possible_crtcs: bitmask of possible CRTCs
   277	 * @funcs: callbacks for the new plane
   278	 * @formats: array of supported formats (DRM_FORMAT\_\*)
   279	 * @format_count: number of elements in @formats
   280	 * @format_modifiers: array of struct drm_format modifiers terminated by
   281	 *                    DRM_FORMAT_MOD_INVALID
   282	 * @type: type of plane (overlay, primary, cursor)
   283	 * @name: printf style format string for the plane name, or NULL for default name
   284	 *
   285	 * Initializes a plane object of type @type.
   286	 *
   287	 * Returns:
   288	 * Zero on success, error code on failure.
   289	 */
   290	int drm_universal_plane_init(struct drm_device *dev, struct drm_plane *plane,
   291				     uint32_t possible_crtcs,
   292				     const struct drm_plane_funcs *funcs,
   293				     const uint32_t *formats, unsigned int format_count,
   294				     const uint64_t *format_modifiers,
   295				     enum drm_plane_type type,
   296				     const char *name, ...)
   297	{
   298		va_list ap;
   299		int ret;
   300	
 > 301		if (name)
   302			va_start(ap, name);
   303		ret = __drm_universal_plane_init(dev, plane, possible_crtcs, funcs,
   304						 formats, format_count, format_modifiers,
   305						 type, name, ap);
   306		if (name)
   307			va_end(ap);
   308		return ret;
   309	}
   310	EXPORT_SYMBOL(drm_universal_plane_init);
   311	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx

Attachment: .config.gz
Description: application/gzip

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel

[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux