Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Javier Abrego <javier.abrego.lorente@gmail.com>
To:     robert.moore@intel.com, rafael.j.wysocki@intel.com, lenb@kernel.org
Cc:     linux-acpi@vger.kernel.org, devel@acpica.org,
        linux-kernel@vger.kernel.org,
        Javier Abrego <javier.abrego.lorente@gmail.com>
Subject: [PATCH] ACPI: acpica: acmacros: fixed linter issues
Date:   Sun,  3 Jul 2022 02:30:14 +0200
Message-Id: <20220703003014.52949-1-javier.abrego.lorente@gmail.com>
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Precedence: bulk

Fixed a coding style issue with the linter.

Signed-off-by: Javier Abrego <javier.abrego.lorente@gmail.com>
---
 drivers/acpi/acpica/acmacros.h | 212 ++++++++++++++++-----------------
 1 file changed, 106 insertions(+), 106 deletions(-)

diff --git a/drivers/acpi/acpica/acmacros.h b/drivers/acpi/acpica/acmacros.h
index 2f3e609df..597070cde 100644
--- a/drivers/acpi/acpica/acmacros.h
+++ b/drivers/acpi/acpica/acmacros.h
@@ -15,18 +15,18 @@
  * get into potential alignment issues -- see the STORE macros below.
  * Use with care.
  */
-#define ACPI_CAST8(ptr)                 ACPI_CAST_PTR (u8, (ptr))
-#define ACPI_CAST16(ptr)                ACPI_CAST_PTR (u16, (ptr))
-#define ACPI_CAST32(ptr)                ACPI_CAST_PTR (u32, (ptr))
-#define ACPI_CAST64(ptr)                ACPI_CAST_PTR (u64, (ptr))
-#define ACPI_GET8(ptr)                  (*ACPI_CAST8 (ptr))
-#define ACPI_GET16(ptr)                 (*ACPI_CAST16 (ptr))
-#define ACPI_GET32(ptr)                 (*ACPI_CAST32 (ptr))
-#define ACPI_GET64(ptr)                 (*ACPI_CAST64 (ptr))
-#define ACPI_SET8(ptr, val)             (*ACPI_CAST8 (ptr) = (u8) (val))
-#define ACPI_SET16(ptr, val)            (*ACPI_CAST16 (ptr) = (u16) (val))
-#define ACPI_SET32(ptr, val)            (*ACPI_CAST32 (ptr) = (u32) (val))
-#define ACPI_SET64(ptr, val)            (*ACPI_CAST64 (ptr) = (u64) (val))
+#define ACPI_CAST8(ptr)                 ACPI_CAST_PTR(u8, (ptr))
+#define ACPI_CAST16(ptr)                ACPI_CAST_PTR(u16, (ptr))
+#define ACPI_CAST32(ptr)                ACPI_CAST_PTR(u32, (ptr))
+#define ACPI_CAST64(ptr)                ACPI_CAST_PTR(u64, (ptr))
+#define ACPI_GET8(ptr)                  (*ACPI_CAST8(ptr))
+#define ACPI_GET16(ptr)                 (*ACPI_CAST16(ptr))
+#define ACPI_GET32(ptr)                 (*ACPI_CAST32(ptr))
+#define ACPI_GET64(ptr)                 (*ACPI_CAST64(ptr))
+#define ACPI_SET8(ptr, val)             (*ACPI_CAST8(ptr) = (u8) (val))
+#define ACPI_SET16(ptr, val)            (*ACPI_CAST16(ptr) = (u16) (val))
+#define ACPI_SET32(ptr, val)            (*ACPI_CAST32(ptr) = (u32) (val))
+#define ACPI_SET64(ptr, val)            (*ACPI_CAST64(ptr) = (u64) (val))
 
 /*
  * printf() format helper. This macro is a workaround for the difficulties
@@ -51,31 +51,31 @@
 	 /*  Hi...Lo                     Lo...Hi     */
 /* 16-bit source, 16/32/64 destination */
 
