Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755122Ab0KVRmq (ORCPT ); Mon, 22 Nov 2010 12:42:46 -0500 Received: from stargate.chelsio.com ([67.207.112.58]:26775 "EHLO stargate.chelsio.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753602Ab0KVRmn (ORCPT ); Mon, 22 Nov 2010 12:42:43 -0500 From: Casey Leedom Organization: Chelsio Communications, Inc. To: Joe Perches Subject: Re: [PATCH 14/62] cxgb4vf: Use static const Date: Mon, 22 Nov 2010 09:42:40 -0800 User-Agent: KMail/1.13.2 (Linux/2.6.32-25-generic; KDE/4.4.2; x86_64; ; ) Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org References: In-Reply-To: MIME-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-15" Content-Transfer-Encoding: 7bit Message-Id: <201011220942.40414.leedom@chelsio.com> X-OriginalArrivalTime: 22 Nov 2010 17:42:41.0780 (UTC) FILETIME=[A9AA4340:01CB8A6C] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1356 Lines: 36 | From: Joe Perches | Date: Saturday, November 20, 2010 06:38 pm | | Using static const generally increases object text and decreases data size. | It also generally decreases overall object size. | | text data bss dec hex filename | 10179 56 2216 12451 30a3 drivers/net/cxgb4vf/t4vf_hw.o.new | 10179 56 2216 12451 30a3 drivers/net/cxgb4vf/t4vf_hw.o.old | | Signed-off-by: Joe Perches | --- | drivers/net/cxgb4vf/t4vf_hw.c | 2 +- | 1 files changed, 1 insertions(+), 1 deletions(-) | | diff --git a/drivers/net/cxgb4vf/t4vf_hw.c b/drivers/net/cxgb4vf/t4vf_hw.c | index f7d7f97..daedf6e 100644 | --- a/drivers/net/cxgb4vf/t4vf_hw.c | +++ b/drivers/net/cxgb4vf/t4vf_hw.c | @@ -116,7 +116,7 @@ static void dump_mbox(struct adapter *adapter, const | char *tag, u32 mbox_data) int t4vf_wr_mbox_core(struct adapter *adapter, | const void *cmd, int size, void *rpl, bool sleep_ok) | { | - static int delay[] = { | + static const int delay[] = { | 1, 1, 3, 5, 10, 10, 20, 50, 100 | }; Looks okay to me. Thanks! Casey -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/