linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/4] ARM: omap2: avoid section mismatch warning
@ 2018-12-10 21:58 Arnd Bergmann
  2018-12-10 21:58 ` [PATCH 2/4] ARM: tegra: " Arnd Bergmann
                   ` (5 more replies)
  0 siblings, 6 replies; 17+ messages in thread
From: Arnd Bergmann @ 2018-12-10 21:58 UTC (permalink / raw)
  To: arm, Kevin Hilman, Tony Lindgren
  Cc: Arnd Bergmann, Dave Gerlach, Keerthy, Tero Kristo,
	Santosh Shilimkar, linux-omap, linux-arm-kernel, linux-kernel

WARNING: vmlinux.o(.text+0x27530): Section mismatch in reference from the function am43xx_suspend_init() to the function .init.text:am43xx_map_scu()
The function am43xx_suspend_init() references
the function __init am43xx_map_scu().
This is often because am43xx_suspend_init lacks a __init
annotation or the annotation of am43xx_map_scu is wrong.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
 arch/arm/mach-omap2/pm33xx-core.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/mach-omap2/pm33xx-core.c b/arch/arm/mach-omap2/pm33xx-core.c
index f4971e4a86b2..724cf5774a6c 100644
--- a/arch/arm/mach-omap2/pm33xx-core.c
+++ b/arch/arm/mach-omap2/pm33xx-core.c
@@ -28,7 +28,7 @@ static struct clockdomain *gfx_l4ls_clkdm;
 static void __iomem *scu_base;
 static struct omap_hwmod *rtc_oh;
 
-static int __init am43xx_map_scu(void)
+static int am43xx_map_scu(void)
 {
 	scu_base = ioremap(scu_a9_get_base(), SZ_256);
 
-- 
2.20.0


^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 2/4] ARM: tegra: avoid section mismatch warning
  2018-12-10 21:58 [PATCH 1/4] ARM: omap2: avoid section mismatch warning Arnd Bergmann
@ 2018-12-10 21:58 ` Arnd Bergmann
  2018-12-11 10:05   ` Thierry Reding
  2018-12-10 21:58 ` [PATCH 3/4] ARM: ks8695: fix " Arnd Bergmann
                   ` (4 subsequent siblings)
  5 siblings, 1 reply; 17+ messages in thread
From: Arnd Bergmann @ 2018-12-10 21:58 UTC (permalink / raw)
  To: arm, Thierry Reding, Jonathan Hunter, Arnd Bergmann
  Cc: linux-arm-kernel, linux-tegra, linux-kernel

WARNING: vmlinux.o(.text+0x39ecc): Section mismatch in reference from the function tegra114_gic_cpu_pm_registration() to the (unknown reference) .init.rodata:(unknown)
The function tegra114_gic_cpu_pm_registration() references
the (unknown reference) __initconst (unknown).
This is often because tegra114_gic_cpu_pm_registration lacks a __initconst
annotation or the annotation of (unknown) is wrong.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
 arch/arm/mach-tegra/irq.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/mach-tegra/irq.c b/arch/arm/mach-tegra/irq.c
index a69b22d37eed..a186ab663b0b 100644
--- a/arch/arm/mach-tegra/irq.c
+++ b/arch/arm/mach-tegra/irq.c
@@ -72,7 +72,7 @@ static const struct of_device_id tegra114_dt_gic_match[] __initconst = {
 	{ }
 };
 
-static void tegra114_gic_cpu_pm_registration(void)
+static void __init tegra114_gic_cpu_pm_registration(void)
 {
 	struct device_node *dn;
 
@@ -85,7 +85,7 @@ static void tegra114_gic_cpu_pm_registration(void)
 	cpu_pm_register_notifier(&tegra_gic_notifier_block);
 }
 #else
