On 04/15/2012 10:39 AM, Mark Brown wrote: > Since s3c_fb_missing_pixclock() is called from s3c_fb_set_rgb_timing() > which is used in the suspend/resume paths it can't be marked __devinit > as this could result in it being discarded after boot. > > Signed-off-by: Mark Brown <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx> I'm not sure that this can actually happen. But the code looks saner with the patch applied so I applied it. Thanks, Florian Tobias Schandinat > --- > drivers/video/s3c-fb.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/drivers/video/s3c-fb.c b/drivers/video/s3c-fb.c > index 18c84b8..d8f0018 100644 > --- a/drivers/video/s3c-fb.c > +++ b/drivers/video/s3c-fb.c > @@ -1056,7 +1056,7 @@ static struct fb_ops s3c_fb_ops = { > * > * Calculate the pixel clock when none has been given through platform data. > */ > -static void __devinit s3c_fb_missing_pixclock(struct fb_videomode *mode) > +static void s3c_fb_missing_pixclock(struct fb_videomode *mode) > { > u64 pixclk = 1000000000000ULL; > u32 div; -- To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html