From mboxrd@z Thu Jan 1 00:00:00 1970 From: Boris Brezillon Subject: Re: [PATCH v2 4/9] crypto: marvell - Use IPAD/OPAD constant Date: Fri, 19 May 2017 11:17:24 +0200 Message-ID: <20170519111724.3933c91d@bbrezillon> References: <20170519065331.27670-1-clabbe.montjoie@gmail.com> <20170519065331.27670-5-clabbe.montjoie@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20170519065331.27670-5-clabbe.montjoie@gmail.com> Sender: linux-crypto-owner@vger.kernel.org To: Corentin Labbe Cc: herbert@gondor.apana.org.au, davem@davemloft.net, thomas.lendacky@amd.com, gary.hook@amd.com, arno@natisbad.org, matthias.bgg@gmail.com, giovanni.cabiddu@intel.com, salvatore.benedetto@intel.com, linux-crypto@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, qat-linux@intel.com List-Id: linux-mediatek@lists.infradead.org On Fri, 19 May 2017 08:53:26 +0200 Corentin Labbe wrote: > This patch simply replace all occurrence of HMAC IPAD/OPAD value by their > define. > > Signed-off-by: Corentin Labbe Acked-by: Boris Brezillon > --- > drivers/crypto/marvell/hash.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/drivers/crypto/marvell/hash.c b/drivers/crypto/marvell/hash.c > index 77c0fb936f47..e61b08566093 100644 > --- a/drivers/crypto/marvell/hash.c > +++ b/drivers/crypto/marvell/hash.c > @@ -12,6 +12,7 @@ > * by the Free Software Foundation. > */ > > +#include > #include > #include > > @@ -1164,8 +1165,8 @@ static int mv_cesa_ahmac_pad_init(struct ahash_request *req, > memcpy(opad, ipad, blocksize); > > for (i = 0; i < blocksize; i++) { > - ipad[i] ^= 0x36; > - opad[i] ^= 0x5c; > + ipad[i] ^= HMAC_IPAD_VALUE; > + opad[i] ^= HMAC_OPAD_VALUE; > } > > return 0;