From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757424AbcAOKOM (ORCPT ); Fri, 15 Jan 2016 05:14:12 -0500 Received: from mout.web.de ([212.227.15.3]:60386 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757031AbcAOKOJ (ORCPT ); Fri, 15 Jan 2016 05:14:09 -0500 Subject: [PATCH v3 3/3] gianfar: Extend an initialisation clause of a for loop in gfar_ethflow_to_filer_table() To: netdev@vger.kernel.org, Claudiu Manoil References: <566ABCD9.1060404@users.sourceforge.net> <56866E7F.8080609@users.sourceforge.net> <5698C53C.8060204@users.sourceforge.net> Cc: LKML , kernel-janitors@vger.kernel.org, Julia Lawall From: SF Markus Elfring Message-ID: <5698C66A.9000003@users.sourceforge.net> Date: Fri, 15 Jan 2016 11:14:02 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 MIME-Version: 1.0 In-Reply-To: <5698C53C.8060204@users.sourceforge.net> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Provags-ID: V03:K0:pwPW51n2eZnesOPGwcd5GKWjReEQXa/ZHL5uuVkcfz87PIm9s1Z 8bRPRaZt7NjcdNINLoUMpjDzbv5vHl8tKfEGOwExtz8ZJaRiMqLzlGyS+BO7Ot6yS2KB+v8 eTJMIKF0deCPbNAzpDYXpmQizXx4RPBHK5uPtexqkhd926o3cWXBiCoppUndaTeRiGImpH/ V/da3PU5/XQb96F2OzblQ== X-UI-Out-Filterresults: notjunk:1;V01:K0:HNONocx3lpM=:EWMAjSxSVYkGuOjV0hQeQE +y7IE3bEMQ1CEQJT/F4CiwjbsdKC1b2zKUwy4uqLgOVR/gRPKMRICRE86QTXSb9F238M1vLpK RGJR2BGuGYp4KtBKAZkjrkh/aluT0iCPSehqtfXtMNh+OAf6Vo2fiZtNXfp+wCsCeu5qk0TLB EZBK9ai1qizDRUbzGmULu4g7cYTjZf71pFyFfdjaT2KA5CO65FROMOArfiTZLgAGfBfGLtWaS WfbciUrYX8uwlw8IUPjcJD333i+WSTc3Xt2jCaITFZjZXxyyaMOQG1UWC0f1mszszgylIIO4F B+yK3CYBM60bgkKTreed/BbrGMHngqABGgmbQ28d8gUKE5rM0iWxlMMs/yr+Nuy7FJNWDnODb yfzo8kXgFKvLpaqODdtBI+onrQA8Rez0iUxW4CsHKzDNrnIMFVtgu2+Y8ePcF7ggGwlgvJOOY Ne2zTFeIoQV/bN3kHGwzAWUS5ebkQBSrqhrRYPy5+6RcWcDbrfioFhzrXq271QbcdxekEAU4y A8fLcNC8yh3bTk+AjA6qk1T1XoqgpZOGlEYWx4sSgJ+GZok1SJmP1+aQTqnFmpDCAxRVqEgqi 0wh9Z+n1A17w3tWcLOYoKTr/kB4h5UNSrvB2toyBAvgFNUzHE+v23nSbA//LZBMVQa6WOzloN ctWHNUYriFlyERtqFyC3MiDj1mEmXU9iHy1yvDr3kwxSvLSZaZargntqsoMUMGguuw5VZNbI7 7FvSIAS2ylL2BsVIbJjYBRpCDdRan89CHrzfTDXmigUCIu468ecRcyga3jPDA67Lyw5j2n5br bJBQiCp Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Markus Elfring Date: Fri, 15 Jan 2016 10:50:34 +0100 Move the assignment for the variable "j" from the beginning into an initialisation clause of a for loop. Signed-off-by: Markus Elfring --- drivers/net/ethernet/freescale/gianfar_ethtool.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/net/ethernet/freescale/gianfar_ethtool.c b/drivers/net/ethernet/freescale/gianfar_ethtool.c index 8302f7d..2162adc 100644 --- a/drivers/net/ethernet/freescale/gianfar_ethtool.c +++ b/drivers/net/ethernet/freescale/gianfar_ethtool.c @@ -772,8 +772,7 @@ static int gfar_ethflow_to_filer_table(struct gfar_private *priv, u64 ethflow, unsigned int cmp_rqfpr; unsigned int *local_rqfpr; unsigned int *local_rqfcr; - int i, k, l; - int j = MAX_FILER_IDX; + int i, j, k, l; int ret = 1; local_rqfpr = kmalloc_array(MAX_FILER_IDX + 1, sizeof(unsigned int), @@ -807,7 +806,7 @@ static int gfar_ethflow_to_filer_table(struct gfar_private *priv, u64 ethflow, goto free_fcr; } - for (i = 0; i < MAX_FILER_IDX + 1; i++) { + for (i = 0, j = MAX_FILER_IDX; i < MAX_FILER_IDX + 1; i++) { local_rqfpr[j] = priv->ftp_rqfpr[i]; local_rqfcr[j] = priv->ftp_rqfcr[i]; j--; -- 2.6.3