-#define ACPI_MOVE_16_TO_16(d, s)        {((  u8 *)(void *)(d))[0] = ((u8 *)(void *)(s))[1];\
-			  ((  u8 *)(void *)(d))[1] = ((u8 *)(void *)(s))[0];}
+#define ACPI_MOVE_16_TO_16(d, s)        {((u8 *)(void *)(d))[0] = ((u8 *)(void *)(s))[1];\
+			  ((u8 *)(void *)(d))[1] = ((u8 *)(void *)(s))[0]; }
 
-#define ACPI_MOVE_16_TO_32(d, s)        {(*(u32 *)(void *)(d))=0;\
+#define ACPI_MOVE_16_TO_32(d, s)        {(*(u32 *)(void *)(d)) = 0;\
 					  ((u8 *)(void *)(d))[2] = ((u8 *)(void *)(s))[1];\
-					  ((u8 *)(void *)(d))[3] = ((u8 *)(void *)(s))[0];}
+					  ((u8 *)(void *)(d))[3] = ((u8 *)(void *)(s))[0]; }
 
-#define ACPI_MOVE_16_TO_64(d, s)        {(*(u64 *)(void *)(d))=0;\
+#define ACPI_MOVE_16_TO_64(d, s)        {(*(u64 *)(void *)(d)) = 0;\
 							   ((u8 *)(void *)(d))[6] = ((u8 *)(void *)(s))[1];\
-							   ((u8 *)(void *)(d))[7] = ((u8 *)(void *)(s))[0];}
+							   ((u8 *)(void *)(d))[7] = ((u8 *)(void *)(s))[0]; }
 
 /* 32-bit source, 16/32/64 destination */
 
 #define ACPI_MOVE_32_TO_16(d, s)        ACPI_MOVE_16_TO_16(d, s)	/* Truncate to 16 */
 
-#define ACPI_MOVE_32_TO_32(d, s)        {((  u8 *)(void *)(d))[0] = ((u8 *)(void *)(s))[3];\
-									  ((  u8 *)(void *)(d))[1] = ((u8 *)(void *)(s))[2];\
-									  ((  u8 *)(void *)(d))[2] = ((u8 *)(void *)(s))[1];\
-									  ((  u8 *)(void *)(d))[3] = ((u8 *)(void *)(s))[0];}
+#define ACPI_MOVE_32_TO_32(d, s)        {((u8 *)(void *)(d))[0] = ((u8 *)(void *)(s))[3];\
+									  ((u8 *)(void *)(d))[1] = ((u8 *)(void *)(s))[2];\
+									  ((u8 *)(void *)(d))[2] = ((u8 *)(void *)(s))[1];\
+									  ((u8 *)(void *)(d))[3] = ((u8 *)(void *)(s))[0]; }
 
-#define ACPI_MOVE_32_TO_64(d, s)        {(*(u64 *)(void *)(d))=0;\
+#define ACPI_MOVE_32_TO_64(d, s)        {(*(u64 *)(void *)(d)) = 0;\
 										   ((u8 *)(void *)(d))[4] = ((u8 *)(void *)(s))[3];\
 										   ((u8 *)(void *)(d))[5] = ((u8 *)(void *)(s))[2];\
 										   ((u8 *)(void *)(d))[6] = ((u8 *)(void *)(s))[1];\
-										   ((u8 *)(void *)(d))[7] = ((u8 *)(void *)(s))[0];}
+										   ((u8 *)(void *)(d))[7] = ((u8 *)(void *)(s))[0]; }
 
 /* 64-bit source, 16/32/64 destination */
 
@@ -83,14 +83,14 @@
 
 #define ACPI_MOVE_64_TO_32(d, s)        ACPI_MOVE_32_TO_32(d, s)	/* Truncate to 32 */
 
