Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753411AbdIFV1i (ORCPT ); Wed, 6 Sep 2017 17:27:38 -0400 Received: from mail-qk0-f195.google.com ([209.85.220.195]:34393 "EHLO mail-qk0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752354AbdIFV1h (ORCPT ); Wed, 6 Sep 2017 17:27:37 -0400 X-Google-Smtp-Source: AOwi7QAgWN8BaLQR2rH7OgXOvaagNtSqLVuyoD8pO5bFMUKBuFeS77XQx84xjDC1YtvuHKU47vxLUUbeitm8OYSnaVs= MIME-Version: 1.0 In-Reply-To: References: <1504732426-9765-1-git-send-email-srishtishar@gmail.com> From: Srishti Sharma Date: Thu, 7 Sep 2017 02:57:35 +0530 Message-ID: Subject: Re: [Outreachy kernel] [PATCH v3] Staging: ccree: ssi_cipher.c: Remove unused variable. To: Julia Lawall Cc: Greg KH , Gilad Ben-Yossef , linux-crypto@vger.kernel.org, driverdev-devel@linuxdriverproject.org, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org, outreachy-kernel@googlegroups.com Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2629 Lines: 72 On Thu, Sep 7, 2017 at 2:56 AM, Julia Lawall wrote: > > > On Thu, 7 Sep 2017, Srishti Sharma wrote: > >> On Thu, Sep 7, 2017 at 2:47 AM, Julia Lawall wrote: >> > >> > >> > On Thu, 7 Sep 2017, Srishti Sharma wrote: >> > >> >> Remove local variable inflight_counter ,as it is never used. >> > >> > "counter ,as" -> "counter, as" >> > >> >> Signed-off-by: Srishti Sharma >> >> --- >> >> Changes in v3: >> >> - There was no longer a need to make the comment more comprehensible as >> >> I have deleted the variable associated with it because it is unused . >> >> drivers/staging/ccree/ssi_cipher.c | 5 +---- >> >> 1 file changed, 1 insertion(+), 4 deletions(-) >> >> >> >> diff --git a/drivers/staging/ccree/ssi_cipher.c b/drivers/staging/ccree/ssi_cipher.c >> >> index fec2faa..609ebe4 100644 >> >> --- a/drivers/staging/ccree/ssi_cipher.c >> >> +++ b/drivers/staging/ccree/ssi_cipher.c >> >> @@ -702,10 +702,7 @@ static int ssi_blkcipher_complete(struct device *dev, >> >> >> >> ssi_buffer_mgr_unmap_blkcipher_request(dev, req_ctx, ivsize, src, dst); >> >> >> >> - /* >> >> - * Save inflight_counter in a local variable. >> >> - */ >> >> - inflight_counter = ctx_p->drvdata->inflight_counter; >> >> + >> > >> > This is a patch on your previous patch. It should be a patch on the >> > original code. Also, don't add a new blank line. Just remove the line >> > completely. >> >> so , when I resend it should I version it as v3 or send as a new >> patch as it is now fixing a different problem ? > > v3 (or maybe now v4?) would be fine. It's still the same code that is > under consideration. Then it will be clear that he previosu patches are > not needed. Okay , Thanks ! Regards, Srishti > > julia > >> Regards, >> Srishti >> > >> > julia >> > >> >> /*Decrease the inflight counter*/ >> >> if (ctx_p->flow_mode == BYPASS && ctx_p->drvdata->inflight_counter > 0) >> >> ctx_p->drvdata->inflight_counter--; >> >> -- >> >> 2.7.4 >> >> >> >> -- >> >> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group. >> >> To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com. >> >> To post to this group, send email to outreachy-kernel@googlegroups.com. >> >> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/1504732426-9765-1-git-send-email-srishtishar%40gmail.com. >> >> For more options, visit https://groups.google.com/d/optout. >> >> >>