-static void tegra114_gic_cpu_pm_registration(void) { }
+static void __init tegra114_gic_cpu_pm_registration(void) { }
 #endif
 
 static const struct of_device_id tegra_ictlr_match[] __initconst = {
-- 
2.20.0


^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 3/4] ARM: ks8695: fix section mismatch warning
  2018-12-10 21:58 [PATCH 1/4] ARM: omap2: avoid section mismatch warning Arnd Bergmann
  2018-12-10 21:58 ` [PATCH 2/4] ARM: tegra: " Arnd Bergmann
@ 2018-12-10 21:58 ` Arnd Bergmann
  2018-12-12 21:56   ` Olof Johansson
  2018-12-10 21:58 ` [PATCH 4/4] ARM: pxa: avoid " Arnd Bergmann
                   ` (3 subsequent siblings)
  5 siblings, 1 reply; 17+ messages in thread
From: Arnd Bergmann @ 2018-12-10 21:58 UTC (permalink / raw)
  To: arm, Greg Ungerer, Robert Jarzmik, Wolfram Sang, James Hogan,
	Arnd Bergmann
  Cc: Lee Jones, linux-arm-kernel, linux-kernel

WARNING: vmlinux.o(.text+0x13250): Section mismatch in reference from the function acs5k_i2c_init() to the (unknown reference) .init.data:(unknown)
The function acs5k_i2c_init() references
the (unknown reference) __initdata (unknown).
This is often because acs5k_i2c_init lacks a __initdata
annotation or the annotation of (unknown) is wrong.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
 arch/arm/mach-ks8695/board-acs5k.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/mach-ks8695/board-acs5k.c b/arch/arm/mach-ks8695/board-acs5k.c
index ef835d82cdb9..5783062224c3 100644
--- a/arch/arm/mach-ks8695/board-acs5k.c
+++ b/arch/arm/mach-ks8695/board-acs5k.c
@@ -100,7 +100,7 @@ static struct i2c_board_info acs5k_i2c_devs[] __initdata = {
 	},
 };
 
-static void acs5k_i2c_init(void)
+static void __init acs5k_i2c_init(void)
 {
 	/* The gpio interface */
 	gpiod_add_lookup_table(&acs5k_i2c_gpiod_table);
-- 
2.20.0


^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 4/4] ARM: pxa: avoid section mismatch warning
  2018-12-10 21:58 [PATCH 1/4] ARM: omap2: avoid section mismatch warning Arnd Bergmann
  2018-12-10 21:58 ` [PATCH 2/4] ARM: tegra: " Arnd Bergmann
  2018-12-10 21:58 ` [PATCH 3/4] ARM: ks8695: fix " Arnd Bergmann
@ 2018-12-10 21:58 ` Arnd Bergmann
  2018-12-11  9:39   ` Ulf Hansson
  2018-12-12 21:56   ` Olof Johansson
  2018-12-10 22:09 ` [PATCH 1/4] ARM: omap2: " Tony Lindgren
                   ` (2 subsequent siblings)
  5 siblings, 2 replies; 17+ messages in thread
From: Arnd Bergmann @ 2018-12-10 21:58 UTC (permalink / raw)
  To: arm, Daniel Mack, Haojian Zhuang, Robert Jarzmik
  Cc: Arnd Bergmann, Linus Walleij, Ulf Hansson, Igor Grinberg,
	Miquel Raynal, linux-arm-kernel, linux-kernel

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>
---
 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


^ permalink raw reply related	[flat|nested] 17+ messages in thread

* Re: [PATCH 1/4] ARM: omap2: avoid section mismatch warning
  2018-12-10 21:58 [PATCH 1/4] ARM: omap2: avoid section mismatch warning Arnd Bergmann
                   ` (2 preceding siblings ...)
  2018-12-10 21:58 ` [PATCH 4/4] ARM: pxa: avoid " Arnd Bergmann
@ 2018-12-10 22:09 ` Tony Lindgren
  2018-12-12 22:05   ` Olof Johansson
  2018-12-11  2:11 ` Santosh Shilimkar
  2018-12-11 18:12 ` Kevin Hilman
  5 siblings, 1 reply; 17+ messages in thread
From: Tony Lindgren @ 2018-12-10 22:09 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: arm, Kevin Hilman, Dave Gerlach, Keerthy, Tero Kristo,
	Santosh Shilimkar, linux-omap, linux-arm-kernel, linux-kernel

* Arnd Bergmann <arnd@arndb.de> [181210 21:59]:
> WARNING: vmlinux.o(.text+0x27530): Section mismatch in reference from the function am43xx_suspend_init() to the function .init.text:am43xx_map_scu()
> The function am43xx_suspend_init() references
> the function __init am43xx_map_scu().
> This is often because am43xx_suspend_init lacks a __init
> annotation or the annotation of am43xx_map_scu is wrong.

