[PATCH] omap_vout: fix section mismatch in probe function

[PATCH] omap_vout: fix section mismatch in probe function

[Chase Maupin]

Building the OMAP vout driver resulted in the following message:

    WARNING: vmlinux.o(.data+0x38084): Section mismatch in reference
             from the variable omap_vout_driver to the function
             .init.text:omap_vout_probe()
    The variable omap_vout_driver references
    the function __init omap_vout_probe()
    If the reference is valid then annotate the
    variable with __init* or __refdata (see linux/init.h) or name
    the variable:
    *_template, *_timer, *_sht, *_ops, *_probe, *_probe_one,
    *_console

Signed-off-by: Chase Maupin <Chase.Maupin@ti.com>
---
 drivers/media/video/omap/omap_vout.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/media/video/omap/omap_vout.c b/drivers/media/video/omap/omap_vout.c
index b3a5ecd..9b5dad5 100644
--- a/drivers/media/video/omap/omap_vout.c
+++ b/drivers/media/video/omap/omap_vout.c
@@ -2133,7 +2133,7 @@ static int omap_vout_remove(struct platform_device *pdev)
 	return 0;
 }
 
-static int __init omap_vout_probe(struct platform_device *pdev)
+static int omap_vout_probe(struct platform_device *pdev)
 {
 	int ret = 0, i;
 	struct omap_overlay *ovl;
-- 
1.7.0.4

RE: [PATCH] omap_vout: fix section mismatch in probe function

[Maupin, Chase]

> -----Original Message-----
> From: Maupin, Chase
> Sent: Friday, August 26, 2011 11:10 AM
> To: linux-omap@vger.kernel.org
> Cc: Maupin, Chase
> Subject: [PATCH] omap_vout: fix section mismatch in probe function
> 
> Building the OMAP vout driver resulted in the following message:
> 
>     WARNING: vmlinux.o(.data+0x38084): Section mismatch in reference
>              from the variable omap_vout_driver to the function
>              .init.text:omap_vout_probe()
>     The variable omap_vout_driver references
>     the function __init omap_vout_probe()
>     If the reference is valid then annotate the
>     variable with __init* or __refdata (see linux/init.h) or name
>     the variable:
>     *_template, *_timer, *_sht, *_ops, *_probe, *_probe_one,
>     *_console
> 
> Signed-off-by: Chase Maupin <Chase.Maupin@ti.com>

Just wanted to ping on this patch.  Are there any issues?

> ---
>  drivers/media/video/omap/omap_vout.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/media/video/omap/omap_vout.c
> b/drivers/media/video/omap/omap_vout.c
> index b3a5ecd..9b5dad5 100644
> --- a/drivers/media/video/omap/omap_vout.c
> +++ b/drivers/media/video/omap/omap_vout.c
> @@ -2133,7 +2133,7 @@ static int omap_vout_remove(struct
> platform_device *pdev)
>  	return 0;
>  }
> 
> -static int __init omap_vout_probe(struct platform_device *pdev)
> +static int omap_vout_probe(struct platform_device *pdev)
>  {
>  	int ret = 0, i;
>  	struct omap_overlay *ovl;
> --
> 1.7.0.4