Messages in this thread | | | Date | Wed, 18 Jan 2023 13:29:52 +0100 | Subject | Re: [PATCH v7 1/3] phy: mediatek: remove temporary variable @mask_ | From | AngeloGioacchino Del Regno <> |
| |
Il 18/01/23 09:43, Chunfeng Yun ha scritto: > Remove the temporary @mask_, this may cause build warning when use clang > compiler for powerpc, but can't reproduce it when compile for arm64. > the build warning is caused by: > > "warning: result of comparison of constant 18446744073709551615 with > expression of type (aka 'unsigned long') is always false > [-Wtautological-constant-out-of-range-compare]" > > More information provided in below lore link. > > After removing @mask_, there is a "CHECK:MACRO_ARG_REUSE" when run > checkpatch.pl, but due to @mask is constant, no reuse problem will happen. > > Link: https://lore.kernel.org/lkml/202212160357.jJuesD8n-lkp@intel.com/t/ > Fixes: 84513eccd678 ("phy: mediatek: fix build warning of FIELD_PREP()") > Reported-by: kernel test robot <lkp@intel.com> > Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com>
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
| |