[CRYPTO] aes-generic: Coding style cleanup

Signed-off-by: Sebastian Siewior <sebastian@breakpoint.cc>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>

1 files changed, 176 insertions, 149 deletions

diff --git a/crypto/aes_generic.c b/crypto/aes_generic.c
index 6683260475f9..df8df4d346d2 100644
--- a/crypto/aes_generic.c
+++ b/crypto/aes_generic.c
@@ -63,8 +63,7 @@
 /*
  * #define byte(x, nr) ((unsigned char)((x) >> (nr*8))) 
  */
-static inline u8
-byte(const u32 x, const unsigned n)
+static inline u8 byte(const u32 x, const unsigned n)
 {
         return x >> (n << 3);
 }
@@ -88,55 +87,25 @@ static u32 it_tab[4][256];
 static u32 fl_tab[4][256];
 static u32 il_tab[4][256];
 
-static inline u8 __init
-f_mult (u8 a, u8 b)
+static inline u8 __init f_mult(u8 a, u8 b)
 {
         u8 aa = log_tab[a], cc = aa + log_tab[b];
 
         return pow_tab[cc + (cc < aa ? 1 : 0)];
 }
 
-#define ff_mult(a,b)    (a && b ? f_mult(a, b) : 0)
-
-#define f_rn(bo, bi, n, k)                                      \
-    bo[n] =  ft_tab[0][byte(bi[n],0)] ^                         \
-             ft_tab[1][byte(bi[(n + 1) & 3],1)] ^               \
-             ft_tab[2][byte(bi[(n + 2) & 3],2)] ^               \
-             ft_tab[3][byte(bi[(n + 3) & 3],3)] ^ *(k + n)
-
-#define i_rn(bo, bi, n, k)                                      \
-    bo[n] =  it_tab[0][byte(bi[n],0)] ^                         \
-             it_tab[1][byte(bi[(n + 3) & 3],1)] ^               \
-             it_tab[2][byte(bi[(n + 2) & 3],2)] ^               \
-             it_tab[3][byte(bi[(n + 1) & 3],3)] ^ *(k + n)
-
-#define ls_box(x)                               \
-    ( fl_tab[0][byte(x, 0)] ^                   \
-      fl_tab[1][byte(x, 1)] ^                   \
-      fl_tab[2][byte(x, 2)] ^                   \
-      fl_tab[3][byte(x, 3)] )
-
-#define f_rl(bo, bi, n, k)                                      \
-    bo[n] =  fl_tab[0][byte(bi[n],0)] ^                         \
-             fl_tab[1][byte(bi[(n + 1) & 3],1)] ^               \
-             fl_tab[2][byte(bi[(n + 2) & 3],2)] ^               \
-             fl_tab[3][byte(bi[(n + 3) & 3],3)] ^ *(k + n)
-
-#define i_rl(bo, bi, n, k)                                      \
-    bo[n] =  il_tab[0][byte(bi[n],0)] ^                         \
-             il_tab[1][byte(bi[(n + 3) & 3],1)] ^               \
-             il_tab[2][byte(bi[(n + 2) & 3],2)] ^               \
-             il_tab[3][byte(bi[(n + 1) & 3],3)] ^ *(k + n)
-
-static void __init
-gen_tabs (void)
+#define ff_mult(a, b)   (a && b ? f_mult(a, b) : 0)
+
+static void __init gen_tabs(void)
 {
         u32 i, t;
         u8 p, q;
 
-        /* log and power tables for GF(2**8) finite field with
-           0x011b as modular polynomial - the simplest primitive
-           root is 0x03, used here to generate the tables */
+        /*
+         * log and power tables for GF(2**8) finite field with
+         * 0x011b as modular polynomial - the simplest primitive
+         * root is 0x03, used here to generate the tables
+         */
 
         for (i = 0, p = 1; i < 256; ++i) {
                 pow_tab[i] = (u8) p;
@@ -170,9 +139,9 @@ gen_tabs (void)
                 fl_tab[2][i] = rol32(t, 16);
                 fl_tab[3][i] = rol32(t, 24);
 
-                t = ((u32) ff_mult (2, p)) |
+                t = ((u32) ff_mult(2, p)) |
                     ((u32) p << 8) |
-                    ((u32) p << 16) | ((u32) ff_mult (3, p) << 24);
+                    ((u32) p << 16) | ((u32) ff_mult(3, p) << 24);
 
                 ft_tab[0][i] = t;
                 ft_tab[1][i] = rol32(t, 8);
@@ -187,10 +156,10 @@ gen_tabs (void)
                 il_tab[2][i] = rol32(t, 16);
                 il_tab[3][i] = rol32(t, 24);
 
-                t = ((u32) ff_mult (14, p)) |
-                    ((u32) ff_mult (9, p) << 8) |
-                    ((u32) ff_mult (13, p) << 16) |
-                    ((u32) ff_mult (11, p) << 24);
+                t = ((u32) ff_mult(14, p)) |
+                    ((u32) ff_mult(9, p) << 8) |
+                    ((u32) ff_mult(13, p) << 16) |
+                    ((u32) ff_mult(11, p) << 24);
 
                 it_tab[0][i] = t;
                 it_tab[1][i] = rol32(t, 8);
@@ -199,53 +168,80 @@ gen_tabs (void)
         }
 }
 
