Re: [PATCH 2/2] EDAC/igen6: Make masks of {MCHBAR, TOM, TOUUD, ECC_ERROR_LOG} configurable

["Luck, Tony" <tony.luck@intel.com>]

On Mon, Nov 24, 2025 at 02:54:57PM +0800, Qiuxu Zhuo wrote:
> diff --git a/drivers/edac/igen6_edac.c b/drivers/edac/igen6_edac.c
> index 5d887a3115f0..db4aa9f581e3 100644
> --- a/drivers/edac/igen6_edac.c
> +++ b/drivers/edac/igen6_edac.c
...
> +/* Non-constant mask variant of FIELD_GET() */
> +#define field_get(mask, reg)  (((reg) & (mask)) >> (ffs(mask) - 1))

I applied to v6.19-rc1. But when building with make W=1 I get this warning:

drivers/edac/igen6_edac.c:125:9: warning: "field_get" redefined
  125 | #define field_get(mask, reg)  (((reg) & (mask)) >> (ffs(mask) - 1))
      |         ^~~~~~~~~
In file included from ./include/linux/fortify-string.h:5,
                 from ./include/linux/string.h:386,
                 from ./include/linux/bitmap.h:13,
                 from ./include/linux/cpumask.h:11,
                 from ./arch/x86/include/asm/paravirt.h:21,
                 from ./arch/x86/include/asm/cpuid/api.h:57,
                 from ./arch/x86/include/asm/processor.h:19,
                 from ./arch/x86/include/asm/timex.h:5,
                 from ./include/linux/timex.h:67,
                 from ./include/linux/time32.h:13,
                 from ./include/linux/time.h:60,
                 from ./include/linux/stat.h:19,
                 from ./include/linux/module.h:13,
                 from drivers/edac/igen6_edac.c:13:
./include/linux/bitfield.h:298:9: note: this is the location of the previous definition
  298 | #define field_get(mask, reg)                                            \

-Tony