From db7194fbe47479bdab69666e2e1717a33d299dcb Mon Sep 17 00:00:00 2001
From: Nadim Taha <ntaha@google.com>
Date: Thu, 28 Jan 2016 17:16:44 -0800
Subject: [PATCH] common: Fixes the SFDP_BITFIELD() macro.

BUG=none
BRANCH=none
TEST=<redacted>

Change-Id: If6e2732a29e084417ba1d2f80732ac412028342a
Signed-off-by: Nadim Taha <ntaha@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/324622
Reviewed-by: Ewout van Bekkum <ewout@chromium.org>
---
 include/sfdp.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/sfdp.h b/include/sfdp.h
index d9b3c13903..4f7ddfb21e 100644
--- a/include/sfdp.h
+++ b/include/sfdp.h
@@ -27,7 +27,7 @@
  * Helper macros to construct SFDP defined double words (32b). Note reserved or
  * unused fields must always be set to all 1's.
  */
-#define SFDP_BITFIELD(name, value) (((value) & name ## _MASK) << name ## _SHIFT)
+#define SFDP_BITFIELD(name, value) (((value) << name ## _SHIFT) & name ## _MASK)
 #define SFDP_UNUSED(hi, lo) (((1ULL << ((hi) - (lo) + 1)) - 1UL) << (lo))
 
 /******************************************************************************/