Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752854Ab0KUJNJ (ORCPT ); Sun, 21 Nov 2010 04:13:09 -0500 Received: from cpsmtpb-ews01.kpnxchange.com ([213.75.39.4]:2343 "EHLO cpsmtpb-ews01.kpnxchange.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751607Ab0KUJNH (ORCPT ); Sun, 21 Nov 2010 04:13:07 -0500 X-Greylist: delayed 849 seconds by postgrey-1.27 at vger.kernel.org; Sun, 21 Nov 2010 04:13:07 EST Message-ID: <4CE8DF4F.5050803@gmail.com> Date: Sun, 21 Nov 2010 09:58:55 +0100 From: Gertjan van Wingerde User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.12) Gecko/20101028 Lanikai/3.1.6 MIME-Version: 1.0 To: Joe Perches CC: Ivo van Doorn , "John W. Linville" , linux-wireless@vger.kernel.org, users@rt2x00.serialmonkey.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 60/62] rt2x00: Use static const References: In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit X-OriginalArrivalTime: 21 Nov 2010 08:58:56.0108 (UTC) FILETIME=[541782C0:01CB895A] X-RcptDomain: vger.kernel.org Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1524 Lines: 37 On 11/21/10 03:39, Joe Perches wrote: > Using static const generally increases object text and decreases data size. > It also generally decreases overall object size. > > text data bss dec hex filename > 40197 56 8336 48589 bdcd drivers/net/wireless/rt2x00/rt2800lib.o.new > 40205 56 8336 48597 bdd5 drivers/net/wireless/rt2x00/rt2800lib.o.old > > Signed-off-by: Joe Perches For rt2x00 this is just a tiny optimization, but for what it's worth: Acked-by: Gertjan van Wingerde > --- > drivers/net/wireless/rt2x00/rt2800lib.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/drivers/net/wireless/rt2x00/rt2800lib.c b/drivers/net/wireless/rt2x00/rt2800lib.c > index ce8df66..7563161 100644 > --- a/drivers/net/wireless/rt2x00/rt2800lib.c > +++ b/drivers/net/wireless/rt2x00/rt2800lib.c > @@ -2166,7 +2166,7 @@ static int rt2800_init_registers(struct rt2x00_dev *rt2x00dev) > SHARED_KEY_MODE_ENTRY(i), 0); > > for (i = 0; i < 256; i++) { > - u32 wcid[2] = { 0xffffffff, 0x00ffffff }; > + static const u32 wcid[2] = { 0xffffffff, 0x00ffffff }; > rt2800_register_multiwrite(rt2x00dev, MAC_WCID_ENTRY(i), > wcid, sizeof(wcid)); > -- 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/