2007-05-14 10:57:55

by Li Yang

[permalink] [raw]
Subject: [PATCH 2/2] powerpc: Fix Section mismatch warnings

This patch fix the following Section mismatch warnings
in powerpc code.

WARNING: arch/powerpc/platforms/built-in.o - Section mismatch: reference to .init.data:mv643xx_eth_pd_devs from .text between 'mv643xx_eth_add_pds' (at offset 0x9ed2) and 'gg2_read_config'
WARNING: arch/powerpc/platforms/built-in.o - Section mismatch: reference to .init.data:mv643xx_eth_pd_devs from .text between 'mv643xx_eth_add_pds' (at offset 0x9ed6) and 'gg2_read_config'
WARNING: arch/powerpc/platforms/built-in.o - Section mismatch: reference to .init.text:note_scsi_host from __ksymtab between '__ksymtab_note_scsi_host' (at offset 0x8) and '__ksymtab_sys_ctrler'

Signed-off-by: Li Yang <[email protected]>
---
arch/powerpc/platforms/chrp/pegasos_eth.c | 2 +-
arch/powerpc/platforms/powermac/setup.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/powerpc/platforms/chrp/pegasos_eth.c b/arch/powerpc/platforms/chrp/pegasos_eth.c
index 7104567..5c9f135 100644
--- a/arch/powerpc/platforms/chrp/pegasos_eth.c
+++ b/arch/powerpc/platforms/chrp/pegasos_eth.c
@@ -169,7 +169,7 @@ static int Enable_SRAM(void)

