lkml.org 
[lkml]   [2009]   [Jul]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH] move efifb's probe function to .devinit.text
    On 07/11/2009 04:52 PM, Uwe Kleine-König wrote:
    > A pointer to efifb_probe is passed to the core via
    > platform_driver_register and so the function must not disappear when the
    > .init sections are discarded. Otherwise (if also having HOTPLUG=y)
    > unbinding and binding a device to the driver via sysfs will result in an
    > oops as does a device being registered late.

    Looks good to me:

    Acked-by: Peter Jones <pjones@redhat.com>

    >
    > An alternative to this patch is using platform_driver_probe instead of
    > platform_driver_register plus removing the pointer to the probe function
    > from the struct platform_driver.
    >
    > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
    > Cc: Peter Jones <pjones@redhat.com>
    > Cc: Krzysztof Helt <krzysztof.h1@poczta.fm>
    > Cc: Geert Uytterhoeven <Geert.Uytterhoeven@sonycom.com>
    > Cc: Jaya Kumar <jayakumar.lkml@gmail.com>
    > Cc: Ralf Baechle <ralf@linux-mips.org>
    > Cc: Maciej W. Rozycki <macro@linux-mips.org>
    > Cc: Andrew Morton <akpm@linux-foundation.org>
    > Cc: Chandramouli Narayanan <mouli@linux.intel.com>
    > Cc: Huang Ying <ying.huang@intel.com>
    > ---
    > drivers/video/efifb.c | 2 +-
    > 1 files changed, 1 insertions(+), 1 deletions(-)
    >
    > diff --git a/drivers/video/efifb.c b/drivers/video/efifb.c
    > index eb12182..2161f21 100644
    > --- a/drivers/video/efifb.c
    > +++ b/drivers/video/efifb.c
    > @@ -201,7 +201,7 @@ static int __init efifb_setup(char *options)
    > return 0;
    > }
    >
    > -static int __init efifb_probe(struct platform_device *dev)
    > +static int __devinit efifb_probe(struct platform_device *dev)
    > {
    > struct fb_info *info;
    > int err;


    --
    Peter

    In computer science, about the only thing we can prove is that traveling
    salesmen can't find their way from one place to another effeciently, and
    that unix hackers can't really decode 40-bit keys by harnessing all the
    computers in the lab. And the hackers do it anyway, and the salesmen still
    make their rounds.
    -- Ron Jeffries, WikiWikiWeb
    --
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2009-07-13 15:55    [W:3.202 / U:0.424 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site