diff options
author | Chen, Gong <gong.chen@linux.intel.com> | 2013-10-18 23:29:07 +0200 |
---|---|---|
committer | Tony Luck <tony.luck@intel.com> | 2013-10-22 00:12:01 +0200 |
commit | 10ef6b0dffe404bcc54e94cb2ca1a5b18445a66b (patch) | |
tree | 6450859ffdc4494f2211442dd411b7f189cf326d /drivers/edac/sb_edac.c | |
parent | ACPI, CPER: Update cper info (diff) | |
download | linux-10ef6b0dffe404bcc54e94cb2ca1a5b18445a66b.tar.xz linux-10ef6b0dffe404bcc54e94cb2ca1a5b18445a66b.zip |
bitops: Introduce a more generic BITMASK macro
GENMASK is used to create a contiguous bitmask([hi:lo]). It is
implemented twice in current kernel. One is in EDAC driver, the other
is in SiS/XGI FB driver. Move it to a more generic place for other
usage.
Signed-off-by: Chen, Gong <gong.chen@linux.intel.com>
Cc: Borislav Petkov <bp@alien8.de>
Cc: Thomas Winischhofer <thomas@winischhofer.net>
Cc: Jean-Christophe Plagniol-Villard <plagnioj@jcrosoft.com>
Cc: Tomi Valkeinen <tomi.valkeinen@ti.com>
Acked-by: Borislav Petkov <bp@suse.de>
Acked-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Signed-off-by: Tony Luck <tony.luck@intel.com>
Diffstat (limited to 'drivers/edac/sb_edac.c')
-rw-r--r-- | drivers/edac/sb_edac.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/edac/sb_edac.c b/drivers/edac/sb_edac.c index e04462b60756..88f60c5fecbc 100644 --- a/drivers/edac/sb_edac.c +++ b/drivers/edac/sb_edac.c @@ -50,7 +50,7 @@ static int probed; * Get a bit field at register value <v>, from bit <lo> to bit <hi> */ #define GET_BITFIELD(v, lo, hi) \ - (((v) & ((1ULL << ((hi) - (lo) + 1)) - 1) << (lo)) >> (lo)) + (((v) & GENMASK_ULL(hi, lo)) >> (lo)) /* * sbridge Memory Controller Registers |