-#define ACPI_MOVE_64_TO_64(d, s)        {((  u8 *)(void *)(d))[0] = ((u8 *)(void *)(s))[7];\
-										 ((  u8 *)(void *)(d))[1] = ((u8 *)(void *)(s))[6];\
-										 ((  u8 *)(void *)(d))[2] = ((u8 *)(void *)(s))[5];\
-										 ((  u8 *)(void *)(d))[3] = ((u8 *)(void *)(s))[4];\
-										 ((  u8 *)(void *)(d))[4] = ((u8 *)(void *)(s))[3];\
-										 ((  u8 *)(void *)(d))[5] = ((u8 *)(void *)(s))[2];\
-										 ((  u8 *)(void *)(d))[6] = ((u8 *)(void *)(s))[1];\
-										 ((  u8 *)(void *)(d))[7] = ((u8 *)(void *)(s))[0];}
+#define ACPI_MOVE_64_TO_64(d, s)        {((u8 *)(void *)(d))[0] = ((u8 *)(void *)(s))[7];\
+										 ((u8 *)(void *)(d))[1] = ((u8 *)(void *)(s))[6];\
+										 ((u8 *)(void *)(d))[2] = ((u8 *)(void *)(s))[5];\
+										 ((u8 *)(void *)(d))[3] = ((u8 *)(void *)(s))[4];\
+										 ((u8 *)(void *)(d))[4] = ((u8 *)(void *)(s))[3];\
+										 ((u8 *)(void *)(d))[5] = ((u8 *)(void *)(s))[2];\
+										 ((u8 *)(void *)(d))[6] = ((u8 *)(void *)(s))[1];\
+										 ((u8 *)(void *)(d))[7] = ((u8 *)(void *)(s))[0]; }
 #else
 /*
  * Macros for little-endian machines
@@ -127,35 +127,35 @@
 
 /* 16-bit source, 16/32/64 destination */
 
-#define ACPI_MOVE_16_TO_16(d, s)        {((  u8 *)(void *)(d))[0] = ((u8 *)(void *)(s))[0];\
-										 ((  u8 *)(void *)(d))[1] = ((u8 *)(void *)(s))[1];}
+#define ACPI_MOVE_16_TO_16(d, s)        {((u8 *)(void *)(d))[0] = ((u8 *)(void *)(s))[0];\
+										 ((u8 *)(void *)(d))[1] = ((u8 *)(void *)(s))[1]; }
 
-#define ACPI_MOVE_16_TO_32(d, s)        {(*(u32 *)(void *)(d)) = 0; ACPI_MOVE_16_TO_16(d, s);}
-#define ACPI_MOVE_16_TO_64(d, s)        {(*(u64 *)(void *)(d)) = 0; ACPI_MOVE_16_TO_16(d, s);}
+#define ACPI_MOVE_16_TO_32(d, s)        {(*(u32 *)(void *)(d)) = 0; ACPI_MOVE_16_TO_16(d, s); }
+#define ACPI_MOVE_16_TO_64(d, s)        {(*(u64 *)(void *)(d)) = 0; ACPI_MOVE_16_TO_16(d, s); }
 
 /* 32-bit source, 16/32/64 destination */
 
 #define ACPI_MOVE_32_TO_16(d, s)        ACPI_MOVE_16_TO_16(d, s)	/* Truncate to 16 */
 
-#define ACPI_MOVE_32_TO_32(d, s)        {((  u8 *)(void *)(d))[0] = ((u8 *)(void *)(s))[0];\
-										 ((  u8 *)(void *)(d))[1] = ((u8 *)(void *)(s))[1];\
-										 ((  u8 *)(void *)(d))[2] = ((u8 *)(void *)(s))[2];\
-										 ((  u8 *)(void *)(d))[3] = ((u8 *)(void *)(s))[3];}
+#define ACPI_MOVE_32_TO_32(d, s)        {((u8 *)(void *)(d))[0] = ((u8 *)(void *)(s))[0];\
+										 ((u8 *)(void *)(d))[1] = ((u8 *)(void *)(s))[1];\
+										 ((u8 *)(void *)(d))[2] = ((u8 *)(void *)(s))[2];\
+										 ((u8 *)(void *)(d))[3] = ((u8 *)(void *)(s))[3]; }
 
