Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
References: <20181210215856.2238253-1-arnd@arndb.de> <20181210215856.2238253-4-arnd@arndb.de>
In-Reply-To: <20181210215856.2238253-4-arnd@arndb.de>
From:   Ulf Hansson <ulf.hansson@linaro.org>
Date:   Tue, 11 Dec 2018 10:39:51 +0100
Message-ID: <CAPDyKFogJv0OeaqdXBUyVAtmFqNTxDOJU_EMgMO7tFYdoQMY4A@mail.gmail.com>
Subject: Re: [PATCH 4/4] ARM: pxa: avoid section mismatch warning
To:     Arnd Bergmann <arnd@arndb.de>
Cc:     arm@kernel.org, Daniel Mack <daniel@zonque.org>,
        Haojian Zhuang <haojian.zhuang@gmail.com>,
        Robert Jarzmik <robert.jarzmik@free.fr>,
        Linus Walleij <linus.walleij@linaro.org>,
        grinberg@compulab.co.il, miquel.raynal@bootlin.com,
        Linux ARM <linux-arm-kernel@lists.infradead.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Mon, 10 Dec 2018 at 23:00, Arnd Bergmann <arnd@arndb.de> wrote:
>
> WARNING: vmlinux.o(.text+0x19f90): Section mismatch in reference from the function littleton_init_lcd() to the function .init.text:pxa_set_fb_info()
> The function littleton_init_lcd() references
> the function __init pxa_set_fb_info().
> This is often because littleton_init_lcd lacks a __init
> annotation or the annotation of pxa_set_fb_info is wrong.
>
> WARNING: vmlinux.o(.text+0xf824): Section mismatch in reference from the function zeus_register_ohci() to the function .init.text:pxa_set_ohci_info()
> The function zeus_register_ohci() references
> the function __init pxa_set_ohci_info().
> This is often because zeus_register_ohci lacks a __init
> annotation or the annotation of pxa_set_ohci_info is wrong.
>
> WARNING: vmlinux.o(.text+0xf95c): Section mismatch in reference from the function cm_x300_init_u2d() to the function .init.text:pxa3xx_set_u2d_info()
> The function cm_x300_init_u2d() references
> the function __init pxa3xx_set_u2d_info().
> This is often because cm_x300_init_u2d lacks a __init
> annotation or the annotation of pxa3xx_set_u2d_info is wrong.
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

I have queued a bunch of pxa changes via my mmc tree for gpio
descriptor conversions, by Linus Walleij.

Are those causing this problem? Should I queue this one via mmc tree as well?

Kind regards
Uffe

> ---
>  arch/arm/mach-pxa/cm-x300.c   | 2 +-
>  arch/arm/mach-pxa/littleton.c | 2 +-
>  arch/arm/mach-pxa/zeus.c      | 2 +-
>  3 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm/mach-pxa/cm-x300.c b/arch/arm/mach-pxa/cm-x300.c
> index 109fab292f94..b76b566280fa 100644
> --- a/arch/arm/mach-pxa/cm-x300.c
> +++ b/arch/arm/mach-pxa/cm-x300.c
> @@ -564,7 +564,7 @@ static struct pxa3xx_u2d_platform_data cm_x300_u2d_platform_data = {
>         .exit           = cm_x300_u2d_exit,
>  };
>
> -static void cm_x300_init_u2d(void)
> +static void __init cm_x300_init_u2d(void)
>  {
>         pxa3xx_set_u2d_info(&cm_x300_u2d_platform_data);
>  }
> diff --git a/arch/arm/mach-pxa/littleton.c b/arch/arm/mach-pxa/littleton.c
> index 8e0b60a33026..39db4898dc4a 100644
> --- a/arch/arm/mach-pxa/littleton.c
> +++ b/arch/arm/mach-pxa/littleton.c
> @@ -182,7 +182,7 @@ static struct pxafb_mach_info littleton_lcd_info = {
>         .lcd_conn               = LCD_COLOR_TFT_16BPP,
>  };
>
> -static void littleton_init_lcd(void)
> +static void __init littleton_init_lcd(void)
>  {
>         pxa_set_fb_info(NULL, &littleton_lcd_info);
>  }
> diff --git a/arch/arm/mach-pxa/zeus.c b/arch/arm/mach-pxa/zeus.c
> index 897ef59fbe0c..c411f79d4cb5 100644
> --- a/arch/arm/mach-pxa/zeus.c
> +++ b/arch/arm/mach-pxa/zeus.c
> @@ -576,7 +576,7 @@ static struct pxaohci_platform_data zeus_ohci_platform_data = {
>         .flags          = ENABLE_PORT_ALL | POWER_SENSE_LOW,
>  };
>
> -static void zeus_register_ohci(void)
> +static void __init zeus_register_ohci(void)
>  {
>         /* Port 2 is shared between host and client interface. */
>         UP2OCR = UP2OCR_HXOE | UP2OCR_HXS | UP2OCR_DMPDE | UP2OCR_DPPDE;
> --
> 2.20.0
>