From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Sun, 8 Jan 2012 11:40:49 +0100 Subject: [U-Boot] [PATCH 01/14] Add AES crypto library In-Reply-To: <201201080409.12513.vapier@gentoo.org> References: <1324927987-13100-1-git-send-email-sjg@chromium.org> <201201080957.48063.marek.vasut@gmail.com> <201201080409.12513.vapier@gentoo.org> Message-ID: <201201081140.49340.marek.vasut@gmail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de > On Sunday 08 January 2012 03:57:47 Marek Vasut wrote: > > > On Monday 26 December 2011 14:32:54 Simon Glass wrote: > > > > --- /dev/null > > > > +++ b/lib/aes.c > > > > > > > > +static u8 sbox[256] = { > > > > +static u8 inv_sbox[256] = { > > > > +static u8 x2_sbox[256] = { > > > > +static u8 x3_sbox[256] = { > > > > +static u8 x_time_9[256] = { > > > > +static u8 x_time_b[256] = { > > > > +static u8 x_time_d[256] = { > > > > +static u8 x_time_e[256] = { > > > > > > const > > > > > > > +static void mix_sub_columns(u8 *state) > > > > +{ > > > > ... > > > > + x3_sbox[state[2]] ^ sbox[state[7]]; > > > > > > looks bad whitespace after that ^ > > > > > > > +static u8 rcon[11] = { > > > > > > const > > > > We already have checkpatch in tools ... run it on the series. > > i don't think checkpatch catches const > -mike No, but it catches whitespaces ... M