* Re: [PATCH 09/10] mmc: core: Add Ext memory addressing for erase
[not found] <20240728081154.1782120-10-avri.altman@wdc.com>
@ 2024-07-28 10:42 ` kernel test robot
0 siblings, 0 replies; only message in thread
From: kernel test robot @ 2024-07-28 10:42 UTC (permalink / raw)
To: Avri Altman, Ulf Hansson, linux-mmc
Cc: llvm, oe-kbuild-all, Ricky WU, Avri Altman
Hi Avri,
kernel test robot noticed the following build warnings:
[auto build test WARNING on linus/master]
[also build test WARNING on v6.10 next-20240726]
[cannot apply to ulf-hansson-mmc-mirror/next]
[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/Avri-Altman/mmc-sd-SDUC-Support-Recognition/20240728-161645
base: linus/master
patch link: https://lore.kernel.org/r/20240728081154.1782120-10-avri.altman%40wdc.com
patch subject: [PATCH 09/10] mmc: core: Add Ext memory addressing for erase
config: i386-buildonly-randconfig-001-20240728 (https://download.01.org/0day-ci/archive/20240728/202407281841.RRgIK067-lkp@intel.com/config)
compiler: clang version 18.1.5 (https://github.com/llvm/llvm-project 617a15a9eac96088ae5e9134248d8236e34b91b1)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240728/202407281841.RRgIK067-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@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202407281841.RRgIK067-lkp@intel.com/
All warnings (new ones prefixed by >>):
>> drivers/mmc/core/core.c:1667:27: warning: shift count >= width of type [-Wshift-count-overflow]
1667 | u8 ext_addr = FIELD_GET(SDUC_ADDR_EXT_MASK, from);
| ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/mmc/core/sd_ops.h:14:28: note: expanded from macro 'SDUC_ADDR_EXT_MASK'
14 | #define SDUC_ADDR_EXT_MASK GENMASK(37, 32)
| ^
include/linux/bits.h:35:31: note: expanded from macro 'GENMASK'
35 | (GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
| ^
include/uapi/linux/bits.h:8:31: note: expanded from macro '__GENMASK'
8 | (((~_UL(0)) - (_UL(1) << (l)) + 1) & \
| ^
note: (skipping 3 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)
include/linux/compiler_types.h:510:22: note: expanded from macro 'compiletime_assert'
510 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
| ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:498:23: note: expanded from macro '_compiletime_assert'
498 | __compiletime_assert(condition, msg, prefix, suffix)
| ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:490:9: note: expanded from macro '__compiletime_assert'
490 | if (!(condition)) \
| ^~~~~~~~~
>> drivers/mmc/core/core.c:1667:27: warning: shift count is negative [-Wshift-count-negative]
1667 | u8 ext_addr = FIELD_GET(SDUC_ADDR_EXT_MASK, from);
| ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/mmc/core/sd_ops.h:14:28: note: expanded from macro 'SDUC_ADDR_EXT_MASK'
14 | #define SDUC_ADDR_EXT_MASK GENMASK(37, 32)
| ^
include/linux/bits.h:35:31: note: expanded from macro 'GENMASK'
35 | (GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
| ^
include/uapi/linux/bits.h:9:19: note: expanded from macro '__GENMASK'
9 | (~_UL(0) >> (__BITS_PER_LONG - 1 - (h))))
| ^
note: (skipping 3 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)
include/linux/compiler_types.h:510:22: note: expanded from macro 'compiletime_assert'
510 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
| ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:498:23: note: expanded from macro '_compiletime_assert'
498 | __compiletime_assert(condition, msg, prefix, suffix)
| ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:490:9: note: expanded from macro '__compiletime_assert'
490 | if (!(condition)) \
| ^~~~~~~~~
>> drivers/mmc/core/core.c:1667:27: warning: shift count >= width of type [-Wshift-count-overflow]
1667 | u8 ext_addr = FIELD_GET(SDUC_ADDR_EXT_MASK, from);
| ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/mmc/core/sd_ops.h:14:28: note: expanded from macro 'SDUC_ADDR_EXT_MASK'
14 | #define SDUC_ADDR_EXT_MASK GENMASK(37, 32)
| ^
include/linux/bits.h:35:31: note: expanded from macro 'GENMASK'
35 | (GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
| ^
include/uapi/linux/bits.h:8:31: note: expanded from macro '__GENMASK'
8 | (((~_UL(0)) - (_UL(1) << (l)) + 1) & \
| ^
note: (skipping 3 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)
include/linux/compiler_types.h:510:22: note: expanded from macro 'compiletime_assert'
510 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
| ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:498:23: note: expanded from macro '_compiletime_assert'
498 | __compiletime_assert(condition, msg, prefix, suffix)
| ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:490:9: note: expanded from macro '__compiletime_assert'
490 | if (!(condition)) \
| ^~~~~~~~~
>> drivers/mmc/core/core.c:1667:27: warning: shift count is negative [-Wshift-count-negative]
1667 | u8 ext_addr = FIELD_GET(SDUC_ADDR_EXT_MASK, from);
| ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/mmc/core/sd_ops.h:14:28: note: expanded from macro 'SDUC_ADDR_EXT_MASK'
14 | #define SDUC_ADDR_EXT_MASK GENMASK(37, 32)
| ^
include/linux/bits.h:35:31: note: expanded from macro 'GENMASK'
35 | (GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
| ^
include/uapi/linux/bits.h:9:19: note: expanded from macro '__GENMASK'
9 | (~_UL(0) >> (__BITS_PER_LONG - 1 - (h))))
| ^
note: (skipping 3 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)
include/linux/compiler_types.h:510:22: note: expanded from macro 'compiletime_assert'
510 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
| ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:498:23: note: expanded from macro '_compiletime_assert'
498 | __compiletime_assert(condition, msg, prefix, suffix)
| ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:490:9: note: expanded from macro '__compiletime_assert'
490 | if (!(condition)) \
| ^~~~~~~~~
>> drivers/mmc/core/core.c:1667:27: warning: shift count >= width of type [-Wshift-count-overflow]
1667 | u8 ext_addr = FIELD_GET(SDUC_ADDR_EXT_MASK, from);
| ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/mmc/core/sd_ops.h:14:28: note: expanded from macro 'SDUC_ADDR_EXT_MASK'
14 | #define SDUC_ADDR_EXT_MASK GENMASK(37, 32)
| ^
include/linux/bits.h:35:31: note: expanded from macro 'GENMASK'
35 | (GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
| ^
include/uapi/linux/bits.h:8:31: note: expanded from macro '__GENMASK'
8 | (((~_UL(0)) - (_UL(1) << (l)) + 1) & \
| ^
note: (skipping 3 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)
include/linux/compiler_types.h:510:22: note: expanded from macro 'compiletime_assert'
510 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
| ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:498:23: note: expanded from macro '_compiletime_assert'
498 | __compiletime_assert(condition, msg, prefix, suffix)
| ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:490:9: note: expanded from macro '__compiletime_assert'
490 | if (!(condition)) \
| ^~~~~~~~~
>> drivers/mmc/core/core.c:1667:27: warning: shift count is negative [-Wshift-count-negative]
1667 | u8 ext_addr = FIELD_GET(SDUC_ADDR_EXT_MASK, from);
| ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/mmc/core/sd_ops.h:14:28: note: expanded from macro 'SDUC_ADDR_EXT_MASK'
14 | #define SDUC_ADDR_EXT_MASK GENMASK(37, 32)
| ^
include/linux/bits.h:35:31: note: expanded from macro 'GENMASK'
35 | (GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
| ^
include/uapi/linux/bits.h:9:19: note: expanded from macro '__GENMASK'
9 | (~_UL(0) >> (__BITS_PER_LONG - 1 - (h))))
| ^
note: (skipping 3 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)
include/linux/compiler_types.h:510:22: note: expanded from macro 'compiletime_assert'
510 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
| ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:498:23: note: expanded from macro '_compiletime_assert'
498 | __compiletime_assert(condition, msg, prefix, suffix)
| ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:490:9: note: expanded from macro '__compiletime_assert'
490 | if (!(condition)) \
| ^~~~~~~~~
>> drivers/mmc/core/core.c:1667:27: warning: shift count >= width of type [-Wshift-count-overflow]
1667 | u8 ext_addr = FIELD_GET(SDUC_ADDR_EXT_MASK, from);
| ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/mmc/core/sd_ops.h:14:28: note: expanded from macro 'SDUC_ADDR_EXT_MASK'
14 | #define SDUC_ADDR_EXT_MASK GENMASK(37, 32)
| ^
include/linux/bits.h:35:31: note: expanded from macro 'GENMASK'
35 | (GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
| ^
include/uapi/linux/bits.h:8:31: note: expanded from macro '__GENMASK'
8 | (((~_UL(0)) - (_UL(1) << (l)) + 1) & \
| ^
note: (skipping 4 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)
include/linux/compiler_types.h:510:22: note: expanded from macro 'compiletime_assert'
510 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
| ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:498:23: note: expanded from macro '_compiletime_assert'
498 | __compiletime_assert(condition, msg, prefix, suffix)
| ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:490:9: note: expanded from macro '__compiletime_assert'
490 | if (!(condition)) \
| ^~~~~~~~~
>> drivers/mmc/core/core.c:1667:27: warning: shift count is negative [-Wshift-count-negative]
1667 | u8 ext_addr = FIELD_GET(SDUC_ADDR_EXT_MASK, from);
| ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/mmc/core/sd_ops.h:14:28: note: expanded from macro 'SDUC_ADDR_EXT_MASK'
14 | #define SDUC_ADDR_EXT_MASK GENMASK(37, 32)
| ^
include/linux/bits.h:35:31: note: expanded from macro 'GENMASK'
35 | (GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
| ^
include/uapi/linux/bits.h:9:19: note: expanded from macro '__GENMASK'
9 | (~_UL(0) >> (__BITS_PER_LONG - 1 - (h))))
| ^
note: (skipping 4 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)
include/linux/compiler_types.h:510:22: note: expanded from macro 'compiletime_assert'
510 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
| ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:498:23: note: expanded from macro '_compiletime_assert'
498 | __compiletime_assert(condition, msg, prefix, suffix)
| ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:490:9: note: expanded from macro '__compiletime_assert'
490 | if (!(condition)) \
| ^~~~~~~~~
>> drivers/mmc/core/core.c:1667:27: warning: shift count >= width of type [-Wshift-count-overflow]
1667 | u8 ext_addr = FIELD_GET(SDUC_ADDR_EXT_MASK, from);
| ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/mmc/core/sd_ops.h:14:28: note: expanded from macro 'SDUC_ADDR_EXT_MASK'
14 | #define SDUC_ADDR_EXT_MASK GENMASK(37, 32)
| ^
include/linux/bits.h:35:31: note: expanded from macro 'GENMASK'
35 | (GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
| ^
include/uapi/linux/bits.h:8:31: note: expanded from macro '__GENMASK'
8 | (((~_UL(0)) - (_UL(1) << (l)) + 1) & \
| ^
note: (skipping 4 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)
include/linux/compiler_types.h:510:22: note: expanded from macro 'compiletime_assert'
510 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
| ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:498:23: note: expanded from macro '_compiletime_assert'
498 | __compiletime_assert(condition, msg, prefix, suffix)
| ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:490:9: note: expanded from macro '__compiletime_assert'
490 | if (!(condition)) \
| ^~~~~~~~~
>> drivers/mmc/core/core.c:1667:27: warning: shift count is negative [-Wshift-count-negative]
1667 | u8 ext_addr = FIELD_GET(SDUC_ADDR_EXT_MASK, from);
| ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/mmc/core/sd_ops.h:14:28: note: expanded from macro 'SDUC_ADDR_EXT_MASK'
14 | #define SDUC_ADDR_EXT_MASK GENMASK(37, 32)
| ^
include/linux/bits.h:35:31: note: expanded from macro 'GENMASK'
35 | (GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
| ^
include/uapi/linux/bits.h:9:19: note: expanded from macro '__GENMASK'
9 | (~_UL(0) >> (__BITS_PER_LONG - 1 - (h))))
| ^
note: (skipping 4 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)
include/linux/compiler_types.h:510:22: note: expanded from macro 'compiletime_assert'
510 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
| ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:498:23: note: expanded from macro '_compiletime_assert'
498 | __compiletime_assert(condition, msg, prefix, suffix)
| ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:490:9: note: expanded from macro '__compiletime_assert'
490 | if (!(condition)) \
| ^~~~~~~~~
>> drivers/mmc/core/core.c:1667:27: warning: shift count >= width of type [-Wshift-count-overflow]
1667 | u8 ext_addr = FIELD_GET(SDUC_ADDR_EXT_MASK, from);
| ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/mmc/core/sd_ops.h:14:28: note: expanded from macro 'SDUC_ADDR_EXT_MASK'
14 | #define SDUC_ADDR_EXT_MASK GENMASK(37, 32)
| ^
include/linux/bits.h:35:31: note: expanded from macro 'GENMASK'
35 | (GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
| ^
include/uapi/linux/bits.h:8:31: note: expanded from macro '__GENMASK'
8 | (((~_UL(0)) - (_UL(1) << (l)) + 1) & \
| ^
note: (skipping 5 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)
include/linux/compiler_types.h:510:22: note: expanded from macro 'compiletime_assert'
510 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
| ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:498:23: note: expanded from macro '_compiletime_assert'
498 | __compiletime_assert(condition, msg, prefix, suffix)
| ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:490:9: note: expanded from macro '__compiletime_assert'
490 | if (!(condition)) \
| ^~~~~~~~~
>> drivers/mmc/core/core.c:1667:27: warning: shift count is negative [-Wshift-count-negative]
1667 | u8 ext_addr = FIELD_GET(SDUC_ADDR_EXT_MASK, from);
| ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/mmc/core/sd_ops.h:14:28: note: expanded from macro 'SDUC_ADDR_EXT_MASK'
14 | #define SDUC_ADDR_EXT_MASK GENMASK(37, 32)
| ^
include/linux/bits.h:35:31: note: expanded from macro 'GENMASK'
35 | (GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
| ^
include/uapi/linux/bits.h:9:19: note: expanded from macro '__GENMASK'
9 | (~_UL(0) >> (__BITS_PER_LONG - 1 - (h))))
| ^
note: (skipping 5 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)
include/linux/compiler_types.h:510:22: note: expanded from macro 'compiletime_assert'
510 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
| ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:498:23: note: expanded from macro '_compiletime_assert'
498 | __compiletime_assert(condition, msg, prefix, suffix)
| ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:490:9: note: expanded from macro '__compiletime_assert'
490 | if (!(condition)) \
| ^~~~~~~~~
>> drivers/mmc/core/core.c:1667:27: warning: shift count >= width of type [-Wshift-count-overflow]
1667 | u8 ext_addr = FIELD_GET(SDUC_ADDR_EXT_MASK, from);
| ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/mmc/core/sd_ops.h:14:28: note: expanded from macro 'SDUC_ADDR_EXT_MASK'
14 | #define SDUC_ADDR_EXT_MASK GENMASK(37, 32)
| ^
include/linux/bits.h:35:31: note: expanded from macro 'GENMASK'
35 | (GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
| ^
include/uapi/linux/bits.h:8:31: note: expanded from macro '__GENMASK'
8 | (((~_UL(0)) - (_UL(1) << (l)) + 1) & \
| ^
note: (skipping 6 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)
include/linux/compiler_types.h:510:22: note: expanded from macro 'compiletime_assert'
510 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
| ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:498:23: note: expanded from macro '_compiletime_assert'
498 | __compiletime_assert(condition, msg, prefix, suffix)
| ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:490:9: note: expanded from macro '__compiletime_assert'
490 | if (!(condition)) \
| ^~~~~~~~~
>> drivers/mmc/core/core.c:1667:27: warning: shift count is negative [-Wshift-count-negative]
1667 | u8 ext_addr = FIELD_GET(SDUC_ADDR_EXT_MASK, from);
| ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/mmc/core/sd_ops.h:14:28: note: expanded from macro 'SDUC_ADDR_EXT_MASK'
14 | #define SDUC_ADDR_EXT_MASK GENMASK(37, 32)
| ^
include/linux/bits.h:35:31: note: expanded from macro 'GENMASK'
35 | (GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
| ^
include/uapi/linux/bits.h:9:19: note: expanded from macro '__GENMASK'
9 | (~_UL(0) >> (__BITS_PER_LONG - 1 - (h))))
| ^
note: (skipping 6 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)
include/linux/compiler_types.h:510:22: note: expanded from macro 'compiletime_assert'
510 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
| ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:498:23: note: expanded from macro '_compiletime_assert'
498 | __compiletime_assert(condition, msg, prefix, suffix)
| ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:490:9: note: expanded from macro '__compiletime_assert'
490 | if (!(condition)) \
| ^~~~~~~~~
>> drivers/mmc/core/core.c:1667:27: warning: shift count >= width of type [-Wshift-count-overflow]
1667 | u8 ext_addr = FIELD_GET(SDUC_ADDR_EXT_MASK, from);
| ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/mmc/core/sd_ops.h:14:28: note: expanded from macro 'SDUC_ADDR_EXT_MASK'
14 | #define SDUC_ADDR_EXT_MASK GENMASK(37, 32)
| ^
include/linux/bits.h:35:31: note: expanded from macro 'GENMASK'
35 | (GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
| ^
include/uapi/linux/bits.h:8:31: note: expanded from macro '__GENMASK'
8 | (((~_UL(0)) - (_UL(1) << (l)) + 1) & \
| ^
note: (skipping 5 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)
include/linux/compiler_types.h:510:22: note: expanded from macro 'compiletime_assert'
510 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
| ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:498:23: note: expanded from macro '_compiletime_assert'
498 | __compiletime_assert(condition, msg, prefix, suffix)
| ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:490:9: note: expanded from macro '__compiletime_assert'
490 | if (!(condition)) \
| ^~~~~~~~~
>> drivers/mmc/core/core.c:1667:27: warning: shift count is negative [-Wshift-count-negative]
1667 | u8 ext_addr = FIELD_GET(SDUC_ADDR_EXT_MASK, from);
| ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/mmc/core/sd_ops.h:14:28: note: expanded from macro 'SDUC_ADDR_EXT_MASK'
14 | #define SDUC_ADDR_EXT_MASK GENMASK(37, 32)
| ^
include/linux/bits.h:35:31: note: expanded from macro 'GENMASK'
35 | (GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
| ^
include/uapi/linux/bits.h:9:19: note: expanded from macro '__GENMASK'
9 | (~_UL(0) >> (__BITS_PER_LONG - 1 - (h))))
| ^
note: (skipping 5 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)
include/linux/compiler_types.h:510:22: note: expanded from macro 'compiletime_assert'
510 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
| ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:498:23: note: expanded from macro '_compiletime_assert'
498 | __compiletime_assert(condition, msg, prefix, suffix)
| ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:490:9: note: expanded from macro '__compiletime_assert'
490 | if (!(condition)) \
| ^~~~~~~~~
>> drivers/mmc/core/core.c:1667:27: warning: shift count >= width of type [-Wshift-count-overflow]
1667 | u8 ext_addr = FIELD_GET(SDUC_ADDR_EXT_MASK, from);
| ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/mmc/core/sd_ops.h:14:28: note: expanded from macro 'SDUC_ADDR_EXT_MASK'
14 | #define SDUC_ADDR_EXT_MASK GENMASK(37, 32)
| ^
include/linux/bits.h:35:31: note: expanded from macro 'GENMASK'
35 | (GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
| ^
include/uapi/linux/bits.h:8:31: note: expanded from macro '__GENMASK'
8 | (((~_UL(0)) - (_UL(1) << (l)) + 1) & \
| ^
note: (skipping 6 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)
include/linux/compiler_types.h:510:22: note: expanded from macro 'compiletime_assert'
510 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
| ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:498:23: note: expanded from macro '_compiletime_assert'
498 | __compiletime_assert(condition, msg, prefix, suffix)
| ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:490:9: note: expanded from macro '__compiletime_assert'
490 | if (!(condition)) \
| ^~~~~~~~~
>> drivers/mmc/core/core.c:1667:27: warning: shift count is negative [-Wshift-count-negative]
1667 | u8 ext_addr = FIELD_GET(SDUC_ADDR_EXT_MASK, from);
| ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/mmc/core/sd_ops.h:14:28: note: expanded from macro 'SDUC_ADDR_EXT_MASK'
14 | #define SDUC_ADDR_EXT_MASK GENMASK(37, 32)
| ^
include/linux/bits.h:35:31: note: expanded from macro 'GENMASK'
35 | (GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
| ^
include/uapi/linux/bits.h:9:19: note: expanded from macro '__GENMASK'
9 | (~_UL(0) >> (__BITS_PER_LONG - 1 - (h))))
| ^
note: (skipping 6 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)
include/linux/compiler_types.h:510:22: note: expanded from macro 'compiletime_assert'
510 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
| ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:498:23: note: expanded from macro '_compiletime_assert'
498 | __compiletime_assert(condition, msg, prefix, suffix)
| ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:490:9: note: expanded from macro '__compiletime_assert'
490 | if (!(condition)) \
| ^~~~~~~~~
>> drivers/mmc/core/core.c:1667:27: warning: shift count >= width of type [-Wshift-count-overflow]
1667 | u8 ext_addr = FIELD_GET(SDUC_ADDR_EXT_MASK, from);
| ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/mmc/core/sd_ops.h:14:28: note: expanded from macro 'SDUC_ADDR_EXT_MASK'
14 | #define SDUC_ADDR_EXT_MASK GENMASK(37, 32)
| ^
include/linux/bits.h:35:31: note: expanded from macro 'GENMASK'
35 | (GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
| ^
include/uapi/linux/bits.h:8:31: note: expanded from macro '__GENMASK'
8 | (((~_UL(0)) - (_UL(1) << (l)) + 1) & \
| ^
include/linux/bitfield.h:156:30: note: expanded from macro 'FIELD_GET'
156 | (typeof(_mask))(((_reg) & (_mask)) >> __bf_shf(_mask)); \
| ^~~~~
>> drivers/mmc/core/core.c:1667:27: warning: shift count is negative [-Wshift-count-negative]
1667 | u8 ext_addr = FIELD_GET(SDUC_ADDR_EXT_MASK, from);
| ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/mmc/core/sd_ops.h:14:28: note: expanded from macro 'SDUC_ADDR_EXT_MASK'
14 | #define SDUC_ADDR_EXT_MASK GENMASK(37, 32)
| ^
include/linux/bits.h:35:31: note: expanded from macro 'GENMASK'
35 | (GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
| ^
include/uapi/linux/bits.h:9:19: note: expanded from macro '__GENMASK'
9 | (~_UL(0) >> (__BITS_PER_LONG - 1 - (h))))
| ^
include/linux/bitfield.h:156:30: note: expanded from macro 'FIELD_GET'
156 | (typeof(_mask))(((_reg) & (_mask)) >> __bf_shf(_mask)); \
| ^~~~~
drivers/mmc/core/core.c:1667:27: warning: shift count >= width of type [-Wshift-count-overflow]
1667 | u8 ext_addr = FIELD_GET(SDUC_ADDR_EXT_MASK, from);
| ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/mmc/core/sd_ops.h:14:28: note: expanded from macro 'SDUC_ADDR_EXT_MASK'
14 | #define SDUC_ADDR_EXT_MASK GENMASK(37, 32)
| ^
include/linux/bits.h:35:31: note: expanded from macro 'GENMASK'
35 | (GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
| ^
include/uapi/linux/bits.h:8:31: note: expanded from macro '__GENMASK'
8 | (((~_UL(0)) - (_UL(1) << (l)) + 1) & \
| ^
include/linux/bitfield.h:156:50: note: expanded from macro 'FIELD_GET'
156 | (typeof(_mask))(((_reg) & (_mask)) >> __bf_shf(_mask)); \
| ~~~~~~~~~^~~~~~
include/linux/bitfield.h:45:38: note: expanded from macro '__bf_shf'
45 | #define __bf_shf(x) (__builtin_ffsll(x) - 1)
| ^
drivers/mmc/core/core.c:1667:27: warning: shift count is negative [-Wshift-count-negative]
1667 | u8 ext_addr = FIELD_GET(SDUC_ADDR_EXT_MASK, from);
| ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/mmc/core/sd_ops.h:14:28: note: expanded from macro 'SDUC_ADDR_EXT_MASK'
14 | #define SDUC_ADDR_EXT_MASK GENMASK(37, 32)
| ^
include/linux/bits.h:35:31: note: expanded from macro 'GENMASK'
35 | (GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
| ^
include/uapi/linux/bits.h:9:19: note: expanded from macro '__GENMASK'
9 | (~_UL(0) >> (__BITS_PER_LONG - 1 - (h))))
| ^
include/linux/bitfield.h:156:50: note: expanded from macro 'FIELD_GET'
156 | (typeof(_mask))(((_reg) & (_mask)) >> __bf_shf(_mask)); \
| ~~~~~~~~~^~~~~~
include/linux/bitfield.h:45:38: note: expanded from macro '__bf_shf'
45 | #define __bf_shf(x) (__builtin_ffsll(x) - 1)
| ^
drivers/mmc/core/core.c:1689:27: warning: shift count >= width of type [-Wshift-count-overflow]
1689 | u8 ext_addr = FIELD_GET(SDUC_ADDR_EXT_MASK, to);
| ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~
drivers/mmc/core/sd_ops.h:14:28: note: expanded from macro 'SDUC_ADDR_EXT_MASK'
14 | #define SDUC_ADDR_EXT_MASK GENMASK(37, 32)
| ^
include/linux/bits.h:35:31: note: expanded from macro 'GENMASK'
35 | (GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
| ^
include/uapi/linux/bits.h:8:31: note: expanded from macro '__GENMASK'
8 | (((~_UL(0)) - (_UL(1) << (l)) + 1) & \
| ^
note: (skipping 3 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)
include/linux/compiler_types.h:510:22: note: expanded from macro 'compiletime_assert'
510 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
| ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:498:23: note: expanded from macro '_compiletime_assert'
498 | __compiletime_assert(condition, msg, prefix, suffix)
| ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:490:9: note: expanded from macro '__compiletime_assert'
490 | if (!(condition)) \
| ^~~~~~~~~
drivers/mmc/core/core.c:1689:27: warning: shift count is negative [-Wshift-count-negative]
1689 | u8 ext_addr = FIELD_GET(SDUC_ADDR_EXT_MASK, to);
| ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~
drivers/mmc/core/sd_ops.h:14:28: note: expanded from macro 'SDUC_ADDR_EXT_MASK'
14 | #define SDUC_ADDR_EXT_MASK GENMASK(37, 32)
| ^
include/linux/bits.h:35:31: note: expanded from macro 'GENMASK'
35 | (GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
| ^
include/uapi/linux/bits.h:9:19: note: expanded from macro '__GENMASK'
9 | (~_UL(0) >> (__BITS_PER_LONG - 1 - (h))))
| ^
note: (skipping 3 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)
include/linux/compiler_types.h:510:22: note: expanded from macro 'compiletime_assert'
510 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
| ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:498:23: note: expanded from macro '_compiletime_assert'
498 | __compiletime_assert(condition, msg, prefix, suffix)
| ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler_types.h:490:9: note: expanded from macro '__compiletime_assert'
490 | if (!(condition)) \
| ^~~~~~~~~
drivers/mmc/core/core.c:1689:27: warning: shift count >= width of type [-Wshift-count-overflow]
1689 | u8 ext_addr = FIELD_GET(SDUC_ADDR_EXT_MASK, to);
| ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~
drivers/mmc/core/sd_ops.h:14:28: note: expanded from macro 'SDUC_ADDR_EXT_MASK'
14 | #define SDUC_ADDR_EXT_MASK GENMASK(37, 32)
| ^
vim +1667 drivers/mmc/core/core.c
1618
1619 static int mmc_do_erase(struct mmc_card *card, sector_t from,
1620 sector_t to, unsigned int arg)
1621 {
1622 struct mmc_command cmd = {};
1623 unsigned int qty = 0, busy_timeout = 0;
1624 bool use_r1b_resp;
1625 int err;
1626
1627 mmc_retune_hold(card->host);
1628
1629 /*
1630 * qty is used to calculate the erase timeout which depends on how many
1631 * erase groups (or allocation units in SD terminology) are affected.
1632 * We count erasing part of an erase group as one erase group.
1633 * For SD, the allocation units are always a power of 2. For MMC, the
1634 * erase group size is almost certainly also power of 2, but it does not
1635 * seem to insist on that in the JEDEC standard, so we fall back to
1636 * division in that case. SD may not specify an allocation unit size,
1637 * in which case the timeout is based on the number of write blocks.
1638 *
1639 * Note that the timeout for secure trim 2 will only be correct if the
1640 * number of erase groups specified is the same as the total of all
1641 * preceding secure trim 1 commands. Since the power may have been
1642 * lost since the secure trim 1 commands occurred, it is generally
1643 * impossible to calculate the secure trim 2 timeout correctly.
1644 */
1645 if (card->erase_shift)
1646 qty += ((to >> card->erase_shift) -
1647 (from >> card->erase_shift)) + 1;
1648 else if (mmc_card_sd(card))
1649 qty += to - from + 1;
1650 else
1651 qty += ((to / card->erase_size) -
1652 (from / card->erase_size)) + 1;
1653
1654 if (!mmc_card_blockaddr(card)) {
1655 from <<= 9;
1656 to <<= 9;
1657 }
1658
1659 if (mmc_card_sd(card))
1660 cmd.opcode = SD_ERASE_WR_BLK_START;
1661 else
1662 cmd.opcode = MMC_ERASE_GROUP_START;
1663 cmd.arg = from & 0xFFFFFFFF;
1664 cmd.flags = MMC_RSP_SPI_R1 | MMC_RSP_R1 | MMC_CMD_AC;
1665
1666 if (mmc_card_is_sduc(card->host)) {
> 1667 u8 ext_addr = FIELD_GET(SDUC_ADDR_EXT_MASK, from);
1668
1669 mmc_send_ext_addr(card->host, ext_addr);
1670 }
1671
1672 err = mmc_wait_for_cmd(card->host, &cmd, 0);
1673 if (err) {
1674 pr_err("mmc_erase: group start error %d, "
1675 "status %#x\n", err, cmd.resp[0]);
1676 err = -EIO;
1677 goto out;
1678 }
1679
1680 memset(&cmd, 0, sizeof(struct mmc_command));
1681 if (mmc_card_sd(card))
1682 cmd.opcode = SD_ERASE_WR_BLK_END;
1683 else
1684 cmd.opcode = MMC_ERASE_GROUP_END;
1685 cmd.arg = to & 0xFFFFFFFF;
1686 cmd.flags = MMC_RSP_SPI_R1 | MMC_RSP_R1 | MMC_CMD_AC;
1687
1688 if (mmc_card_is_sduc(card->host)) {
1689 u8 ext_addr = FIELD_GET(SDUC_ADDR_EXT_MASK, to);
1690
1691 mmc_send_ext_addr(card->host, ext_addr);
1692 }
1693
1694 err = mmc_wait_for_cmd(card->host, &cmd, 0);
1695 if (err) {
1696 pr_err("mmc_erase: group end error %d, status %#x\n",
1697 err, cmd.resp[0]);
1698 err = -EIO;
1699 goto out;
1700 }
1701
1702 memset(&cmd, 0, sizeof(struct mmc_command));
1703 cmd.opcode = MMC_ERASE;
1704 cmd.arg = arg;
1705 busy_timeout = mmc_erase_timeout(card, arg, qty);
1706 use_r1b_resp = mmc_prepare_busy_cmd(card->host, &cmd, busy_timeout);
1707
1708 err = mmc_wait_for_cmd(card->host, &cmd, 0);
1709 if (err) {
1710 pr_err("mmc_erase: erase error %d, status %#x\n",
1711 err, cmd.resp[0]);
1712 err = -EIO;
1713 goto out;
1714 }
1715
1716 if (mmc_host_is_spi(card->host))
1717 goto out;
1718
1719 /*
1720 * In case of when R1B + MMC_CAP_WAIT_WHILE_BUSY is used, the polling
1721 * shall be avoided.
1722 */
1723 if ((card->host->caps & MMC_CAP_WAIT_WHILE_BUSY) && use_r1b_resp)
1724 goto out;
1725
1726 /* Let's poll to find out when the erase operation completes. */
1727 err = mmc_poll_for_busy(card, busy_timeout, false, MMC_BUSY_ERASE);
1728
1729 out:
1730 mmc_retune_release(card->host);
1731 return err;
1732 }
1733
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] only message in thread