-#define ACPI_MOVE_32_TO_64(d, s)        {(*(u64 *)(void *)(d)) = 0; ACPI_MOVE_32_TO_32(d, s);}
+#define ACPI_MOVE_32_TO_64(d, s)        {(*(u64 *)(void *)(d)) = 0; ACPI_MOVE_32_TO_32(d, s); }
 
 /* 64-bit source, 16/32/64 destination */
 
 #define ACPI_MOVE_64_TO_16(d, s)        ACPI_MOVE_16_TO_16(d, s)	/* Truncate to 16 */
 #define ACPI_MOVE_64_TO_32(d, s)        ACPI_MOVE_32_TO_32(d, s)	/* Truncate to 32 */
-#define ACPI_MOVE_64_TO_64(d, s)        {((  u8 *)(void *)(d))[0] = ((u8 *)(void *)(s))[0];\
-										 ((  u8 *)(void *)(d))[1] = ((u8 *)(void *)(s))[1];\
-										 ((  u8 *)(void *)(d))[2] = ((u8 *)(void *)(s))[2];\
-										 ((  u8 *)(void *)(d))[3] = ((u8 *)(void *)(s))[3];\
-										 ((  u8 *)(void *)(d))[4] = ((u8 *)(void *)(s))[4];\
-										 ((  u8 *)(void *)(d))[5] = ((u8 *)(void *)(s))[5];\
-										 ((  u8 *)(void *)(d))[6] = ((u8 *)(void *)(s))[6];\
-										 ((  u8 *)(void *)(d))[7] = ((u8 *)(void *)(s))[7];}
+#define ACPI_MOVE_64_TO_64(d, s)        {((u8 *)(void *)(d))[0] = ((u8 *)(void *)(s))[0];\
+										 ((u8 *)(void *)(d))[1] = ((u8 *)(void *)(s))[1];\
+										 ((u8 *)(void *)(d))[2] = ((u8 *)(void *)(s))[2];\
+										 ((u8 *)(void *)(d))[3] = ((u8 *)(void *)(s))[3];\
+										 ((u8 *)(void *)(d))[4] = ((u8 *)(void *)(s))[4];\
+										 ((u8 *)(void *)(d))[5] = ((u8 *)(void *)(s))[5];\
+										 ((u8 *)(void *)(d))[6] = ((u8 *)(void *)(s))[6];\
+										 ((u8 *)(void *)(d))[7] = ((u8 *)(void *)(s))[7]; }
 #endif
 #endif
 
@@ -164,7 +164,7 @@
  */
 #define _ACPI_DIV(value, power_of2)     ((u32) ((value) >> (power_of2)))
 #define _ACPI_MUL(value, power_of2)     ((u32) ((value) << (power_of2)))
-#define _ACPI_MOD(value, divisor)       ((u32) ((value) & ((divisor) -1)))
+#define _ACPI_MOD(value, divisor)       ((u32) ((value) & ((divisor) - 1)))
 
 #define ACPI_DIV_2(a)                   _ACPI_DIV(a, 1)
 #define ACPI_MUL_2(a)                   _ACPI_MUL(a, 1)
@@ -232,64 +232,64 @@
 
 #define __ACPI_FIND_LAST_BIT_2(a, r)        ((((u8)  (a)) & 0x02) ? (r)+1 : (r))
 #define __ACPI_FIND_LAST_BIT_4(a, r)        ((((u8)  (a)) & 0x0C) ? \
-											 __ACPI_FIND_LAST_BIT_2  ((a)>>2,  (r)+2) : \
-											 __ACPI_FIND_LAST_BIT_2  ((a), (r)))
+											 __ACPI_FIND_LAST_BIT_2((a)>>2,  (r)+2) : \
+											 __ACPI_FIND_LAST_BIT_2((a), (r)))
 #define __ACPI_FIND_LAST_BIT_8(a, r)        ((((u8)  (a)) & 0xF0) ? \