-#define star_x(x) (((x) & 0x7f7f7f7f) << 1) ^ ((((x) & 0x80808080) >> 7) * 0x1b)
-
-#define imix_col(y,x)       \
-    u   = star_x(x);        \
-    v   = star_x(u);        \
-    w   = star_x(v);        \
-    t   = w ^ (x);          \
-   (y)  = u ^ v ^ w;        \
-   (y) ^= ror32(u ^ t,  8) ^ \
-          ror32(v ^ t, 16) ^ \
-          ror32(t,24)
-
 /* initialise the key schedule from the user supplied key */
 
-#define loop4(i)                                    \
-{   t = ror32(t,  8); t = ls_box(t) ^ rco_tab[i];    \
-    t ^= E_KEY[4 * i];     E_KEY[4 * i + 4] = t;    \
-    t ^= E_KEY[4 * i + 1]; E_KEY[4 * i + 5] = t;    \
-    t ^= E_KEY[4 * i + 2]; E_KEY[4 * i + 6] = t;    \
-    t ^= E_KEY[4 * i + 3]; E_KEY[4 * i + 7] = t;    \
-}
-
-#define loop6(i)                                    \
-{   t = ror32(t,  8); t = ls_box(t) ^ rco_tab[i];    \
-    t ^= E_KEY[6 * i];     E_KEY[6 * i + 6] = t;    \
-    t ^= E_KEY[6 * i + 1]; E_KEY[6 * i + 7] = t;    \
-    t ^= E_KEY[6 * i + 2]; E_KEY[6 * i + 8] = t;    \
-    t ^= E_KEY[6 * i + 3]; E_KEY[6 * i + 9] = t;    \
-    t ^= E_KEY[6 * i + 4]; E_KEY[6 * i + 10] = t;   \
-    t ^= E_KEY[6 * i + 5]; E_KEY[6 * i + 11] = t;   \
-}
+#define star_x(x) (((x) & 0x7f7f7f7f) << 1) ^ ((((x) & 0x80808080) >> 7) * 0x1b)
 
-#define loop8(i)                                    \
-{   t = ror32(t,  8); ; t = ls_box(t) ^ rco_tab[i];  \
-    t ^= E_KEY[8 * i];     E_KEY[8 * i + 8] = t;    \
-    t ^= E_KEY[8 * i + 1]; E_KEY[8 * i + 9] = t;    \
-    t ^= E_KEY[8 * i + 2]; E_KEY[8 * i + 10] = t;   \
-    t ^= E_KEY[8 * i + 3]; E_KEY[8 * i + 11] = t;   \
-    t  = E_KEY[8 * i + 4] ^ ls_box(t);    \
-    E_KEY[8 * i + 12] = t;                \
-    t ^= E_KEY[8 * i + 5]; E_KEY[8 * i + 13] = t;   \
-    t ^= E_KEY[8 * i + 6]; E_KEY[8 * i + 14] = t;   \
-    t ^= E_KEY[8 * i + 7]; E_KEY[8 * i + 15] = t;   \
-}
+#define imix_col(y,x)   do {            \
+        u       = star_x(x);            \
+        v       = star_x(u);            \
+        w       = star_x(v);            \
+        t       = w ^ (x);              \
+        (y)     = u ^ v ^ w;            \
+        (y)     ^= ror32(u ^ t, 8) ^    \
+                ror32(v ^ t, 16) ^      \
+                ror32(t, 24);           \
+} while (0)
+
+#define ls_box(x)               \
+        fl_tab[0][byte(x, 0)] ^ \
+        fl_tab[1][byte(x, 1)] ^ \
+        fl_tab[2][byte(x, 2)] ^ \
+        fl_tab[3][byte(x, 3)]
+
+#define loop4(i)        do {            \
+        t = ror32(t, 8);                \
+        t = ls_box(t) ^ rco_tab[i];     \
+        t ^= E_KEY[4 * i];              \
+        E_KEY[4 * i + 4] = t;           \
+        t ^= E_KEY[4 * i + 1];          \
+        E_KEY[4 * i + 5] = t;           \
+        t ^= E_KEY[4 * i + 2];          \
+        E_KEY[4 * i + 6] = t;           \
+        t ^= E_KEY[4 * i + 3];          \
+        E_KEY[4 * i + 7] = t;           \
+} while (0)
+
+#define loop6(i)        do {            \
+        t = ror32(t, 8);                \
+        t = ls_box(t) ^ rco_tab[i];     \
+        t ^= E_KEY[6 * i];              \
+        E_KEY[6 * i + 6] = t;           \
+        t ^= E_KEY[6 * i + 1];          \
+        E_KEY[6 * i + 7] = t;           \
+        t ^= E_KEY[6 * i + 2];          \