From: Cyrille Pitchen Subject: Re: [BUG] crypto: atmel-aes - erro when compiling with VERBOSE_DEBUG enable Date: Mon, 3 Oct 2016 12:20:32 +0200 Message-ID: <4a6d15ff-fe3d-c781-8db1-6c11df19334f@atmel.com> References: <20160922092629.GA5508@gondor.apana.org.au> <1474548357.12981.19.camel@inria.fr> <20161002143858.GE18268@gondor.apana.org.au> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 8bit Cc: levent demir , To: Herbert Xu Return-path: Received: from eusmtp01.atmel.com ([212.144.249.242]:55207 "EHLO eusmtp01.atmel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751581AbcJCKUh (ORCPT ); Mon, 3 Oct 2016 06:20:37 -0400 In-Reply-To: <20161002143858.GE18268@gondor.apana.org.au> Sender: linux-crypto-owner@vger.kernel.org List-ID: Hi all, Le 02/10/2016 ? 16:38, Herbert Xu a ?crit : > On Tue, Sep 27, 2016 at 06:45:18PM +0200, Cyrille Pitchen wrote: >> Hi Levent, >> >> there is a typo in the subject line: erroR. >> Also it would be better to start the summary phrase of the subject line with a >> verb: >> >> crypto: atmel-aes: fix compiler error when VERBODE_DEBUG is defined >> >> Le 22/09/2016 ? 14:45, levent demir a ?crit : >>> Fix debug function call in atmel_aes_write >>> >>> Signed-off-by: Levent DEMIR >>> --- >>> drivers/crypto/atmel-aes.c | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/drivers/crypto/atmel-aes.c b/drivers/crypto/atmel-aes.c >>> index e3d40a8..2b0f926 100644 >>> --- a/drivers/crypto/atmel-aes.c >>> +++ b/drivers/crypto/atmel-aes.c >>> @@ -317,7 +317,7 @@ static inline void atmel_aes_write(struct >>> atmel_aes_dev *dd, >>> char tmp[16]; >>> >>> dev_vdbg(dd->dev, "write 0x%08x into %s\n", value, >>> - atmel_aes_reg_name(offset, tmp)); >>> + atmel_aes_reg_name(offset, tmp, sizeof(tmp))); >> It looks like a space has been removed. > > It's been completely mangled by the mailer and cannot be applied. > I've sent a new version in this thread: https://lkml.org/lkml/2016/9/29/463 I added a Reported-by tag for Levent but if you want to use a Signed-off-by tag instead, it's fine with me! Best regards, Cyrille