-											 __ACPI_FIND_LAST_BIT_4  ((a)>>4,  (r)+4) : \
-											 __ACPI_FIND_LAST_BIT_4  ((a), (r)))
+											 __ACPI_FIND_LAST_BIT_4((a)>>4,  (r)+4) : \
+											 __ACPI_FIND_LAST_BIT_4((a), (r)))
 #define __ACPI_FIND_LAST_BIT_16(a, r)       ((((u16) (a)) & 0xFF00) ? \
-											 __ACPI_FIND_LAST_BIT_8  ((a)>>8,  (r)+8) : \
-											 __ACPI_FIND_LAST_BIT_8  ((a), (r)))
+											 __ACPI_FIND_LAST_BIT_8((a)>>8,  (r)+8) : \
+											 __ACPI_FIND_LAST_BIT_8((a), (r)))
 #define __ACPI_FIND_LAST_BIT_32(a, r)       ((((u32) (a)) & 0xFFFF0000) ? \
-											 __ACPI_FIND_LAST_BIT_16 ((a)>>16, (r)+16) : \
-											 __ACPI_FIND_LAST_BIT_16 ((a), (r)))
+											 __ACPI_FIND_LAST_BIT_16((a)>>16, (r)+16) : \
+											 __ACPI_FIND_LAST_BIT_16((a), (r)))
 #define __ACPI_FIND_LAST_BIT_64(a, r)       ((((u64) (a)) & 0xFFFFFFFF00000000) ? \
-											 __ACPI_FIND_LAST_BIT_32 ((a)>>32, (r)+32) : \
-											 __ACPI_FIND_LAST_BIT_32 ((a), (r)))
+											 __ACPI_FIND_LAST_BIT_32((a)>>32, (r)+32) : \
+											 __ACPI_FIND_LAST_BIT_32((a), (r)))
 
-#define ACPI_FIND_LAST_BIT_8(a)             ((a) ? __ACPI_FIND_LAST_BIT_8 (a, 1) : 0)
-#define ACPI_FIND_LAST_BIT_16(a)            ((a) ? __ACPI_FIND_LAST_BIT_16 (a, 1) : 0)
-#define ACPI_FIND_LAST_BIT_32(a)            ((a) ? __ACPI_FIND_LAST_BIT_32 (a, 1) : 0)
-#define ACPI_FIND_LAST_BIT_64(a)            ((a) ? __ACPI_FIND_LAST_BIT_64 (a, 1) : 0)
+#define ACPI_FIND_LAST_BIT_8(a)             ((a) ? __ACPI_FIND_LAST_BIT_8(a, 1) : 0)
+#define ACPI_FIND_LAST_BIT_16(a)            ((a) ? __ACPI_FIND_LAST_BIT_16(a, 1) : 0)
+#define ACPI_FIND_LAST_BIT_32(a)            ((a) ? __ACPI_FIND_LAST_BIT_32(a, 1) : 0)
+#define ACPI_FIND_LAST_BIT_64(a)            ((a) ? __ACPI_FIND_LAST_BIT_64(a, 1) : 0)
 
 #define __ACPI_FIND_FIRST_BIT_2(a, r)       ((((u8) (a)) & 0x01) ? (r) : (r)+1)
 #define __ACPI_FIND_FIRST_BIT_4(a, r)       ((((u8) (a)) & 0x03) ? \
-											 __ACPI_FIND_FIRST_BIT_2  ((a), (r)) : \
-											 __ACPI_FIND_FIRST_BIT_2  ((a)>>2, (r)+2))
+											 __ACPI_FIND_FIRST_BIT_2((a), (r)) : \
+											 __ACPI_FIND_FIRST_BIT_2((a)>>2, (r)+2))
 #define __ACPI_FIND_FIRST_BIT_8(a, r)       ((((u8) (a)) & 0x0F) ? \