Acked-by: Tony Lindgren <tony@atomide.com>

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH 1/4] ARM: omap2: avoid section mismatch warning
  2018-12-10 21:58 [PATCH 1/4] ARM: omap2: avoid section mismatch warning Arnd Bergmann
                   ` (3 preceding siblings ...)
  2018-12-10 22:09 ` [PATCH 1/4] ARM: omap2: " Tony Lindgren
@ 2018-12-11  2:11 ` Santosh Shilimkar
  2018-12-11 18:12 ` Kevin Hilman
  5 siblings, 0 replies; 17+ messages in thread
From: Santosh Shilimkar @ 2018-12-11  2:11 UTC (permalink / raw)
  To: Arnd Bergmann, arm, Kevin Hilman, Tony Lindgren
  Cc: Dave Gerlach, Keerthy, Tero Kristo, Santosh Shilimkar,
	linux-omap, linux-arm-kernel, linux-kernel

On 12/10/2018 1:58 PM, Arnd Bergmann wrote:
> WARNING: vmlinux.o(.text+0x27530): Section mismatch in reference from the function am43xx_suspend_init() to the function .init.text:am43xx_map_scu()
> The function am43xx_suspend_init() references
> the function __init am43xx_map_scu().
> This is often because am43xx_suspend_init lacks a __init
> annotation or the annotation of am43xx_map_scu is wrong.
> 
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> ---
Acked-by: Santosh Shilimkar <ssantosh@kernel.org>

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH 4/4] ARM: pxa: avoid section mismatch warning
  2018-12-10 21:58 ` [PATCH 4/4] ARM: pxa: avoid " Arnd Bergmann
@ 2018-12-11  9:39   ` Ulf Hansson
  2018-12-11 10:10     ` Arnd Bergmann
  2018-12-11 10:44     ` Linus Walleij
  2018-12-12 21:56   ` Olof Johansson
  1 sibling, 2 replies; 17+ messages in thread
From: Ulf Hansson @ 2018-12-11  9:39 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: arm, Daniel Mack, Haojian Zhuang, Robert Jarzmik, Linus Walleij,
	grinberg, miquel.raynal, Linux ARM, Linux Kernel Mailing List

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
>

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH 2/4] ARM: tegra: avoid section mismatch warning
  2018-12-10 21:58 ` [PATCH 2/4] ARM: tegra: " Arnd Bergmann
@ 2018-12-11 10:05   ` Thierry Reding
  2018-12-11 10:08     ` Arnd Bergmann
  0 siblings, 1 reply; 17+ messages in thread
From: Thierry Reding @ 2018-12-11 10:05 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: arm, Jonathan Hunter, linux-arm-kernel, linux-tegra, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 955 bytes --]

On Mon, Dec 10, 2018 at 10:58:37PM +0100, Arnd Bergmann wrote:
> WARNING: vmlinux.o(.text+0x39ecc): Section mismatch in reference from the function tegra114_gic_cpu_pm_registration() to the (unknown reference) .init.rodata:(unknown)
> The function tegra114_gic_cpu_pm_registration() references
> the (unknown reference) __initconst (unknown).
> This is often because tegra114_gic_cpu_pm_registration lacks a __initconst
> annotation or the annotation of (unknown) is wrong.
> 
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> ---
>  arch/arm/mach-tegra/irq.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

I'm not seeing these. Do I need to enable any special options to trigger
these? Or are these just so new that I haven't run into them yet?

In either case, I see that this is called by tegra_init_irq() and that
does have an __init annotation, so this looks good to me:

Acked-by: Thierry Reding <treding@nvidia.com>

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH 2/4] ARM: tegra: avoid section mismatch warning
  2018-12-11 10:05   ` Thierry Reding
@ 2018-12-11 10:08     ` Arnd Bergmann
  2018-12-11 10:28       ` Thierry Reding
  0 siblings, 1 reply; 17+ messages in thread
From: Arnd Bergmann @ 2018-12-11 10:08 UTC (permalink / raw)
  To: Thierry Reding
  Cc: arm-soc, Jonathan Hunter, Linux ARM,
	open list:TEGRA ARCHITECTURE SUPPORT, Linux Kernel Mailing List

On Tue, Dec 11, 2018 at 11:05 AM Thierry Reding
<thierry.reding@gmail.com> wrote:
>
> On Mon, Dec 10, 2018 at 10:58:37PM +0100, Arnd Bergmann wrote:
> > WARNING: vmlinux.o(.text+0x39ecc): Section mismatch in reference from the function tegra114_gic_cpu_pm_registration() to the (unknown reference) .init.rodata:(unknown)
> > The function tegra114_gic_cpu_pm_registration() references
> > the (unknown reference) __initconst (unknown).
> > This is often because tegra114_gic_cpu_pm_registration lacks a __initconst
> > annotation or the annotation of (unknown) is wrong.
> >
> > Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> > ---
> >  arch/arm/mach-tegra/irq.c | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
>
> I'm not seeing these. Do I need to enable any special options to trigger
> these? Or are these just so new that I haven't run into them yet?

My guess would be that this is a result of the CONFIG_NO_AUTO_INLINE
option that is now available in linux-next.

       Arnd

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH 4/4] ARM: pxa: avoid section mismatch warning
  2018-12-11  9:39   ` Ulf Hansson
@ 2018-12-11 10:10     ` Arnd Bergmann
  2018-12-11 10:44     ` Linus Walleij
  1 sibling, 0 replies; 17+ messages in thread
From: Arnd Bergmann @ 2018-12-11 10:10 UTC (permalink / raw)
  To: Ulf Hansson
  Cc: arm-soc, Daniel Mack, Haojian Zhuang, Robert Jarzmik,
	Linus Walleij, grinberg, Miquel Raynal, Linux ARM,
	Linux Kernel Mailing List

On Tue, Dec 11, 2018 at 10:40 AM Ulf Hansson <ulf.hansson@linaro.org> wrote:
>
> 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?

Sorry for the lack of description in the changelog. I'm fairly sure that these
are old bugs that only appeared now if you build with CONFIG_NO_AUTO_INLINE,
we should take these through arm-soc, and I should probably add an explanation.

      Arnd

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH 2/4] ARM: tegra: avoid section mismatch warning
  2018-12-11 10:08     ` Arnd Bergmann
@ 2018-12-11 10:28       ` Thierry Reding
  2018-12-12 21:57         ` Olof Johansson
  0 siblings, 1 reply; 17+ messages in thread
From: Thierry Reding @ 2018-12-11 10:28 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: arm-soc, Jonathan Hunter, Linux ARM,
	open list:TEGRA ARCHITECTURE SUPPORT, Linux Kernel Mailing List

[-- Attachment #1: Type: text/plain, Size: 1295 bytes --]

On Tue, Dec 11, 2018 at 11:08:53AM +0100, Arnd Bergmann wrote:
> On Tue, Dec 11, 2018 at 11:05 AM Thierry Reding
> <thierry.reding@gmail.com> wrote:
> >
> > On Mon, Dec 10, 2018 at 10:58:37PM +0100, Arnd Bergmann wrote:
> > > WARNING: vmlinux.o(.text+0x39ecc): Section mismatch in reference from the function tegra114_gic_cpu_pm_registration() to the (unknown reference) .init.rodata:(unknown)
> > > The function tegra114_gic_cpu_pm_registration() references
> > > the (unknown reference) __initconst (unknown).
> > > This is often because tegra114_gic_cpu_pm_registration lacks a __initconst
> > > annotation or the annotation of (unknown) is wrong.
> > >
> > > Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> > > ---
> > >  arch/arm/mach-tegra/irq.c | 4 ++--
> > >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > I'm not seeing these. Do I need to enable any special options to trigger
> > these? Or are these just so new that I haven't run into them yet?
> 
> My guess would be that this is a result of the CONFIG_NO_AUTO_INLINE
> option that is now available in linux-next.

Okay, that would explain. It's not set in my configurations currently.

Do you want to apply this to ARM-SoC directly with my Acked-by, or
should I queue it up for v4.22?

Thierry

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH 4/4] ARM: pxa: avoid section mismatch warning
  2018-12-11  9:39   ` Ulf Hansson
  2018-12-11 10:10     ` Arnd Bergmann
@ 2018-12-11 10:44     ` Linus Walleij
  1 sibling, 0 replies; 17+ messages in thread
