Hi Hsin-Yi, Thank you for the patch! Yet something to improve: [auto build test ERROR on wsa/i2c/for-next] [also build test ERROR on robh/for-next mediatek/for-next v5.13-rc3 next-20210527] [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/Hsin-Yi-Wang/add-power-control-in-i2c/20210527-155826 base: https://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux.git i2c/for-next config: x86_64-randconfig-a011-20210526 (attached as .config) compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 6505c630407c5feec818f0bb1c284f9eeebf2071) 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 x86_64 cross compiling tool for clang build # apt-get install binutils-x86-64-linux-gnu # https://github.com/0day-ci/linux/commit/e76ae7334b07a6721160299cf30e55b200bb5466 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Hsin-Yi-Wang/add-power-control-in-i2c/20210527-155826 git checkout e76ae7334b07a6721160299cf30e55b200bb5466 # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot All errors (new ones prefixed by >>): In file included from drivers/gpu/drm/i915/i915_gem.c:1250: >> drivers/gpu/drm/i915/selftests/i915_gem.c:97:13: error: static declaration of 'pm_suspend' follows non-static declaration static void pm_suspend(struct drm_i915_private *i915) ^ include/linux/suspend.h:331:12: note: previous declaration is here extern int pm_suspend(suspend_state_t state); ^ In file included from drivers/gpu/drm/i915/i915_gem.c:1250: drivers/gpu/drm/i915/selftests/i915_gem.c:155:13: error: incompatible pointer to integer conversion passing 'struct drm_i915_private *' to parameter of type 'suspend_state_t' (aka 'int') [-Werror,-Wint-conversion] pm_suspend(i915); ^~~~ include/linux/suspend.h:331:39: note: passing argument to parameter 'state' here extern int pm_suspend(suspend_state_t state); ^ 2 errors generated. vim +/pm_suspend +97 drivers/gpu/drm/i915/selftests/i915_gem.c 3f51b7e1f36a37 Chris Wilson 2018-08-30 96 3f51b7e1f36a37 Chris Wilson 2018-08-30 @97 static void pm_suspend(struct drm_i915_private *i915) 3f51b7e1f36a37 Chris Wilson 2018-08-30 98 { c9d08cc3e3393e Chris Wilson 2019-01-14 99 intel_wakeref_t wakeref; c9d08cc3e3393e Chris Wilson 2019-01-14 100 c447ff7db34807 Daniele Ceraolo Spurio 2019-06-13 101 with_intel_runtime_pm(&i915->runtime_pm, wakeref) { e986209c67024c Chris Wilson 2020-01-30 102 i915_ggtt_suspend(&i915->ggtt); 3f51b7e1f36a37 Chris Wilson 2018-08-30 103 i915_gem_suspend_late(i915); d4225a535b3b08 Chris Wilson 2019-01-14 104 } 3f51b7e1f36a37 Chris Wilson 2018-08-30 105 } 3f51b7e1f36a37 Chris Wilson 2018-08-30 106 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org