-											 __ACPI_FIND_FIRST_BIT_4  ((a), (r)) : \
-											 __ACPI_FIND_FIRST_BIT_4  ((a)>>4, (r)+4))
+											 __ACPI_FIND_FIRST_BIT_4((a), (r)) : \
+											 __ACPI_FIND_FIRST_BIT_4((a)>>4, (r)+4))
 #define __ACPI_FIND_FIRST_BIT_16(a, r)      ((((u16) (a)) & 0x00FF) ? \
-											 __ACPI_FIND_FIRST_BIT_8  ((a), (r)) : \
-											 __ACPI_FIND_FIRST_BIT_8  ((a)>>8, (r)+8))
+											 __ACPI_FIND_FIRST_BIT_8((a), (r)) : \
+											 __ACPI_FIND_FIRST_BIT_8((a)>>8, (r)+8))
 #define __ACPI_FIND_FIRST_BIT_32(a, r)      ((((u32) (a)) & 0x0000FFFF) ? \
-											 __ACPI_FIND_FIRST_BIT_16 ((a), (r)) : \
-											 __ACPI_FIND_FIRST_BIT_16 ((a)>>16, (r)+16))
+											 __ACPI_FIND_FIRST_BIT_16((a), (r)) : \
+											 __ACPI_FIND_FIRST_BIT_16((a)>>16, (r)+16))
 #define __ACPI_FIND_FIRST_BIT_64(a, r)      ((((u64) (a)) & 0x00000000FFFFFFFF) ? \
-											 __ACPI_FIND_FIRST_BIT_32 ((a), (r)) : \
-											 __ACPI_FIND_FIRST_BIT_32 ((a)>>32, (r)+32))
+											 __ACPI_FIND_FIRST_BIT_32((a), (r)) : \
+											 __ACPI_FIND_FIRST_BIT_32((a)>>32, (r)+32))
 
-#define ACPI_FIND_FIRST_BIT_8(a)            ((a) ? __ACPI_FIND_FIRST_BIT_8 (a, 1) : 0)
-#define ACPI_FIND_FIRST_BIT_16(a)           ((a) ? __ACPI_FIND_FIRST_BIT_16 (a, 1) : 0)
-#define ACPI_FIND_FIRST_BIT_32(a)           ((a) ? __ACPI_FIND_FIRST_BIT_32 (a, 1) : 0)
-#define ACPI_FIND_FIRST_BIT_64(a)           ((a) ? __ACPI_FIND_FIRST_BIT_64 (a, 1) : 0)
+#define ACPI_FIND_FIRST_BIT_8(a)            ((a) ? __ACPI_FIND_FIRST_BIT_8(a, 1) : 0)
+#define ACPI_FIND_FIRST_BIT_16(a)           ((a) ? __ACPI_FIND_FIRST_BIT_16(a, 1) : 0)
+#define ACPI_FIND_FIRST_BIT_32(a)           ((a) ? __ACPI_FIND_FIRST_BIT_32(a, 1) : 0)
+#define ACPI_FIND_FIRST_BIT_64(a)           ((a) ? __ACPI_FIND_FIRST_BIT_64(a, 1) : 0)
 
 #endif				/* ACPI_USE_NATIVE_BIT_FINDER */
 
 /* Generic (power-of-two) rounding */
 
 #define ACPI_ROUND_UP_POWER_OF_TWO_8(a)     ((u8) \
-											(((u16) 1) <<  ACPI_FIND_LAST_BIT_8  ((a)  - 1)))
+											(((u16) 1) <<  ACPI_FIND_LAST_BIT_8((a)  - 1)))
 #define ACPI_ROUND_DOWN_POWER_OF_TWO_8(a)   ((u8) \
-											(((u16) 1) << (ACPI_FIND_LAST_BIT_8  ((a)) - 1)))
+											(((u16) 1) << (ACPI_FIND_LAST_BIT_8((a)) - 1)))
 #define ACPI_ROUND_UP_POWER_OF_TWO_16(a)    ((u16) \
