Re: [PATCH] GPU: drm: meson inline drm_simple_encoder_init()

kernel test robot <[email protected]>
Newsgroups org.freedesktop.lists.dri-devel,dev.linux.lists.oe-kbuild-all,org.kernel.vger.linux-kernel
Message-ID <[email protected]> 
Hi Liam,

kernel test robot noticed the following build errors:

[auto build test ERROR on drm-misc/drm-misc-next]
[also build test ERROR on daeinki-drm-exynos/exynos-drm-next drm/drm-next drm-i915/for-linux-next drm-i915/for-linux-next-fixes drm-tip/drm-tip linus/master v7.2-rc7 next-20260811]
[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/Liam-Zuiderhoek/GPU-drm-meson-inline-drm_simple_encoder_init/20260813-051125
base:   https://gitlab.freedesktop.org/drm/misc/kernel.git drm-misc-next
patch link:    https://lore.kernel.org/r/20260622180832.28143-1-zuiderhoekl%40gmail.com
patch subject: [PATCH] GPU: drm: meson inline drm_simple_encoder_init()
config: powerpc-allmodconfig (https://download.01.org/0day-ci/archive/20260813/[email protected]/config)
compiler: powerpc64-linux-gcc (GCC) 16.1.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260813/[email protected]/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 <[email protected]>
| Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/

All errors (new ones prefixed by >>):

   In file included from include/drm/drm_property.h:30,
                    from include/drm/drm_connector.h:33,
                    from include/drm/drm_modes.h:33,
                    from include/drm/drm_crtc.h:32,
                    from include/drm/drm_atomic_helper.h:31,
                    from drivers/gpu/drm/meson/meson_encoder_hdmi.c:21:
   drivers/gpu/drm/meson/meson_encoder_hdmi.c: In function 'meson_encoder_hdmi_probe':
>> include/uapi/drm/drm_mode.h:358:33: error: passing argument 3 of 'drm_encoder_init' makes pointer from integer without a cast [-Wint-conversion]
     358 | #define DRM_MODE_ENCODER_TMDS   2
         |                                 ^
         |                                 |
         |                                 int
   drivers/gpu/drm/meson/meson_encoder_hdmi.c:411:32: note: in expansion of macro 'DRM_MODE_ENCODER_TMDS'
     411 |                                DRM_MODE_ENCODER_TMDS, NULL);
         |                                ^~~~~~~~~~~~~~~~~~~~~
   In file included from include/drm/drm_modeset_helper_vtables.h:33,
                    from include/drm/drm_atomic_helper.h:32:
   include/drm/drm_encoder.h:212:54: note: expected 'const struct drm_encoder_funcs *' but argument is of type 'int'
     212 |                      const struct drm_encoder_funcs *funcs,
         |                      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~
   In file included from include/uapi/linux/posix_types.h:5,
                    from include/uapi/linux/types.h:14,
                    from include/linux/types.h:5,
                    from include/linux/kasan-checks.h:5,
                    from include/asm-generic/rwonce.h:26,
                    from ./arch/powerpc/include/generated/asm/rwonce.h:1,
                    from include/linux/compiler.h:369,
                    from include/linux/err.h:5,
                    from include/linux/clk.h:12,
                    from drivers/gpu/drm/meson/meson_encoder_hdmi.c:8:
>> include/linux/stddef.h:8:14: error: passing argument 4 of 'drm_encoder_init' makes integer from pointer without a cast [-Wint-conversion]
       8 | #define NULL ((void *)0)
         |              ^~~~~~~~~~~
         |              |
         |              void *
   drivers/gpu/drm/meson/meson_encoder_hdmi.c:411:55: note: in expansion of macro 'NULL'
     411 |                                DRM_MODE_ENCODER_TMDS, NULL);
         |                                                       ^~~~
   include/drm/drm_encoder.h:213:26: note: expected 'int' but argument is of type 'void *'
     213 |                      int encoder_type, const char *name, ...);
         |                      ~~~~^~~~~~~~~~~~
