[drm-msm:msm-next-staging 13/16] drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c:867:21: error: initialization of 'void (*)(struct msm_disp_state *, struct msm_kms *)' from incompatible pointer type 'void (*)(struct msm_disp_state *, struct msm_kms *)'

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

[-- Attachment #1: Type: text/plain, Size: 7482 bytes --]

tree:   https://gitlab.freedesktop.org/drm/msm.git msm-next-staging
head:   889098074648f7e585ccad8e041f2116a00a0c3c
commit: 1c3b7ac1a71d4266d7243fe5f7cd530322c59583 [13/16] drm/msm: pass dump state as a function argument
config: arm64-randconfig-r003-20210526 (attached as .config)
compiler: aarch64-linux-gcc (GCC) 9.3.0
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
        git remote add drm-msm https://gitlab.freedesktop.org/drm/msm.git
        git fetch --no-tags drm-msm msm-next-staging
        git checkout 1c3b7ac1a71d4266d7243fe5f7cd530322c59583
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=arm64 

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

All error/warnings (new ones prefixed by >>):

   In file included from drivers/gpu/drm/msm/hdmi/hdmi_connector.c:11:
>> drivers/gpu/drm/msm/msm_kms.h:127:26: warning: 'struct msm_disp_state' declared inside parameter list will not be visible outside of this definition or declaration
     127 |  void (*snapshot)(struct msm_disp_state *disp_state, struct msm_kms *kms);
         |                          ^~~~~~~~~~~~~~
--
   In file included from drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h:16,
                    from drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys.h:11,
                    from drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c:8:
>> drivers/gpu/drm/msm/msm_kms.h:127:26: warning: 'struct msm_disp_state' declared inside parameter list will not be visible outside of this definition or declaration
     127 |  void (*snapshot)(struct msm_disp_state *disp_state, struct msm_kms *kms);
         |                          ^~~~~~~~~~~~~~
   drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c: In function 'dpu_encoder_phys_cmd_wait_for_commit_done':
   drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c:692:31: warning: variable 'cmd_enc' set but not used [-Wunused-but-set-variable]
     692 |  struct dpu_encoder_phys_cmd *cmd_enc;
         |                               ^~~~~~~
--
   In file included from drivers/gpu/drm/msm/disp/msm_disp_snapshot.h:29,
                    from drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c:22:
>> drivers/gpu/drm/msm/msm_kms.h:127:26: warning: 'struct msm_disp_state' declared inside parameter list will not be visible outside of this definition or declaration
     127 |  void (*snapshot)(struct msm_disp_state *disp_state, struct msm_kms *kms);
         |                          ^~~~~~~~~~~~~~
>> drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c:867:21: error: initialization of 'void (*)(struct msm_disp_state *, struct msm_kms *)' from incompatible pointer type 'void (*)(struct msm_disp_state *, struct msm_kms *)' [-Werror=incompatible-pointer-types]
     867 |  .snapshot        = dpu_kms_mdp_snapshot,
         |                     ^~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c:867:21: note: (near initialization for 'kms_funcs.snapshot')
   cc1: some warnings being treated as errors
--
   In file included from drivers/gpu/drm/msm/disp/msm_disp_snapshot.h:29,
                    from drivers/gpu/drm/msm/disp/msm_disp_snapshot_util.c:8:
>> drivers/gpu/drm/msm/msm_kms.h:127:26: warning: 'struct msm_disp_state' declared inside parameter list will not be visible outside of this definition or declaration
     127 |  void (*snapshot)(struct msm_disp_state *disp_state, struct msm_kms *kms);
         |                          ^~~~~~~~~~~~~~
   drivers/gpu/drm/msm/disp/msm_disp_snapshot_util.c: In function 'msm_disp_snapshot_capture_state':
   drivers/gpu/drm/msm/disp/msm_disp_snapshot_util.c:130:3: error: implicit declaration of function 'msm_dp_snapshot'; did you mean 'msm_dsi_snapshot'? [-Werror=implicit-function-declaration]
     130 |   msm_dp_snapshot(disp_state, priv->dp);
         |   ^~~~~~~~~~~~~~~
         |   msm_dsi_snapshot
>> drivers/gpu/drm/msm/disp/msm_disp_snapshot_util.c:140:24: error: passing argument 1 of 'kms->funcs->snapshot' from incompatible pointer type [-Werror=incompatible-pointer-types]
     140 |   kms->funcs->snapshot(disp_state, kms);
         |                        ^~~~~~~~~~
         |                        |
         |                        struct msm_disp_state *
   drivers/gpu/drm/msm/disp/msm_disp_snapshot_util.c:140:24: note: expected 'struct msm_disp_state *' but argument is of type 'struct msm_disp_state *'
   cc1: some warnings being treated as errors


vim +867 drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c

e5aa9914db304c Abhinav Kumar     2021-04-16  846  
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  847  static const struct msm_kms_funcs kms_funcs = {
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  848  	.hw_init         = dpu_kms_hw_init,
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  849  	.irq_preinstall  = dpu_irq_preinstall,
220b856a3d3742 Tanmay Shah       2020-08-27  850  	.irq_postinstall = dpu_irq_postinstall,
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  851  	.irq_uninstall   = dpu_irq_uninstall,
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  852  	.irq             = dpu_irq,
e35a29d5c43250 Rob Clark         2019-08-29  853  	.enable_commit   = dpu_kms_enable_commit,
e35a29d5c43250 Rob Clark         2019-08-29  854  	.disable_commit  = dpu_kms_disable_commit,
cd6d923167b1bf Rob Clark         2019-08-29  855  	.vsync_time      = dpu_kms_vsync_time,
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  856  	.prepare_commit  = dpu_kms_prepare_commit,
9f6b65642bd2d7 Rob Clark         2019-08-29  857  	.flush_commit    = dpu_kms_flush_commit,
d4d2c60497cfc5 Rob Clark         2019-08-29  858  	.wait_flush      = dpu_kms_wait_flush,
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  859  	.complete_commit = dpu_kms_complete_commit,
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  860  	.enable_vblank   = dpu_kms_enable_vblank,
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  861  	.disable_vblank  = dpu_kms_disable_vblank,
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  862  	.check_modified_format = dpu_format_check_modified_format,
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  863  	.get_format      = dpu_get_msm_format,
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  864  	.round_pixclk    = dpu_kms_round_pixclk,
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  865  	.destroy         = dpu_kms_destroy,
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  866  	.set_encoder_mode = _dpu_kms_set_encoder_mode,
e5aa9914db304c Abhinav Kumar     2021-04-16 @867  	.snapshot        = dpu_kms_mdp_snapshot,
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  868  #ifdef CONFIG_DEBUG_FS
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  869  	.debugfs_init    = dpu_kms_debugfs_init,
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  870  #endif
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  871  };
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27  872  

:::::: The code at line 867 was first introduced by commit
:::::: e5aa9914db304c8cf1d6a11350555dd449f58bff drm/msm/disp/dpu1: add API to take DPU register snapshot

:::::: TO: Abhinav Kumar <abhinavk@codeaurora.org>
:::::: CC: Rob Clark <robdclark@chromium.org>

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

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 44542 bytes --]