Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753316AbdH2NLn (ORCPT ); Tue, 29 Aug 2017 09:11:43 -0400 Received: from terminus.zytor.com ([65.50.211.136]:51139 "EHLO terminus.zytor.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753124AbdH2NLh (ORCPT ); Tue, 29 Aug 2017 09:11:37 -0400 Date: Tue, 29 Aug 2017 06:06:55 -0700 From: tip-bot for Colin Ian King Message-ID: Cc: andriy.shevchenko@linux.intel.com, hpa@zytor.com, lukas@wunner.de, peterz@infradead.org, mingo@kernel.org, torvalds@linux-foundation.org, colin.king@canonical.com, bhelgaas@google.com, linux-kernel@vger.kernel.org, tglx@linutronix.de Reply-To: tglx@linutronix.de, linux-kernel@vger.kernel.org, colin.king@canonical.com, bhelgaas@google.com, lukas@wunner.de, hpa@zytor.com, peterz@infradead.org, mingo@kernel.org, torvalds@linux-foundation.org, andriy.shevchenko@linux.intel.com In-Reply-To: <20170825163206.23250-1-colin.king@canonical.com> References: <20170825163206.23250-1-colin.king@canonical.com> To: linux-tip-commits@vger.kernel.org Subject: [tip:x86/platform] x86/platform/intel-mid: Make several arrays static, to make code smaller Git-Commit-ID: 3308376a914b9c3f57a12072c063814403d983a8 X-Mailer: tip-git-log-daemon Robot-ID: Robot-Unsubscribe: Contact to get blacklisted from these emails MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset=UTF-8 Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2180 Lines: 56 Commit-ID: 3308376a914b9c3f57a12072c063814403d983a8 Gitweb: http://git.kernel.org/tip/3308376a914b9c3f57a12072c063814403d983a8 Author: Colin Ian King AuthorDate: Fri, 25 Aug 2017 17:32:06 +0100 Committer: Ingo Molnar CommitDate: Tue, 29 Aug 2017 13:30:16 +0200 x86/platform/intel-mid: Make several arrays static, to make code smaller Don't populate arrays on the stack, instead make them static. Makes the object code smaller by 76 bytes: Before: text data bss dec hex filename 4217 1540 128 5885 16fd arch/x86/platform/intel-mid/pwr.o After: text data bss dec hex filename 3981 1700 128 5809 16b1 arch/x86/platform/intel-mid/pwr.o Signed-off-by: Colin Ian King Reviewed-by: Andy Shevchenko Cc: Bjorn Helgaas Cc: Linus Torvalds Cc: Lukas Wunner Cc: Peter Zijlstra Cc: Thomas Gleixner Cc: kernel-janitors@vger.kernel.org Link: http://lkml.kernel.org/r/20170825163206.23250-1-colin.king@canonical.com Signed-off-by: Ingo Molnar --- arch/x86/platform/intel-mid/pwr.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/x86/platform/intel-mid/pwr.c b/arch/x86/platform/intel-mid/pwr.c index ef03852..49ec5b9 100644 --- a/arch/x86/platform/intel-mid/pwr.c +++ b/arch/x86/platform/intel-mid/pwr.c @@ -444,7 +444,7 @@ static int mid_set_initial_state(struct mid_pwr *pwr, const u32 *states) static int pnw_set_initial_state(struct mid_pwr *pwr) { /* On Penwell SRAM must stay powered on */ - const u32 states[] = { + static const u32 states[] = { 0xf00fffff, /* PM_SSC(0) */ 0xffffffff, /* PM_SSC(1) */ 0xffffffff, /* PM_SSC(2) */ @@ -455,7 +455,7 @@ static int pnw_set_initial_state(struct mid_pwr *pwr) static int tng_set_initial_state(struct mid_pwr *pwr) { - const u32 states[] = { + static const u32 states[] = { 0xffffffff, /* PM_SSC(0) */ 0xffffffff, /* PM_SSC(1) */ 0xffffffff, /* PM_SSC(2) */