From: Linus Walleij @ 2018-12-11 10:44 UTC (permalink / raw)
  To: Ulf Hansson
  Cc: Arnd Bergmann, arm-soc, Daniel Mack, Haojian Zhuang,
	Robert Jarzmik, Igor Grinberg, Miquèl Raynal, Linux ARM,
	linux-kernel, Mark Brown

On Tue, Dec 11, 2018 at 10:40 AM Ulf Hansson <ulf.hansson@linaro.org> wrote:
> 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?

Nopes just correlation without causation...

I also made a bunch of patches to PXA through the regulator tree,
those are not related either.

It *seems* they touch so different code lines that it shouldn't
be a problem to merge orthogonally in ARM SoC.

Yours,
Linus Walleij

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH 1/4] ARM: omap2: avoid section mismatch warning
  2018-12-10 21:58 [PATCH 1/4] ARM: omap2: avoid section mismatch warning Arnd Bergmann
                   ` (4 preceding siblings ...)
  2018-12-11  2:11 ` Santosh Shilimkar
@ 2018-12-11 18:12 ` Kevin Hilman
  5 siblings, 0 replies; 17+ messages in thread
From: Kevin Hilman @ 2018-12-11 18:12 UTC (permalink / raw)
  To: Arnd Bergmann, arm, Kevin Hilman, Tony Lindgren
  Cc: Arnd Bergmann, Dave Gerlach, Keerthy, Tero Kristo,
	Santosh Shilimkar, linux-omap, linux-arm-kernel, linux-kernel

Arnd Bergmann <arnd@arndb.de> writes:

> WARNING: vmlinux.o(.text+0x27530): Section mismatch in reference from the function am43xx_suspend_init() to the function .init.text:am43xx_map_scu()
> The function am43xx_suspend_init() references
> the function __init am43xx_map_scu().
> This is often because am43xx_suspend_init lacks a __init
> annotation or the annotation of am43xx_map_scu is wrong.
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

Reviewed-by: Kevin Hilman <khilman@baylibre.com>

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH 4/4] ARM: pxa: avoid section mismatch warning
  2018-12-10 21:58 ` [PATCH 4/4] ARM: pxa: avoid " Arnd Bergmann
  2018-12-11  9:39   ` Ulf Hansson
@ 2018-12-12 21:56   ` Olof Johansson
  1 sibling, 0 replies; 17+ messages in thread
From: Olof Johansson @ 2018-12-12 21:56 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: arm, Daniel Mack, Haojian Zhuang, Robert Jarzmik, Linus Walleij,
	Ulf Hansson, Igor Grinberg, Miquel Raynal, linux-arm-kernel,
	linux-kernel

On Mon, Dec 10, 2018 at 10:58:39PM +0100, Arnd Bergmann 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>

Applied to next/soc.


-Olof

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH 3/4] ARM: ks8695: fix section mismatch warning
  2018-12-10 21:58 ` [PATCH 3/4] ARM: ks8695: fix " Arnd Bergmann
@ 2018-12-12 21:56   ` Olof Johansson
  0 siblings, 0 replies; 17+ messages in thread
From: Olof Johansson @ 2018-12-12 21:56 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: arm, Greg Ungerer, Robert Jarzmik, Wolfram Sang, James Hogan,
	Lee Jones, linux-arm-kernel, linux-kernel

On Mon, Dec 10, 2018 at 10:58:38PM +0100, Arnd Bergmann wrote:
> WARNING: vmlinux.o(.text+0x13250): Section mismatch in reference from the function acs5k_i2c_init() to the (unknown reference) .init.data:(unknown)
> The function acs5k_i2c_init() references
> the (unknown reference) __initdata (unknown).
> This is often because acs5k_i2c_init lacks a __initdata
> annotation or the annotation of (unknown) is wrong.
> 
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

Applied to next/soc in arm-soc now.


-Olof

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH 2/4] ARM: tegra: avoid section mismatch warning
  2018-12-11 10:28       ` Thierry Reding
@ 2018-12-12 21:57         ` Olof Johansson
  0 siblings, 0 replies; 17+ messages in thread
From: Olof Johansson @ 2018-12-12 21:57 UTC (permalink / raw)
  To: Thierry Reding
  Cc: Arnd Bergmann, arm-soc, Jonathan Hunter, Linux ARM,
	open list:TEGRA ARCHITECTURE SUPPORT, Linux Kernel Mailing List

