On Tue, Dec 20, 2011 at 12:26 AM, Russell King - ARM Linux
[off-list ref] wrote:

On Mon, Dec 19, 2011 at 08:03:22PM -0800, Olof Johansson wrote:

quoted

Silence a bunch of section warnings, and mark some of the init functions as
__init.

Signed-off-by: Olof Johansson <redacted>
---
?arch/arm/mach-tegra/board-harmony-pinmux.c ? | ? ?8 ++++----
?arch/arm/mach-tegra/board-paz00-pinmux.c ? ? | ? ?8 ++++----
?arch/arm/mach-tegra/board-seaboard-pinmux.c ?| ? 16 ++++++++--------
?arch/arm/mach-tegra/board-trimslice-pinmux.c | ? ?4 ++--
?4 files changed, 18 insertions(+), 18 deletions(-)

diff --git a/arch/arm/mach-tegra/board-harmony-pinmux.c b/arch/arm/mach-tegra/board-harmony-pinmux.c
index 465808c..84cfbef 100644
--- a/arch/arm/mach-tegra/board-harmony-pinmux.c
+++ b/arch/arm/mach-tegra/board-harmony-pinmux.c

@@ -25,7 +25,7 @@

?#include "board-harmony.h"
?#include "board-pinmux.h"

-static struct tegra_pingroup_config harmony_pinmux[] = {
+static __initdata struct tegra_pingroup_config harmony_pinmux[] = {

__initdata goes at the end of the declaration - just before the '='.

Thanks for pointing it out -- looks like a couple of other instances
use the wrong location as well (probably added by me in the past).
I'll respin the patch, and fix up those cases as well.

-Olof