From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 7AC91D69113 for ; Thu, 28 Nov 2024 15:06:07 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id B05A36B0082; Thu, 28 Nov 2024 10:06:06 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id A8E3D6B0085; Thu, 28 Nov 2024 10:06:06 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 908526B0088; Thu, 28 Nov 2024 10:06:06 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0012.hostedemail.com [216.40.44.12]) by kanga.kvack.org (Postfix) with ESMTP id 6DCC96B0082 for ; Thu, 28 Nov 2024 10:06:06 -0500 (EST) Received: from smtpin02.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay09.hostedemail.com (Postfix) with ESMTP id D749680391 for ; Thu, 28 Nov 2024 15:06:05 +0000 (UTC) X-FDA: 82835829048.02.760D985 Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.20]) by imf30.hostedemail.com (Postfix) with ESMTP id 9916480021 for ; Thu, 28 Nov 2024 15:05:49 +0000 (UTC) Authentication-Results: imf30.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b=UxI5s5lR; dmarc=pass (policy=none) header.from=intel.com; spf=pass (imf30.hostedemail.com: domain of lkp@intel.com designates 198.175.65.20 as permitted sender) smtp.mailfrom=lkp@intel.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1732806360; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=tO1EFpTtOGUrCZ+p7lPZV9Q7NuJra9BWaok9AStly7s=; b=6HFEYmK0ChggYRMQjgBn+wkCmGb58kCjRLdHvqRWLRgcdRuOHnJa6QGpYv98IsRWADEE9W IHUrDSvzA9+xaLv2AMwaPg3grkdVEFNVoKBviCiP23vWvIehyu1OD5My2CBDsfJcZXRZQy ulmh+vuZDkPfn7QnLpq1beB+T3QjOXQ= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1732806360; a=rsa-sha256; cv=none; b=Cnr+FaNwVvlIy0PsHCEptD6n7oMi/wShJfECyAuNKh6QQWFPYc0Kq1ihA0OP+D0M2N/OSW IoXyk+0r4wEaLiY+J9N68/01bHk/OKE7aH0LqKRojK4EJqAALJRcBghY3Nt2uG4w+oRL+8 4EDYRqtHL8bD1MD9pZRDA5DyNPhaT9s= ARC-Authentication-Results: i=1; imf30.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b=UxI5s5lR; dmarc=pass (policy=none) header.from=intel.com; spf=pass (imf30.hostedemail.com: domain of lkp@intel.com designates 198.175.65.20 as permitted sender) smtp.mailfrom=lkp@intel.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1732806363; x=1764342363; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=Rt/78/cxh3j3w2s9KmYWZ56N6kK9udqbOk1XcHGYS8M=; b=UxI5s5lRSUwR4PMv4+A+lsqu+VClEasgde7/tw4hmaR5x1SqE8qOX6Zd U83+aUSIcewPtzXo9udv5q/fXx9UL8GidRWLIun+m37SoJhow3YR6cDqs p0bbCSUSuAJofY4gzWIxne5fUPVYRqKLfaQ5PF64c2EGiWq7YVnBIqcbX cy3xzkRoYQFb/gAFHXLj7MfQt9pVQ3CqSzQNIo1t2AjwkKLygeAVqUym/ ZnSbb7j8Y+aBEBOgsYFfrq2oAp4xGSaXUkN0ipudqcVCfNheM59hcwHEQ MMR5o+bA+GnlfELAQXAAwC0u5PGEWRiFtVfmioEdK6AvfVMrEOQqJJhU0 Q==; X-CSE-ConnectionGUID: KCiIvCctQrCK4OENltCjtg== X-CSE-MsgGUID: bKyMUfZdRAmunkdOAQfi3Q== X-IronPort-AV: E=McAfee;i="6700,10204,11270"; a="32788444" X-IronPort-AV: E=Sophos;i="6.12,192,1728975600"; d="scan'208";a="32788444" Received: from fmviesa003.fm.intel.com ([10.60.135.143]) by orvoesa112.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 28 Nov 2024 07:06:01 -0800 X-CSE-ConnectionGUID: LSAnbzK9TUSjtwBZ8I8p0A== X-CSE-MsgGUID: FR7LH26mRWe4M/jOu7VF4A== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.12,192,1728975600"; d="scan'208";a="96349291" Received: from lkp-server01.sh.intel.com (HELO 8122d2fc1967) ([10.239.97.150]) by fmviesa003.fm.intel.com with ESMTP; 28 Nov 2024 07:05:56 -0800 Received: from kbuild by 8122d2fc1967 with local (Exim 4.96) (envelope-from ) id 1tGg5a-0009gx-1K; Thu, 28 Nov 2024 15:05:54 +0000 Date: Thu, 28 Nov 2024 23:05:06 +0800 From: kernel test robot To: David Laight , Linus Torvalds Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev, LKML , 'Arnd Bergmann' , 'Jens Axboe' , 'Matthew Wilcox' , 'Christoph Hellwig' , 'Andrew Morton' , Linux Memory Management List , 'Andy Shevchenko' , 'Dan Carpenter' , "'Jason A . Donenfeld'" , "'pedro.falcato@gmail.com'" , 'Mateusz Guzik' , 'Lorenzo Stoakes' Subject: Re: [PATCH next 6/7] minmax.h: Simplify the variants of clamp() Message-ID: <202411282222.oF0B4110-lkp@intel.com> References: <8f69f4deac014f558bab186444bac2e8@AcuMS.aculab.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <8f69f4deac014f558bab186444bac2e8@AcuMS.aculab.com> X-Stat-Signature: xjhwhohutiinjjdapmjo7ui6hb9qp4a4 X-Rspamd-Queue-Id: 9916480021 X-Rspam-User: X-Rspamd-Server: rspam01 X-HE-Tag: 1732806349-744277 X-HE-Meta: U2FsdGVkX1//DVNbb4lERY1SQTk/oE5CjaC8yAhkEAwDEKVNwqHPuSbdASNgnh7mgcsoYwk+wcstfUQ0b5VpXzbajmiLfPYuVeUjuS8/EhahccJrJTqhBzsAlpUypQnhULYF2nvdWGE4E7BfV8KeSFKOUTRpSS4jvLj+ah3XvfeSw+p7iTwVRlU4FeNku7SXWVXL2aQ8b/hbK5w74jCO3m8sr2bJ3TbkOn88FARtpCQutfNvq8HWP5MCM16HGyfsRLgzRHQgXTQueBiWw8ZFVNdQa53sqXOsakAI8muLe0y33qLX4dfhEqhIWQDW+RvzYPXgy3cuS5axd3tEXGE5tturBMZVP/FRs/30ieO3DlwyDnF4vzzY/ZvgDecxSkO/Vhe3cIf49PAO0/rmalNl9+NWTax76SEsAFcznl8R/50wQzNEIpagWcbmCreimOiS+WODJb15MoHPSAZ/KpLAgSm6NDDS5X4Cjp7Yo5BIjdZ5mFHKREc5FuE/hWJRjNDKeWsVmGqg5CZCW2N1C7ueuEZgLbSGTeOhS+sI6pPUOKntLIuQEmXsFxiAwioRO1XLIY0oVeos5KLtGRB0qdg/z1bEqf4+Vad/RfGLyEby+GPjnjwkHqpBbUs/zxzlgt2euTqJpXrpTQ5ZOWqZWwQxtBDWopM9fJ1QeUft2m0RHUDoc5o+5D7M1wvZKw9kzjhAIUSFwNGKofzeJb6KDwmk2YjNAKV3EflkyeKWPKy9jd/HjKJnzrM08sf4kLKqDtbCDmlJbDWn0xf0Qo+yCmStYgAdLvJ2YB7vL9rSBWrqrtCxpTuGbyFMk04hy4TrQJr9vUsTjfDx4/MZNXhUNo5a4YPZIr6rGdqkSJYzCguTv+jwiuxBEBW57Z0L1YArOXUk+DnOmjuto+ygUq7sYOInaQF5auSm2kBX2TA7EybFcQ7ZwjhFz14QpVRIuE8RHQSn+shIJXhIwArwlIPzUSX oE98/gBj 2u1YQ5TYhjfio6z07LNjXx3Yzfx4HTgwVctowp4kxCwFHPHP8fQjhpLim00jqaScbkrkhYZcQF5NzR8+W16LoKCNzvdCeI3GeW35J+3qVDheE3y54tEuWqSc91iONj/NFD7xYsU+QsrYgzxJXGkgnFrMElLvWjaWjgNr3uI66di6uyT75u9iDoeg84Q7UeciZhQTAk/Hzng+sLOB/Yd51qPHYWI29HLECHxBXPMFOUVZ6aOj7fXjuEgsA43anFc1dFw23Xs1Lc+rAp71h1DtSVShL77jD05O04maM4497bn9AHLFBsnDgxBIQFvEoQxZD3Alo8y2ZT//qGigjhR1Jqep9QGGS8uAiiaqv24cSJGksoJXpQA5SOH2G2mX6S5PJdvTfoBauBAmExy8grbUKv3BK57JCw0GRiS/GEivPPKUgexRxvxigc10HPlRNRHxA1WRNvs5F+NwZiAvsyflfx39Rpw== X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: Hi David, kernel test robot noticed the following build warnings: [auto build test WARNING on next-20241121] url: https://github.com/intel-lab-lkp/linux/commits/David-Laight/minmax-h-Add-whitespace-around-operators-and-after-commas/20241121-152617 base: next-20241121 patch link: https://lore.kernel.org/r/8f69f4deac014f558bab186444bac2e8%40AcuMS.aculab.com patch subject: [PATCH next 6/7] minmax.h: Simplify the variants of clamp() config: s390-allmodconfig (https://download.01.org/0day-ci/archive/20241128/202411282222.oF0B4110-lkp@intel.com/config) compiler: clang version 20.0.0git (https://github.com/llvm/llvm-project 592c0fe55f6d9a811028b5f3507be91458ab2713) reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20241128/202411282222.oF0B4110-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 | Closes: https://lore.kernel.org/oe-kbuild-all/202411282222.oF0B4110-lkp@intel.com/ All warnings (new ones prefixed by >>): In file included from drivers/iio/magnetometer/yamaha-yas530.c:30: In file included from include/linux/i2c.h:13: In file included from include/linux/acpi.h:14: In file included from include/linux/device.h:32: In file included from include/linux/device/driver.h:21: In file included from include/linux/module.h:19: In file included from include/linux/elf.h:6: In file included from arch/s390/include/asm/elf.h:181: In file included from arch/s390/include/asm/mmu_context.h:11: In file included from arch/s390/include/asm/pgalloc.h:18: In file included from include/linux/mm.h:2223: include/linux/vmstat.h:504:43: warning: arithmetic between different enumeration types ('enum zone_stat_item' and 'enum numa_stat_item') [-Wenum-enum-conversion] 504 | return vmstat_text[NR_VM_ZONE_STAT_ITEMS + | ~~~~~~~~~~~~~~~~~~~~~ ^ 505 | item]; | ~~~~ include/linux/vmstat.h:511:43: warning: arithmetic between different enumeration types ('enum zone_stat_item' and 'enum numa_stat_item') [-Wenum-enum-conversion] 511 | return vmstat_text[NR_VM_ZONE_STAT_ITEMS + | ~~~~~~~~~~~~~~~~~~~~~ ^ 512 | NR_VM_NUMA_EVENT_ITEMS + | ~~~~~~~~~~~~~~~~~~~~~~ include/linux/vmstat.h:518:36: warning: arithmetic between different enumeration types ('enum node_stat_item' and 'enum lru_list') [-Wenum-enum-conversion] 518 | return node_stat_name(NR_LRU_BASE + lru) + 3; // skip "nr_" | ~~~~~~~~~~~ ^ ~~~ include/linux/vmstat.h:524:43: warning: arithmetic between different enumeration types ('enum zone_stat_item' and 'enum numa_stat_item') [-Wenum-enum-conversion] 524 | return vmstat_text[NR_VM_ZONE_STAT_ITEMS + | ~~~~~~~~~~~~~~~~~~~~~ ^ 525 | NR_VM_NUMA_EVENT_ITEMS + | ~~~~~~~~~~~~~~~~~~~~~~ >> drivers/iio/magnetometer/yamaha-yas530.c:414:20: warning: implicit conversion from 'unsigned long' to 'typeof (h[i])' (aka 'int') changes value from 18446744073709543424 to -8192 [-Wconstant-conversion] 414 | clamp_val(h[i], -BIT(13), BIT(13) - 1); | ~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~ include/linux/minmax.h:233:66: note: expanded from macro 'clamp_val' 233 | #define clamp_val(val, lo, hi) __careful_clamp(typeof(val), val, lo, hi) | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~ include/linux/minmax.h:197:26: note: expanded from macro '__careful_clamp' 197 | __clamp_once(type, val, lo, hi, __UNIQUE_ID(v_), __UNIQUE_ID(l_), __UNIQUE_ID(h_)) | ~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ include/linux/minmax.h:188:14: note: expanded from macro '__clamp_once' 188 | type ulo = (lo); \ | ~~~ ^~ 5 warnings generated. vim +414 drivers/iio/magnetometer/yamaha-yas530.c de8860b1ed4701 Linus Walleij 2020-12-24 358 65f79b50103067 Jakob Hauser 2022-08-13 359 /** 65f79b50103067 Jakob Hauser 2022-08-13 360 * yas537_measure() - Make a measure from the hardware 65f79b50103067 Jakob Hauser 2022-08-13 361 * @yas5xx: The device state 65f79b50103067 Jakob Hauser 2022-08-13 362 * @t: the raw temperature measurement 65f79b50103067 Jakob Hauser 2022-08-13 363 * @x: the raw x axis measurement 65f79b50103067 Jakob Hauser 2022-08-13 364 * @y1: the y1 axis measurement 65f79b50103067 Jakob Hauser 2022-08-13 365 * @y2: the y2 axis measurement 65f79b50103067 Jakob Hauser 2022-08-13 366 * @return: 0 on success or error code 65f79b50103067 Jakob Hauser 2022-08-13 367 */ 65f79b50103067 Jakob Hauser 2022-08-13 368 static int yas537_measure(struct yas5xx *yas5xx, u16 *t, u16 *x, u16 *y1, u16 *y2) de8860b1ed4701 Linus Walleij 2020-12-24 369 { 65f79b50103067 Jakob Hauser 2022-08-13 370 struct yas5xx_calibration *c = &yas5xx->calibration; 65f79b50103067 Jakob Hauser 2022-08-13 371 unsigned int busy; 65f79b50103067 Jakob Hauser 2022-08-13 372 u8 data[8]; 65f79b50103067 Jakob Hauser 2022-08-13 373 u16 xy1y2[3]; 65f79b50103067 Jakob Hauser 2022-08-13 374 s32 h[3], s[3]; 65f79b50103067 Jakob Hauser 2022-08-13 375 int i, ret; 65f79b50103067 Jakob Hauser 2022-08-13 376 65f79b50103067 Jakob Hauser 2022-08-13 377 mutex_lock(&yas5xx->lock); 65f79b50103067 Jakob Hauser 2022-08-13 378 65f79b50103067 Jakob Hauser 2022-08-13 379 /* Contrary to YAS530/532, also a "cont" bit is set, meaning unknown */ 65f79b50103067 Jakob Hauser 2022-08-13 380 ret = regmap_write(yas5xx->map, YAS537_MEASURE, YAS5XX_MEASURE_START | 65f79b50103067 Jakob Hauser 2022-08-13 381 YAS5XX_MEASURE_CONT); 65f79b50103067 Jakob Hauser 2022-08-13 382 if (ret < 0) 65f79b50103067 Jakob Hauser 2022-08-13 383 goto out_unlock; 65f79b50103067 Jakob Hauser 2022-08-13 384 65f79b50103067 Jakob Hauser 2022-08-13 385 /* Use same timeout like YAS530/532 but the bit is in data row 2 */ 65f79b50103067 Jakob Hauser 2022-08-13 386 ret = regmap_read_poll_timeout(yas5xx->map, YAS5XX_MEASURE_DATA + 2, busy, 65f79b50103067 Jakob Hauser 2022-08-13 387 !(busy & YAS5XX_MEASURE_DATA_BUSY), 65f79b50103067 Jakob Hauser 2022-08-13 388 500, 20000); 65f79b50103067 Jakob Hauser 2022-08-13 389 if (ret) { 65f79b50103067 Jakob Hauser 2022-08-13 390 dev_err(yas5xx->dev, "timeout waiting for measurement\n"); 65f79b50103067 Jakob Hauser 2022-08-13 391 goto out_unlock; 65f79b50103067 Jakob Hauser 2022-08-13 392 } 65f79b50103067 Jakob Hauser 2022-08-13 393 65f79b50103067 Jakob Hauser 2022-08-13 394 ret = regmap_bulk_read(yas5xx->map, YAS5XX_MEASURE_DATA, 65f79b50103067 Jakob Hauser 2022-08-13 395 data, sizeof(data)); 65f79b50103067 Jakob Hauser 2022-08-13 396 if (ret) 65f79b50103067 Jakob Hauser 2022-08-13 397 goto out_unlock; 65f79b50103067 Jakob Hauser 2022-08-13 398 65f79b50103067 Jakob Hauser 2022-08-13 399 mutex_unlock(&yas5xx->lock); 65f79b50103067 Jakob Hauser 2022-08-13 400 65f79b50103067 Jakob Hauser 2022-08-13 401 *t = get_unaligned_be16(&data[0]); 65f79b50103067 Jakob Hauser 2022-08-13 402 xy1y2[0] = FIELD_GET(GENMASK(13, 0), get_unaligned_be16(&data[2])); 65f79b50103067 Jakob Hauser 2022-08-13 403 xy1y2[1] = get_unaligned_be16(&data[4]); 65f79b50103067 Jakob Hauser 2022-08-13 404 xy1y2[2] = get_unaligned_be16(&data[6]); 65f79b50103067 Jakob Hauser 2022-08-13 405 65f79b50103067 Jakob Hauser 2022-08-13 406 /* The second version of YAS537 needs to include calibration coefficients */ 65f79b50103067 Jakob Hauser 2022-08-13 407 if (yas5xx->version == YAS537_VERSION_1) { 65f79b50103067 Jakob Hauser 2022-08-13 408 for (i = 0; i < 3; i++) 65f79b50103067 Jakob Hauser 2022-08-13 409 s[i] = xy1y2[i] - BIT(13); 65f79b50103067 Jakob Hauser 2022-08-13 410 h[0] = (c->k * (128 * s[0] + c->a2 * s[1] + c->a3 * s[2])) / BIT(13); 65f79b50103067 Jakob Hauser 2022-08-13 411 h[1] = (c->k * (c->a4 * s[0] + c->a5 * s[1] + c->a6 * s[2])) / BIT(13); 65f79b50103067 Jakob Hauser 2022-08-13 412 h[2] = (c->k * (c->a7 * s[0] + c->a8 * s[1] + c->a9 * s[2])) / BIT(13); 65f79b50103067 Jakob Hauser 2022-08-13 413 for (i = 0; i < 3; i++) { 65f79b50103067 Jakob Hauser 2022-08-13 @414 clamp_val(h[i], -BIT(13), BIT(13) - 1); 65f79b50103067 Jakob Hauser 2022-08-13 415 xy1y2[i] = h[i] + BIT(13); 65f79b50103067 Jakob Hauser 2022-08-13 416 } 65f79b50103067 Jakob Hauser 2022-08-13 417 } 65f79b50103067 Jakob Hauser 2022-08-13 418 65f79b50103067 Jakob Hauser 2022-08-13 419 *x = xy1y2[0]; 65f79b50103067 Jakob Hauser 2022-08-13 420 *y1 = xy1y2[1]; 65f79b50103067 Jakob Hauser 2022-08-13 421 *y2 = xy1y2[2]; 65f79b50103067 Jakob Hauser 2022-08-13 422 65f79b50103067 Jakob Hauser 2022-08-13 423 return 0; 65f79b50103067 Jakob Hauser 2022-08-13 424 65f79b50103067 Jakob Hauser 2022-08-13 425 out_unlock: 65f79b50103067 Jakob Hauser 2022-08-13 426 mutex_unlock(&yas5xx->lock); 65f79b50103067 Jakob Hauser 2022-08-13 427 return ret; 65f79b50103067 Jakob Hauser 2022-08-13 428 } 65f79b50103067 Jakob Hauser 2022-08-13 429 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki