[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <202510161626.f3OG4u62-lkp@intel.com>
Date: Thu, 16 Oct 2025 16:29:00 +0800
From: kernel test robot <lkp@...el.com>
To: Karunika Choo <karunika.choo@....com>, dri-devel@...ts.freedesktop.org
Cc: oe-kbuild-all@...ts.linux.dev, nd@....com,
Boris Brezillon <bbrezillon@...nel.org>,
Steven Price <steven.price@....com>,
Liviu Dudau <liviu.dudau@....com>,
Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Maxime Ripard <mripard@...nel.org>,
Thomas Zimmermann <tzimmermann@...e.de>,
David Airlie <airlied@...il.com>, Simona Vetter <simona@...ll.ch>,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH v1 06/10] drm/panthor: Implement L2 power on/off via
PWR_CONTROL
Hi Karunika,
kernel test robot noticed the following build warnings:
[auto build test WARNING on drm-misc/drm-misc-next]
[also build test WARNING on drm/drm-next drm-exynos/exynos-drm-next drm-intel/for-linux-next drm-intel/for-linux-next-fixes drm-tip/drm-tip linus/master v6.18-rc1 next-20251015]
[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/Karunika-Choo/drm-panthor-Factor-out-GPU_ID-register-read-into-separate-function/20251014-174729
base: git://anongit.freedesktop.org/drm/drm-misc drm-misc-next
patch link: https://lore.kernel.org/r/20251014094337.1009601-7-karunika.choo%40arm.com
patch subject: [PATCH v1 06/10] drm/panthor: Implement L2 power on/off via PWR_CONTROL
config: arm-randconfig-r132-20251016 (https://download.01.org/0day-ci/archive/20251016/202510161626.f3OG4u62-lkp@intel.com/config)
compiler: clang version 22.0.0git (https://github.com/llvm/llvm-project 39f292ffa13d7ca0d1edff27ac8fd55024bb4d19)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20251016/202510161626.f3OG4u62-lkp@intel.com/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 <lkp@...el.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202510161626.f3OG4u62-lkp@intel.com/
All warnings (new ones prefixed by >>):
>> drivers/gpu/drm/panthor/panthor_pwr.c:153:13: warning: shift count >= width of type [-Wshift-count-overflow]
152 | ret = gpu_read64_poll_timeout(ptdev, pwrtrans_reg, val,
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
153 | !(PWR_ALL_CORES_MASK & val), 100,
| ~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
154 | timeout_us);
| ~~~~~~~~~~~
drivers/gpu/drm/panthor/panthor_pwr.c:26:29: note: expanded from macro 'PWR_ALL_CORES_MASK'
26 | #define PWR_ALL_CORES_MASK GENMASK(63, 0)
| ^
include/linux/bits.h:51:24: note: expanded from macro 'GENMASK'
51 | #define GENMASK(h, l) GENMASK_TYPE(unsigned long, h, l)
| ^
include/linux/bits.h:49:20: note: expanded from macro 'GENMASK_TYPE'
49 | type_max(t) >> (BITS_PER_TYPE(t) - 1 - (h)))))
| ^
drivers/gpu/drm/panthor/panthor_device.h:533:37: note: expanded from macro 'gpu_read64_poll_timeout'
533 | read_poll_timeout(gpu_read64, val, cond, delay_us, timeout_us, false, \
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
534 | dev, reg)
| ~~~~~~~~~
include/linux/iopoll.h:145:36: note: expanded from macro 'read_poll_timeout'
145 | poll_timeout_us((val) = op(args), cond, sleep_us, timeout_us, sleep_before_read)
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/iopoll.h:49:7: note: expanded from macro 'poll_timeout_us'
49 | if (cond) { \
| ^~~~
drivers/gpu/drm/panthor/panthor_pwr.c:260:13: warning: shift count >= width of type [-Wshift-count-overflow]
259 | ret = gpu_read64_poll_timeout(ptdev, PWR_STATUS, val,
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
260 | !(PWR_STATUS_RETRACT_PENDING & val), 0,
| ~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
261 | PWR_RETRACT_TIMEOUT_US);
| ~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/panthor/panthor_regs.h:249:40: note: expanded from macro 'PWR_STATUS_RETRACT_PENDING'
249 | #define PWR_STATUS_RETRACT_PENDING BIT(43)
| ^
include/vdso/bits.h:7:26: note: expanded from macro 'BIT'
7 | #define BIT(nr) (UL(1) << (nr))
| ^
drivers/gpu/drm/panthor/panthor_device.h:533:37: note: expanded from macro 'gpu_read64_poll_timeout'
533 | read_poll_timeout(gpu_read64, val, cond, delay_us, timeout_us, false, \
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
534 | dev, reg)
| ~~~~~~~~~
include/linux/iopoll.h:145:36: note: expanded from macro 'read_poll_timeout'
145 | poll_timeout_us((val) = op(args), cond, sleep_us, timeout_us, sleep_before_read)
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/iopoll.h:49:7: note: expanded from macro 'poll_timeout_us'
49 | if (cond) { \
| ^~~~
2 warnings generated.
vim +153 drivers/gpu/drm/panthor/panthor_pwr.c
144
145 static int panthor_pwr_domain_wait_transition(struct panthor_device *ptdev, u32 domain,
146 u32 timeout_us)
147 {
148 u32 pwrtrans_reg = get_domain_pwrtrans_reg(domain);
149 u64 val;
150 int ret = 0;
151
152 ret = gpu_read64_poll_timeout(ptdev, pwrtrans_reg, val,
> 153 !(PWR_ALL_CORES_MASK & val), 100,
154 timeout_us);
155 if (ret) {
156 drm_err(&ptdev->base, "%s domain power in transition, pwrtrans(0x%llx)",
157 get_domain_name(domain), val);
158 return ret;
159 }
160
161 return 0;
162 }
163
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
Powered by blists - more mailing lists