From patchwork Thu Aug 20 13:56:39 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Corentin Labbe X-Patchwork-Id: 7044401 X-Patchwork-Delegate: herbert@gondor.apana.org.au Return-Path: X-Original-To: patchwork-linux-crypto@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.136]) by patchwork2.web.kernel.org (Postfix) with ESMTP id 4347AC05AC for ; Thu, 20 Aug 2015 13:56:47 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 5B0802057F for ; Thu, 20 Aug 2015 13:56:46 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 6377320515 for ; Thu, 20 Aug 2015 13:56:45 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751443AbbHTN4o (ORCPT ); Thu, 20 Aug 2015 09:56:44 -0400 Received: from mail-wi0-f171.google.com ([209.85.212.171]:36715 "EHLO mail-wi0-f171.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751344AbbHTN4n (ORCPT ); Thu, 20 Aug 2015 09:56:43 -0400 Received: by wicja10 with SMTP id ja10so146364379wic.1; Thu, 20 Aug 2015 06:56:42 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=subject:to:references:from:message-id:date:user-agent:mime-version :in-reply-to:content-type:content-transfer-encoding; bh=KUUSZq3xIM6WvmTyKbV20/bec9ZLSibdMajPi6M57qk=; b=ewIPZEqYTohceK8sHpKpuBVxGD7xyIN8qUfbLS4+06TDEJBTpVj7TpRiIgCLRE6Ptr mA4hrYTJzzUIA3KY+HKNgZKblHxh1vUADdgmhl0JB64WYrLTQH18sHNTPobHp/qSRIuW Zo9iVY1buYQzIOBy+KlgiBPxepDLZQf/cT5LVwcjEqxc1i0TIvECF/S04XCaSoK0FIfG D0M3/cdJEvHZ67NUpbCVcrGB9WlLEI1G5gZULN9vNReyXEV3ESY0epdOqGlc9B/GOder d1oAZW9+MMjpoJsV9s+ghdvM1rJvmqFcOgr7vOiT9hGT1DEfPA8OWXEznd6EeWvImPUx TQrw== X-Received: by 10.194.78.84 with SMTP id z20mr6380527wjw.141.1440079002009; Thu, 20 Aug 2015 06:56:42 -0700 (PDT) Received: from ?IPv6:2a01:240:fe00:142::2? (cl-323.mrs-01.fr.sixxs.net. [2a01:240:fe00:142::2]) by smtp.googlemail.com with ESMTPSA id j7sm5718754wjz.11.2015.08.20.06.56.40 (version=TLSv1/SSLv3 cipher=OTHER); Thu, 20 Aug 2015 06:56:41 -0700 (PDT) Subject: Re: un4i-ss-cipher.c warning To: Michal Suchanek , Herbert Xu , "David S. Miller" , Maxime Ripard , Linux Kernel Mailing List , linux-crypto@vger.kernel.org, "linux-arm-kernel@lists.infradead.org" References: From: Corentin LABBE Message-ID: <55D5DC97.1040209@gmail.com> Date: Thu, 20 Aug 2015 15:56:39 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.1.0 MIME-Version: 1.0 In-Reply-To: Sender: linux-crypto-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-crypto@vger.kernel.org X-Spam-Status: No, score=-7.4 required=5.0 tests=BAYES_00, DKIM_ADSP_CUSTOM_MED, DKIM_SIGNED, FREEMAIL_FROM, RCVD_IN_DNSWL_HI, RP_MATCHES_RCVD, T_DKIM_INVALID, UNPARSEABLE_RELAY autolearn=ham version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP On 19/08/2015 16:35, Michal Suchanek wrote: > Hello, > > when building a kernel with sunxi crypto driver as merged into the > sinxi-wip branch I get a compiler warning. > > I am not sure this is the latest version of the driver. It does not > seem to be in mainline yet. > > Thanks > > Michal > > In file included from /scratch/build/linux/include/linux/printk.h:277:0, > from /scratch/build/linux/include/linux/kernel.h:13, > from /scratch/build/linux/include/linux/clk.h:16, > from > /scratch/build/linux/drivers/crypto/sunxi-ss/sun4i-ss.h:15, > from > /scratch/build/linux/drivers/crypto/sunxi-ss/sun4i-ss-cipher.c:17: > /scratch/build/linux/drivers/crypto/sunxi-ss/sun4i-ss-cipher.c: In > function ‘sun4i_ss_cipher_poll’: > /scratch/build/linux/include/linux/dynamic_debug.h:86:3: warning: > ‘todo’ may be used uninitialized in this function > [-Wmaybe-uninitialized] > __dynamic_dev_dbg(&descriptor, dev, fmt, \ > ^ > /scratch/build/linux/drivers/crypto/sunxi-ss/sun4i-ss-cipher.c:144:15: > note: ‘todo’ was declared here > unsigned int todo; > Hello Thanks for the report, strange that I do not have that warning. todo is always initialized, since oleft and ileft are always > 0 at start. But setting todo to 0 is a simple fix. The following patch will suppress the warning. Regards -- >8 -- From 67f2d0749041b08c4c4ad671ab710b3b83f6dfd3 Mon Sep 17 00:00:00 2001 From: LABBE Corentin Date: Thu, 20 Aug 2015 12:18:46 +0200 Subject: [PATCH] crypto: sun4i-ss: suppress the todo uninitialized warning Signed-off-by: LABBE Corentin --- drivers/crypto/sunxi-ss/sun4i-ss-cipher.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/crypto/sunxi-ss/sun4i-ss-cipher.c b/drivers/crypto/sunxi-ss/sun4i-ss-cipher.c index e070c31..9299914 100644 --- a/drivers/crypto/sunxi-ss/sun4i-ss-cipher.c +++ b/drivers/crypto/sunxi-ss/sun4i-ss-cipher.c @@ -32,7 +32,7 @@ static int sun4i_ss_opti_poll(struct ablkcipher_request *areq) int i, err = 0; unsigned int ileft = areq->nbytes; unsigned int oleft = areq->nbytes; - unsigned int todo; + unsigned int todo = 0; struct sg_mapping_iter mi, mo; unsigned int oi, oo; /* offset for in and out */ @@ -141,7 +141,7 @@ static int sun4i_ss_cipher_poll(struct ablkcipher_request *areq) int i, err = 0; unsigned int ileft = areq->nbytes; unsigned int oleft = areq->nbytes; - unsigned int todo; + unsigned int todo = 0; struct sg_mapping_iter mi, mo; unsigned int oi, oo; /* offset for in and out */ char buf[4 * SS_RX_MAX];/* buffer for linearize SG src */