From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751973AbcAAMYk (ORCPT ); Fri, 1 Jan 2016 07:24:40 -0500 Received: from mout.web.de ([212.227.17.12]:60311 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751682AbcAAMYe (ORCPT ); Fri, 1 Jan 2016 07:24:34 -0500 Subject: [PATCH 3/3] net-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> Cc: LKML , kernel-janitors@vger.kernel.org, Julia Lawall From: SF Markus Elfring Message-ID: <56866FFB.7060909@users.sourceforge.net> Date: Fri, 1 Jan 2016 13:24:27 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.4.0 MIME-Version: 1.0 In-Reply-To: <56866E7F.8080609@users.sourceforge.net> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Provags-ID: V03:K0:4wHqVNk2bQdiSl3DjJEgPfeupQk0KfoKwcqZFITI/yvmrB0uxiG GIEVAgHscjZO2KcVRJShiSnphpRLm0MkopWZ4+TZbu8rel3TCQqbx07RmBP3/tVXQjk938X k+3b0//X3G3gVy0/8EdiJpFKQFtKVOijWiUVX7Do5h/4RTmmVRreNrMCPdSpLyyMR1f7Q/n mhkcy6rNFOsubUktEQmXw== X-UI-Out-Filterresults: notjunk:1;V01:K0:Amcbxe64Dso=:tSYjftyfOFk515jnl1HS0a BlzYrLMiLVHCm3HyDWE5/Sf5pZM24C7FFQSi8dg1OZoJXN9xhUUfvMc6mj/3Kc+2h8hSUfnp3 0gr3/3DfJszcp0vBShAYiYbrca5IvlHw7ZIwGb1pv9ouHCNns4Q0s/UGY8/CQCTVR50FPbtKQ vJmAcc+P0rUCd6g9SvrR+O2W4Rmw2znkjTaER0tWnxGETcKbuZxhtA/uryXG+/OzJksw9FRhA uP2pCDIznNeYVuvma2OnLX4VjbxBxuxq9iahkhdoagzSX388N0dAPvynnRjxjA4XkWFsejMWt Jw8F3woNzk0dBYgHhQ4ZOHmyB6NmGgJwRZRgH/FU5vd1X0pEqdjKU7wYgd5fmXkAhwfSA1gc+ u4RnaBG2XCdD2nKzI9ryhXvoy66w9kK/i056bGxbMRKmKRO/iWqxOpUP/W2TlIGyGm+df+9LE zeUdWzJp8EfAFAmS73eaMdx0s9UQjCoexYJYZ7xRR9G6j2mvAvVDcN/DkZCNQftw1LKAtN5kF c5kJ7Q/9Gaek4NZ9U3iHeb/9ELISCFNf2K9f4fXFpSMK4XhSP7FgSmEyRdk5MAZs/wUID8X45 3FhfDMLIHd5TJlVdFnXbWGCNdr0qVkPDR3EYelCrfJRW1MgCbxvuaB4GzssnIutpuzqSRcuH4 vWkc7VY0ZfAbs8VeysaDf6UYItXCTM+EwhcGwXlyBzSdrCS9Lv6Z9gNqiM0oWvRPN6KnZEIWw K4utq7TY2xxz2MNhlyhg6c6hUajBY7L+miVpZuHNgZbNrN63cNJIHo/Zlz+0MECJZNfH9ibWQ Xkz+57k Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Markus Elfring Date: Fri, 1 Jan 2016 13:00:06 +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 508be89..6a7b035 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 From mboxrd@z Thu Jan 1 00:00:00 1970 From: SF Markus Elfring Date: Fri, 01 Jan 2016 12:24:27 +0000 Subject: [PATCH 3/3] net-gianfar: Extend an initialisation clause of a for loop in gfar_ethflow_to_filer_tabl Message-Id: <56866FFB.7060909@users.sourceforge.net> List-Id: References: <566ABCD9.1060404@users.sourceforge.net> <56866E7F.8080609@users.sourceforge.net> In-Reply-To: <56866E7F.8080609@users.sourceforge.net> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: netdev@vger.kernel.org, Claudiu Manoil Cc: LKML , kernel-janitors@vger.kernel.org, Julia Lawall From: Markus Elfring Date: Fri, 1 Jan 2016 13:00:06 +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 508be89..6a7b035 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