-											(((u32) 1) <<  ACPI_FIND_LAST_BIT_16 ((a)  - 1)))
+											(((u32) 1) <<  ACPI_FIND_LAST_BIT_16((a)  - 1)))
 #define ACPI_ROUND_DOWN_POWER_OF_TWO_16(a)  ((u16) \
-											(((u32) 1) << (ACPI_FIND_LAST_BIT_16 ((a)) - 1)))
+											(((u32) 1) << (ACPI_FIND_LAST_BIT_16((a)) - 1)))
 #define ACPI_ROUND_UP_POWER_OF_TWO_32(a)    ((u32) \
-											(((u64) 1) <<  ACPI_FIND_LAST_BIT_32 ((a)  - 1)))
+											(((u64) 1) <<  ACPI_FIND_LAST_BIT_32((a)  - 1)))
 #define ACPI_ROUND_DOWN_POWER_OF_TWO_32(a)  ((u32) \
-											(((u64) 1) << (ACPI_FIND_LAST_BIT_32 ((a)) - 1)))
+											(((u64) 1) << (ACPI_FIND_LAST_BIT_32((a)) - 1)))
 #define ACPI_IS_ALIGNED(a, s)               (((a) & ((s) - 1)) == 0)
 #define ACPI_IS_POWER_OF_TWO(a)             ACPI_IS_ALIGNED(a, a)
 
@@ -353,10 +353,10 @@
 
 /* Macros to extract flag bits from position one and above */
 
-#define ACPI_EXTRACT_1BIT_FLAG(field, position)     (ACPI_GET_1BIT_FLAG ((field) >> position))
-#define ACPI_EXTRACT_2BIT_FLAG(field, position)     (ACPI_GET_2BIT_FLAG ((field) >> position))
-#define ACPI_EXTRACT_3BIT_FLAG(field, position)     (ACPI_GET_3BIT_FLAG ((field) >> position))
-#define ACPI_EXTRACT_4BIT_FLAG(field, position)     (ACPI_GET_4BIT_FLAG ((field) >> position))
+#define ACPI_EXTRACT_1BIT_FLAG(field, position)     (ACPI_GET_1BIT_FLAG((field) >> position))
+#define ACPI_EXTRACT_2BIT_FLAG(field, position)     (ACPI_GET_2BIT_FLAG((field) >> position))
+#define ACPI_EXTRACT_3BIT_FLAG(field, position)     (ACPI_GET_3BIT_FLAG((field) >> position))
+#define ACPI_EXTRACT_4BIT_FLAG(field, position)     (ACPI_GET_4BIT_FLAG((field) >> position))
 
 /* ACPI Pathname helpers */
 
@@ -379,7 +379,7 @@
 /*
  * Macros for the master AML opcode table
  */
-#if defined (ACPI_DISASSEMBLER) || defined (ACPI_DEBUG_OUTPUT)
+#if defined(ACPI_DISASSEMBLER) || defined(ACPI_DEBUG_OUTPUT)
 #define ACPI_OP(name, Pargs, Iargs, obj_type, class, type, flags) \
 	{name, (u32)(Pargs), (u32)(Iargs), (u32)(flags), obj_type, class, type}
 #else
@@ -421,8 +421,8 @@
  * the plist contains a set of parens to allow variable-length lists.
  * These macros are used for both the debug and non-debug versions of the code.
  */
-#define ACPI_ERROR_NAMESPACE(s, p, e)       acpi_ut_prefixed_namespace_error (AE_INFO, s, p, e);
-#define ACPI_ERROR_METHOD(s, n, p, e)       acpi_ut_method_error (AE_INFO, s, n, p, e);
+#define ACPI_ERROR_NAMESPACE(s, p, e)       acpi_ut_prefixed_namespace_error(AE_INFO, s, p, e);
+#define ACPI_ERROR_METHOD(s, n, p, e)       acpi_ut_method_error(AE_INFO, s, n, p, e);
 #define ACPI_WARN_PREDEFINED(plist)         acpi_ut_predefined_warning plist
 #define ACPI_INFO_PREDEFINED(plist)         acpi_ut_predefined_info plist
 #define ACPI_BIOS_ERROR_PREDEFINED(plist)   acpi_ut_predefined_bios_error plist
