CC: [email protected]
CC: [email protected]
TO: Stephen Boyd <[email protected]>
CC: Mark Brown <[email protected]>
CC: Arnd Bergmann <[email protected]>

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 
master
head:   9cdbf6467424045617cd6e79dcaad06bb8efa31c
commit: bbd7ffdbef6888459f301c5889f3b14ada38b913 clk: Allow the common clk 
framework to be selectable
date:   12 months ago
:::::: branch date: 5 hours ago
:::::: commit date: 12 months ago
compiler: m68k-linux-gcc (GCC) 9.3.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <[email protected]>


cppcheck possible warnings: (new ones prefixed by >>, may not real problems)

>> drivers/gpu/drm/vc4/vc4_crtc.c:595:50: warning: Function 
>> 'vc4_crtc_get_margins' argument order different: declaration 'state, right, 
>> left, top, bottom' definition 'state, left, right, top, bottom' 
>> [funcArgOrderDifferent]
   void vc4_crtc_get_margins(struct drm_crtc_state *state,
                                                    ^
   drivers/gpu/drm/vc4/vc4_drv.h:757:50: note: Function 'vc4_crtc_get_margins' 
argument order different: declaration 'state, right, left, top, bottom' 
definition 'state, left, right, top, bottom'
   void vc4_crtc_get_margins(struct drm_crtc_state *state,
                                                    ^
   drivers/gpu/drm/vc4/vc4_crtc.c:595:50: note: Function 'vc4_crtc_get_margins' 
argument order different: declaration 'state, right, left, top, bottom' 
definition 'state, left, right, top, bottom'
   void vc4_crtc_get_margins(struct drm_crtc_state *state,
                                                    ^

vim +595 drivers/gpu/drm/vc4/vc4_crtc.c

acc1be1d351e8d Mario Kleiner   2016-07-19  594  
666e73587f90f4 Boris Brezillon 2018-12-06 @595  void 
vc4_crtc_get_margins(struct drm_crtc_state *state,
666e73587f90f4 Boris Brezillon 2018-12-06  596                            
unsigned int *left, unsigned int *right,
666e73587f90f4 Boris Brezillon 2018-12-06  597                            
unsigned int *top, unsigned int *bottom)
666e73587f90f4 Boris Brezillon 2018-12-06  598  {
666e73587f90f4 Boris Brezillon 2018-12-06  599          struct vc4_crtc_state 
*vc4_state = to_vc4_crtc_state(state);
666e73587f90f4 Boris Brezillon 2018-12-06  600          struct 
drm_connector_state *conn_state;
666e73587f90f4 Boris Brezillon 2018-12-06  601          struct drm_connector 
*conn;
666e73587f90f4 Boris Brezillon 2018-12-06  602          int i;
666e73587f90f4 Boris Brezillon 2018-12-06  603  
666e73587f90f4 Boris Brezillon 2018-12-06  604          *left = 
vc4_state->margins.left;
666e73587f90f4 Boris Brezillon 2018-12-06  605          *right = 
vc4_state->margins.right;
666e73587f90f4 Boris Brezillon 2018-12-06  606          *top = 
vc4_state->margins.top;
666e73587f90f4 Boris Brezillon 2018-12-06  607          *bottom = 
vc4_state->margins.bottom;
666e73587f90f4 Boris Brezillon 2018-12-06  608  
666e73587f90f4 Boris Brezillon 2018-12-06  609          /* We have to interate 
over all new connector states because
666e73587f90f4 Boris Brezillon 2018-12-06  610           * 
vc4_crtc_get_margins() might be called before
666e73587f90f4 Boris Brezillon 2018-12-06  611           * 
vc4_crtc_atomic_check() which means margins info in vc4_crtc_state
666e73587f90f4 Boris Brezillon 2018-12-06  612           * might be outdated.
666e73587f90f4 Boris Brezillon 2018-12-06  613           */
666e73587f90f4 Boris Brezillon 2018-12-06  614          
for_each_new_connector_in_state(state->state, conn, conn_state, i) {
666e73587f90f4 Boris Brezillon 2018-12-06  615                  if 
(conn_state->crtc != state->crtc)
666e73587f90f4 Boris Brezillon 2018-12-06  616                          
continue;
666e73587f90f4 Boris Brezillon 2018-12-06  617  
666e73587f90f4 Boris Brezillon 2018-12-06  618                  *left = 
conn_state->tv.margins.left;
666e73587f90f4 Boris Brezillon 2018-12-06  619                  *right = 
conn_state->tv.margins.right;
666e73587f90f4 Boris Brezillon 2018-12-06  620                  *top = 
conn_state->tv.margins.top;
666e73587f90f4 Boris Brezillon 2018-12-06  621                  *bottom = 
conn_state->tv.margins.bottom;
666e73587f90f4 Boris Brezillon 2018-12-06  622                  break;
666e73587f90f4 Boris Brezillon 2018-12-06  623          }
666e73587f90f4 Boris Brezillon 2018-12-06  624  }
666e73587f90f4 Boris Brezillon 2018-12-06  625  

:::::: The code at line 595 was first introduced by commit
:::::: 666e73587f90f42d90385c1bea1009a650bf73f4 drm/vc4: Take margin setup into 
account when updating planes

:::::: TO: Boris Brezillon <[email protected]>
:::::: CC: Boris Brezillon <[email protected]>

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/[email protected]
_______________________________________________
kbuild mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to