From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matthias Brugger Subject: Re: [PATCH v2 8/9] crypto: mediatek - Use IPAD/OPAD constant Date: Fri, 19 May 2017 11:11:06 +0200 Message-ID: References: <20170519065331.27670-1-clabbe.montjoie@gmail.com> <20170519065331.27670-9-clabbe.montjoie@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20170519065331.27670-9-clabbe.montjoie@gmail.com> Content-Language: en-US Sender: linux-crypto-owner@vger.kernel.org To: Corentin Labbe , herbert@gondor.apana.org.au, davem@davemloft.net, thomas.lendacky@amd.com, gary.hook@amd.com, boris.brezillon@free-electrons.com, arno@natisbad.org, giovanni.cabiddu@intel.com, salvatore.benedetto@intel.com Cc: 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 19/05/17 08:53, Corentin Labbe wrote: > This patch simply replace all occurrence of HMAC IPAD/OPAD value by their > define. > > Signed-off-by: Corentin Labbe > --- Reviewed-by: Matthias Brugger > drivers/crypto/mediatek/mtk-sha.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/drivers/crypto/mediatek/mtk-sha.c b/drivers/crypto/mediatek/mtk-sha.c > index 2226f12d1c7a..5f4f845adbb8 100644 > --- a/drivers/crypto/mediatek/mtk-sha.c > +++ b/drivers/crypto/mediatek/mtk-sha.c > @@ -12,6 +12,7 @@ > * Some ideas are from atmel-sha.c and omap-sham.c drivers. > */ > > +#include > #include > #include "mtk-platform.h" > > @@ -825,8 +826,8 @@ static int mtk_sha_setkey(struct crypto_ahash *tfm, const u8 *key, > memcpy(bctx->opad, bctx->ipad, bs); > > for (i = 0; i < bs; i++) { > - bctx->ipad[i] ^= 0x36; > - bctx->opad[i] ^= 0x5c; > + bctx->ipad[i] ^= HMAC_IPAD_VALUE; > + bctx->opad[i] ^= HMAC_OPAD_VALUE; > } > > return 0; >