/***********/
/***********/
-int mv643xx_eth_add_pds(void)
+int __init mv643xx_eth_add_pds(void)
{
int ret = 0;
static struct pci_device_id pci_marvell_mv64360[] = {
diff --git a/arch/powerpc/platforms/powermac/setup.c b/arch/powerpc/platforms/powermac/setup.c
index a410bc7..07b1c4e 100644
--- a/arch/powerpc/platforms/powermac/setup.c
+++ b/arch/powerpc/platforms/powermac/setup.c
@@ -384,7 +384,7 @@ int boot_part;
static dev_t boot_dev;

#ifdef CONFIG_SCSI
-void __init note_scsi_host(struct device_node *node, void *host)
+void note_scsi_host(struct device_node *node, void *host)
{
int l;
char *p;



2007-05-14 14:03:28

by Kumar Gala

[permalink] [raw]
Subject: Re: [PATCH 2/2] powerpc: Fix Section mismatch warnings


On May 14, 2007, at 5:59 AM, Li Yang wrote:

> This patch fix the following Section mismatch warnings
> in powerpc code.
>
> WARNING: arch/powerpc/platforms/built-in.o - Section mismatch:
> reference to .init.data:mv643xx_eth_pd_devs from .text between
> 'mv643xx_eth_add_pds' (at offset 0x9ed2) and 'gg2_read_config'
> WARNING: arch/powerpc/platforms/built-in.o - Section mismatch:
> reference to .init.data:mv643xx_eth_pd_devs from .text between
> 'mv643xx_eth_add_pds' (at offset 0x9ed6) and 'gg2_read_config'
> WARNING: arch/powerpc/platforms/built-in.o - Section mismatch:
> reference to .init.text:note_scsi_host from __ksymtab between
> '__ksymtab_note_scsi_host' (at offset 0x8) and '__ksymtab_sys_ctrler'
>
> Signed-off-by: Li Yang <[email protected]>
> ---
> arch/powerpc/platforms/chrp/pegasos_eth.c | 2 +-
> arch/powerpc/platforms/powermac/setup.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/powerpc/platforms/chrp/pegasos_eth.c b/arch/
> powerpc/platforms/chrp/pegasos_eth.c
> index 7104567..5c9f135 100644
> --- a/arch/powerpc/platforms/chrp/pegasos_eth.c
> +++ b/arch/powerpc/platforms/chrp/pegasos_eth.c
> @@ -169,7 +169,7 @@ static int Enable_SRAM(void)
> /***********/
> /***********/
> -int mv643xx_eth_add_pds(void)
> +int __init mv643xx_eth_add_pds(void)

it should probably be static as well.

> {
> int ret = 0;
> static struct pci_device_id pci_marvell_mv64360[] = {
> diff --git a/arch/powerpc/platforms/powermac/setup.c b/arch/powerpc/
> platforms/powermac/setup.c
> index a410bc7..07b1c4e 100644
> --- a/arch/powerpc/platforms/powermac/setup.c
> +++ b/arch/powerpc/platforms/powermac/setup.c
> @@ -384,7 +384,7 @@ int boot_part;
> static dev_t boot_dev;
> #ifdef CONFIG_SCSI
> -void __init note_scsi_host(struct device_node *node, void *host)
> +void note_scsi_host(struct device_node *node, void *host)
> {
> int l;
> char *p;
>
>
> -
> To unsubscribe from this list: send the line "unsubscribe linux-
> kernel" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/

2007-05-15 14:07:57

by Kumar Gala

[permalink] [raw]
Subject: Re: [PATCH 2/2] powerpc: Fix Section mismatch warnings


On May 14, 2007, at 5:59 AM, Li Yang wrote:

> This patch fix the following Section mismatch warnings
> in powerpc code.
>
> WARNING: arch/powerpc/platforms/built-in.o - Section mismatch:
> reference to .init.data:mv643xx_eth_pd_devs from .text between
> 'mv643xx_eth_add_pds' (at offset 0x9ed2) and 'gg2_read_config'
> WARNING: arch/powerpc/platforms/built-in.o - Section mismatch:
> reference to .init.data:mv643xx_eth_pd_devs from .text between
> 'mv643xx_eth_add_pds' (at offset 0x9ed6) and 'gg2_read_config'
> WARNING: arch/powerpc/platforms/built-in.o - Section mismatch:
> reference to .init.text:note_scsi_host from __ksymtab between
> '__ksymtab_note_scsi_host' (at offset 0x8) and '__ksymtab_sys_ctrler'
>
> Signed-off-by: Li Yang <[email protected]>
> ---
> arch/powerpc/platforms/chrp/pegasos_eth.c | 2 +-
> arch/powerpc/platforms/powermac/setup.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)

Did you change anything?

- k

> diff --git a/arch/powerpc/platforms/chrp/pegasos_eth.c b/arch/
> powerpc/platforms/chrp/pegasos_eth.c
> index 7104567..5c9f135 100644
> --- a/arch/powerpc/platforms/chrp/pegasos_eth.c
> +++ b/arch/powerpc/platforms/chrp/pegasos_eth.c
> @@ -169,7 +169,7 @@ static int Enable_SRAM(void)
>
> /***********/
> /***********/
> -int mv643xx_eth_add_pds(void)
> +int __init mv643xx_eth_add_pds(void)
> {
> int ret = 0;
> static struct pci_device_id pci_marvell_mv64360[] = {
> diff --git a/arch/powerpc/platforms/powermac/setup.c b/arch/powerpc/
> platforms/powermac/setup.c
> index a410bc7..07b1c4e 100644
> --- a/arch/powerpc/platforms/powermac/setup.c
> +++ b/arch/powerpc/platforms/powermac/setup.c
> @@ -384,7 +384,7 @@ int boot_part;
> static dev_t boot_dev;
>
> #ifdef CONFIG_SCSI
> -void __init note_scsi_host(struct device_node *node, void *host)
> +void note_scsi_host(struct device_node *node, void *host)
> {
> int l;
> char *p;
>
>
> _______________________________________________
> Linuxppc-dev mailing list
> [email protected]
> https://ozlabs.org/mailman/listinfo/linuxppc-dev

2007-05-16 02:08:16

by Li Yang

[permalink] [raw]
Subject: RE: [PATCH 2/2] powerpc: Fix Section mismatch warnings

> -----Original Message-----
> From: Kumar Gala [mailto:[email protected]]
> Sent: Tuesday, May 15, 2007 10:07 PM
> To: Li Yang-r58472
> Cc: Paul; [email protected]; Linux Kernel
> Subject: Re: [PATCH 2/2] powerpc: Fix Section mismatch warnings
>
>
> On May 14, 2007, at 5:59 AM, Li Yang wrote:
>
> > This patch fix the following Section mismatch warnings
> > in powerpc code.
> >
> > WARNING: arch/powerpc/platforms/built-in.o - Section mismatch:
> > reference to .init.data:mv643xx_eth_pd_devs from .text between
> > 'mv643xx_eth_add_pds' (at offset 0x9ed2) and 'gg2_read_config'
> > WARNING: arch/powerpc/platforms/built-in.o - Section mismatch:
> > reference to .init.data:mv643xx_eth_pd_devs from .text between
> > 'mv643xx_eth_add_pds' (at offset 0x9ed6) and 'gg2_read_config'
> > WARNING: arch/powerpc/platforms/built-in.o - Section mismatch:
> > reference to .init.text:note_scsi_host from __ksymtab between
> > '__ksymtab_note_scsi_host' (at offset 0x8) and
'__ksymtab_sys_ctrler'
> >
> > Signed-off-by: Li Yang <[email protected]>
> > ---
> > arch/powerpc/platforms/chrp/pegasos_eth.c | 2 +-
> > arch/powerpc/platforms/powermac/setup.c | 2 +-
> > 2 files changed, 2 insertions(+), 2 deletions(-)
>
> Did you change anything?

Nothing yet. I think it's a delay of mail system.

- Leo