CC: [email protected] BCC: [email protected] CC: Linux Memory Management List <[email protected]> TO: Thomas Zimmermann <[email protected]> CC: Lyude Paul <[email protected]> CC: Javier Martinez Canillas <[email protected]> CC: Alex Deucher <[email protected]>
tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master head: 5469f0c06732a077c70a759a81f2a1f00b277694 commit: da68386d9edb1f57abf886febe5c5169ebd4d2c9 [5845/7959] drm: Rename dp/ to display/ :::::: branch date: 7 hours ago :::::: commit date: 4 days ago compiler: sparc64-linux-gcc (GCC) 11.3.0 reproduce (cppcheck warning): # apt-get install cppcheck git checkout da68386d9edb1f57abf886febe5c5169ebd4d2c9 cppcheck --quiet --enable=style,performance,portability --template=gcc FILE 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/drm_mipi_dsi.c:307:73: warning: Parameter 'node' can be >> declared with const [constParameter] struct mipi_dsi_host *of_find_mipi_dsi_host_by_node(struct device_node *node) ^ -- >> drivers/gpu/drm/display/drm_dp_aux_dev.c:251:76: warning: Parameter 'aux' >> can be declared with const [constParameter] static struct drm_dp_aux_dev *drm_dp_aux_dev_get_by_aux(struct drm_dp_aux *aux) ^ >> drivers/gpu/drm/display/drm_dp_aux_dev.c:263:13: warning: Uninitialized >> variable: iter->aux [uninitvar] if (iter->aux == aux) { ^ -- >> drivers/gpu/drm/display/drm_dp_mst_topology.c:5128:34: warning: Parameter >> 'branch' can be declared with const [constParameter] struct drm_dp_mst_branch *branch) ^ vim +/aux +251 drivers/gpu/drm/display/drm_dp_aux_dev.c e94cb37b34eb8a8 drivers/gpu/drm/drm_dp_aux_dev.c Rafael Antognolli 2016-01-21 250 e94cb37b34eb8a8 drivers/gpu/drm/drm_dp_aux_dev.c Rafael Antognolli 2016-01-21 @251 static struct drm_dp_aux_dev *drm_dp_aux_dev_get_by_aux(struct drm_dp_aux *aux) e94cb37b34eb8a8 drivers/gpu/drm/drm_dp_aux_dev.c Rafael Antognolli 2016-01-21 252 { e94cb37b34eb8a8 drivers/gpu/drm/drm_dp_aux_dev.c Rafael Antognolli 2016-01-21 253 struct drm_dp_aux_dev *iter, *aux_dev = NULL; e94cb37b34eb8a8 drivers/gpu/drm/drm_dp_aux_dev.c Rafael Antognolli 2016-01-21 254 int id; e94cb37b34eb8a8 drivers/gpu/drm/drm_dp_aux_dev.c Rafael Antognolli 2016-01-21 255 e94cb37b34eb8a8 drivers/gpu/drm/drm_dp_aux_dev.c Rafael Antognolli 2016-01-21 256 /* don't increase kref count here because this function should only be e94cb37b34eb8a8 drivers/gpu/drm/drm_dp_aux_dev.c Rafael Antognolli 2016-01-21 257 * used by drm_dp_aux_unregister_devnode. Thus, it will always have at e94cb37b34eb8a8 drivers/gpu/drm/drm_dp_aux_dev.c Rafael Antognolli 2016-01-21 258 * least one reference - the one that drm_dp_aux_register_devnode e94cb37b34eb8a8 drivers/gpu/drm/drm_dp_aux_dev.c Rafael Antognolli 2016-01-21 259 * created e94cb37b34eb8a8 drivers/gpu/drm/drm_dp_aux_dev.c Rafael Antognolli 2016-01-21 260 */ e94cb37b34eb8a8 drivers/gpu/drm/drm_dp_aux_dev.c Rafael Antognolli 2016-01-21 261 mutex_lock(&aux_idr_mutex); e94cb37b34eb8a8 drivers/gpu/drm/drm_dp_aux_dev.c Rafael Antognolli 2016-01-21 262 idr_for_each_entry(&aux_idr, iter, id) { e94cb37b34eb8a8 drivers/gpu/drm/drm_dp_aux_dev.c Rafael Antognolli 2016-01-21 @263 if (iter->aux == aux) { e94cb37b34eb8a8 drivers/gpu/drm/drm_dp_aux_dev.c Rafael Antognolli 2016-01-21 264 aux_dev = iter; e94cb37b34eb8a8 drivers/gpu/drm/drm_dp_aux_dev.c Rafael Antognolli 2016-01-21 265 break; e94cb37b34eb8a8 drivers/gpu/drm/drm_dp_aux_dev.c Rafael Antognolli 2016-01-21 266 } e94cb37b34eb8a8 drivers/gpu/drm/drm_dp_aux_dev.c Rafael Antognolli 2016-01-21 267 } e94cb37b34eb8a8 drivers/gpu/drm/drm_dp_aux_dev.c Rafael Antognolli 2016-01-21 268 mutex_unlock(&aux_idr_mutex); e94cb37b34eb8a8 drivers/gpu/drm/drm_dp_aux_dev.c Rafael Antognolli 2016-01-21 269 return aux_dev; e94cb37b34eb8a8 drivers/gpu/drm/drm_dp_aux_dev.c Rafael Antognolli 2016-01-21 270 } e94cb37b34eb8a8 drivers/gpu/drm/drm_dp_aux_dev.c Rafael Antognolli 2016-01-21 271 :::::: The code at line 251 was first introduced by commit :::::: e94cb37b34eb8a88fe847438dba55c3f18bf024a drm/dp: Add a drm_aux-dev module for reading/writing dpcd registers. :::::: TO: Rafael Antognolli <[email protected]> :::::: CC: Daniel Vetter <[email protected]> -- 0-DAY CI Kernel Test Service https://01.org/lkp _______________________________________________ kbuild mailing list -- [email protected] To unsubscribe send an email to [email protected]