On Tue, Dec 11, 2018 at 11:28:56AM +0100, Thierry Reding wrote:
> On Tue, Dec 11, 2018 at 11:08:53AM +0100, Arnd Bergmann wrote:
> > On Tue, Dec 11, 2018 at 11:05 AM Thierry Reding
> > <thierry.reding@gmail.com> wrote:
> > >
> > > On Mon, Dec 10, 2018 at 10:58:37PM +0100, Arnd Bergmann wrote:
> > > > WARNING: vmlinux.o(.text+0x39ecc): Section mismatch in reference from the function tegra114_gic_cpu_pm_registration() to the (unknown reference) .init.rodata:(unknown)
> > > > The function tegra114_gic_cpu_pm_registration() references
> > > > the (unknown reference) __initconst (unknown).
> > > > This is often because tegra114_gic_cpu_pm_registration lacks a __initconst
> > > > annotation or the annotation of (unknown) is wrong.
> > > >
> > > > Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> > > > ---
> > > >  arch/arm/mach-tegra/irq.c | 4 ++--
> > > >  1 file changed, 2 insertions(+), 2 deletions(-)
> > >
> > > I'm not seeing these. Do I need to enable any special options to trigger
> > > these? Or are these just so new that I haven't run into them yet?
> > 
> > My guess would be that this is a result of the CONFIG_NO_AUTO_INLINE
> > option that is now available in linux-next.
> 
> Okay, that would explain. It's not set in my configurations currently.
> 
> Do you want to apply this to ARM-SoC directly with my Acked-by, or
> should I queue it up for v4.22?

I'm picking up a few others of these, so I'll include it in that batch.

Applied to arm-soc next/soc.


Thanks!


-Olof


^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH 1/4] ARM: omap2: avoid section mismatch warning
  2018-12-10 22:09 ` [PATCH 1/4] ARM: omap2: " Tony Lindgren
@ 2018-12-12 22:05   ` Olof Johansson
  0 siblings, 0 replies; 17+ messages in thread
From: Olof Johansson @ 2018-12-12 22:05 UTC (permalink / raw)
  To: Tony Lindgren
  Cc: Arnd Bergmann, arm, Kevin Hilman, Dave Gerlach, Keerthy,
	Tero Kristo, Santosh Shilimkar, linux-omap, linux-arm-kernel,
	linux-kernel

On Mon, Dec 10, 2018 at 02:09:26PM -0800, Tony Lindgren wrote:
> * Arnd Bergmann <arnd@arndb.de> [181210 21:59]:
> > WARNING: vmlinux.o(.text+0x27530): Section mismatch in reference from the function am43xx_suspend_init() to the function .init.text:am43xx_map_scu()
> > The function am43xx_suspend_init() references
> > the function __init am43xx_map_scu().
> > This is often because am43xx_suspend_init lacks a __init
> > annotation or the annotation of am43xx_map_scu is wrong.
> 
> Acked-by: Tony Lindgren <tony@atomide.com>

Applied, thanks!


-Olof

^ permalink raw reply	[flat|nested] 17+ messages in thread

end of thread, other threads:[~2018-12-12 22:17 UTC | newest]

Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-12-10 21:58 [PATCH 1/4] ARM: omap2: avoid section mismatch warning Arnd Bergmann
2018-12-10 21:58 ` [PATCH 2/4] ARM: tegra: " Arnd Bergmann
2018-12-11 10:05   ` Thierry Reding
2018-12-11 10:08     ` Arnd Bergmann
2018-12-11 10:28       ` Thierry Reding
2018-12-12 21:57         ` Olof Johansson
2018-12-10 21:58 ` [PATCH 3/4] ARM: ks8695: fix " Arnd Bergmann
2018-12-12 21:56   ` Olof Johansson
2018-12-10 21:58 ` [PATCH 4/4] ARM: pxa: avoid " Arnd Bergmann
2018-12-11  9:39   ` Ulf Hansson
2018-12-11 10:10     ` Arnd Bergmann
2018-12-11 10:44     ` Linus Walleij
2018-12-12 21:56   ` Olof Johansson
2018-12-10 22:09 ` [PATCH 1/4] ARM: omap2: " Tony Lindgren
2018-12-12 22:05   ` Olof Johansson
2018-12-11  2:11 ` Santosh Shilimkar
2018-12-11 18:12 ` Kevin Hilman

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).