>> drivers/gpu/drm/meson/meson_encoder_hdmi.c:410:15: error: too few arguments to function 'drm_encoder_init'; expected at least 5, have 4
     410 |         ret = drm_encoder_init(priv->drm, &meson_encoder_hdmi->encoder,
         |               ^~~~~~~~~~~~~~~~
   include/drm/drm_encoder.h:210:5: note: declared here
     210 | int drm_encoder_init(struct drm_device *dev,
         |     ^~~~~~~~~~~~~~~~
--
   In file included from include/drm/drm_property.h:30,
                    from include/drm/drm_connector.h:33,
                    from include/drm/drm_modes.h:33,
                    from include/drm/drm_crtc.h:32,
                    from include/drm/drm_atomic_helper.h:31,
                    from drm/meson/meson_encoder_hdmi.c:21:
   drm/meson/meson_encoder_hdmi.c: In function 'meson_encoder_hdmi_probe':
>> include/uapi/drm/drm_mode.h:358:33: error: passing argument 3 of 'drm_encoder_init' makes pointer from integer without a cast [-Wint-conversion]
     358 | #define DRM_MODE_ENCODER_TMDS   2
         |                                 ^
         |                                 |
         |                                 int
   drm/meson/meson_encoder_hdmi.c:411:32: note: in expansion of macro 'DRM_MODE_ENCODER_TMDS'
     411 |                                DRM_MODE_ENCODER_TMDS, NULL);
         |                                ^~~~~~~~~~~~~~~~~~~~~
   In file included from include/drm/drm_modeset_helper_vtables.h:33,
                    from include/drm/drm_atomic_helper.h:32:
   include/drm/drm_encoder.h:212:54: note: expected 'const struct drm_encoder_funcs *' but argument is of type 'int'
     212 |                      const struct drm_encoder_funcs *funcs,
         |                      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~
   In file included from include/uapi/linux/posix_types.h:5,
                    from include/uapi/linux/types.h:14,
                    from include/linux/types.h:5,
                    from include/linux/kasan-checks.h:5,
                    from include/asm-generic/rwonce.h:26,
                    from arch/powerpc/include/generated/asm/rwonce.h:1,
                    from include/linux/compiler.h:369,
                    from include/linux/err.h:5,
                    from include/linux/clk.h:12,
                    from drm/meson/meson_encoder_hdmi.c:8:
>> include/linux/stddef.h:8:14: error: passing argument 4 of 'drm_encoder_init' makes integer from pointer without a cast [-Wint-conversion]
       8 | #define NULL ((void *)0)
         |              ^~~~~~~~~~~
         |              |
         |              void *
   drm/meson/meson_encoder_hdmi.c:411:55: note: in expansion of macro 'NULL'
     411 |                                DRM_MODE_ENCODER_TMDS, NULL);
         |                                                       ^~~~
   include/drm/drm_encoder.h:213:26: note: expected 'int' but argument is of type 'void *'
     213 |                      int encoder_type, const char *name, ...);
         |                      ~~~~^~~~~~~~~~~~
   drm/meson/meson_encoder_hdmi.c:410:15: error: too few arguments to function 'drm_encoder_init'; expected at least 5, have 4
     410 |         ret = drm_encoder_init(priv->drm, &meson_encoder_hdmi->encoder,
         |               ^~~~~~~~~~~~~~~~
   include/drm/drm_encoder.h:210:5: note: declared here
     210 | int drm_encoder_init(struct drm_device *dev,
         |     ^~~~~~~~~~~~~~~~


vim +/drm_encoder_init +358 include/uapi/drm/drm_mode.h

8cf5c917715153 include/drm/drm_mode.h      Jesse Barnes     2011-11-14  355  
f453ba0460742a include/drm/drm_mode.h      Dave Airlie      2008-11-07  356  #define DRM_MODE_ENCODER_NONE	0
f453ba0460742a include/drm/drm_mode.h      Dave Airlie      2008-11-07  357  #define DRM_MODE_ENCODER_DAC	1
f453ba0460742a include/drm/drm_mode.h      Dave Airlie      2008-11-07 @358  #define DRM_MODE_ENCODER_TMDS	2
f453ba0460742a include/drm/drm_mode.h      Dave Airlie      2008-11-07  359  #define DRM_MODE_ENCODER_LVDS	3
f453ba0460742a include/drm/drm_mode.h      Dave Airlie      2008-11-07  360  #define DRM_MODE_ENCODER_TVDAC	4
a7331e5cb2cc68 include/drm/drm_mode.h      Thomas Hellstrom 2011-10-22  361  #define DRM_MODE_ENCODER_VIRTUAL 5
b89232732f642b include/uapi/drm/drm_mode.h Shobhit Kumar    2013-08-27  362  #define DRM_MODE_ENCODER_DSI	6
182407a6ed5333 include/uapi/drm/drm_mode.h Dave Airlie      2014-05-02  363  #define DRM_MODE_ENCODER_DPMST	7
0b27c02a7f1c69 include/uapi/drm/drm_mode.h Eric Anholt      2016-03-18  364  #define DRM_MODE_ENCODER_DPI	8
f453ba0460742a include/drm/drm_mode.h      Dave Airlie      2008-11-07  365  

--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.