@@ -467,17 +467,17 @@
 #ifdef ACPI_ASL_COMPILER
 
 #define ASL_CV_LABEL_FILENODE(a)         cv_label_file_node(a);
-#define ASL_CV_CAPTURE_COMMENTS_ONLY(a)   cv_capture_comments_only (a);
-#define ASL_CV_CAPTURE_COMMENTS(a)       cv_capture_comments (a);
-#define ASL_CV_TRANSFER_COMMENTS(a)      cv_transfer_comments (a);
-#define ASL_CV_CLOSE_PAREN(a,b)          cv_close_paren_write_comment(a,b);
-#define ASL_CV_CLOSE_BRACE(a,b)          cv_close_brace_write_comment(a,b);
-#define ASL_CV_SWITCH_FILES(a,b)         cv_switch_files(a,b);
+#define ASL_CV_CAPTURE_COMMENTS_ONLY(a)   cv_capture_comments_only(a);
+#define ASL_CV_CAPTURE_COMMENTS(a)       cv_capture_comments(a);
+#define ASL_CV_TRANSFER_COMMENTS(a)      cv_transfer_comments(a);
+#define ASL_CV_CLOSE_PAREN(a, b)          cv_close_paren_write_comment(a, b);
+#define ASL_CV_CLOSE_BRACE(a, b)          cv_close_brace_write_comment(a, b);
+#define ASL_CV_SWITCH_FILES(a, b)         cv_switch_files(a, b);
 #define ASL_CV_CLEAR_OP_COMMENTS(a)       cv_clear_op_comments(a);
-#define ASL_CV_PRINT_ONE_COMMENT(a,b,c,d) cv_print_one_comment_type (a,b,c,d);
-#define ASL_CV_PRINT_ONE_COMMENT_LIST(a,b) cv_print_one_comment_list (a,b);
+#define ASL_CV_PRINT_ONE_COMMENT(a, b, c, d) cv_print_one_comment_type (a, b, c, d);
+#define ASL_CV_PRINT_ONE_COMMENT_LIST(a, b) cv_print_one_comment_list (a, b);
 #define ASL_CV_FILE_HAS_SWITCHED(a)       cv_file_has_switched(a)
-#define ASL_CV_INIT_FILETREE(a,b)      cv_init_file_tree(a,b);
+#define ASL_CV_INIT_FILETREE(a, b)      cv_init_file_tree(a, b);
 
 #else
 
@@ -485,14 +485,14 @@
 #define ASL_CV_CAPTURE_COMMENTS_ONLY(a)
 #define ASL_CV_CAPTURE_COMMENTS(a)
 #define ASL_CV_TRANSFER_COMMENTS(a)
-#define ASL_CV_CLOSE_PAREN(a,b)          acpi_os_printf (")");
-#define ASL_CV_CLOSE_BRACE(a,b)          acpi_os_printf ("}");
-#define ASL_CV_SWITCH_FILES(a,b)
+#define ASL_CV_CLOSE_PAREN(a, b)          acpi_os_printf (")");
+#define ASL_CV_CLOSE_BRACE(a, b)          acpi_os_printf ("}");
+#define ASL_CV_SWITCH_FILES(a, b)
 #define ASL_CV_CLEAR_OP_COMMENTS(a)
-#define ASL_CV_PRINT_ONE_COMMENT(a,b,c,d)
-#define ASL_CV_PRINT_ONE_COMMENT_LIST(a,b)
+#define ASL_CV_PRINT_ONE_COMMENT(a, b, c, d)
+#define ASL_CV_PRINT_ONE_COMMENT_LIST(a, b)
 #define ASL_CV_FILE_HAS_SWITCHED(a)       0
-#define ASL_CV_INIT_FILETREE(a,b)
+#define ASL_CV_INIT_FILETREE(a, b)
 
 #endif
 
-- 
2.25.1