Message ID | 1340727063-23870-1-git-send-email-meravs@broadcom.com |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show |
On Tue, 2012-06-26 at 19:11 +0300, Merav Sicron wrote: > This patch changes the definition of bnx2x_tests_str_arr from char to static > char. [] > diff --git a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_ethtool.c b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_ethtool.c [] > @@ -1600,7 +1600,7 @@ static int bnx2x_set_pauseparam(struct net_device *dev, > return 0; > } > > -char *bnx2x_tests_str_arr[BNX2X_NUM_TESTS_SF] = { > +static char *bnx2x_tests_str_arr[BNX2X_NUM_TESTS_SF] = { > "register_test (offline) ", > "memory_test (offline) ", > "int_loopback_test (offline)", Should probably be const too. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
From: "Merav Sicron" <meravs@broadcom.com> Date: Tue, 26 Jun 2012 19:11:03 +0300 > This patch changes the definition of bnx2x_tests_str_arr from char to static > char. This correction will also eliminate the sparse warning created in commit > cf2c1df62e065bfc15e38daf2d3479a56b320f29. > > Signed-off-by: Merav Sicron <meravs@broadcom.com> > Signed-off-by: Eilon Greenstein <eilong@broadcom.com> Applied. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_ethtool.c b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_ethtool.c index 1f8c156..bff3129 100644 --- a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_ethtool.c +++ b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_ethtool.c @@ -1600,7 +1600,7 @@ static int bnx2x_set_pauseparam(struct net_device *dev, return 0; } -char *bnx2x_tests_str_arr[BNX2X_NUM_TESTS_SF] = { +static char *bnx2x_tests_str_arr[BNX2X_NUM_TESTS_SF] = { "register_test (offline) ", "memory_test (offline) ", "int_loopback_test (offline)",