Hi Geert

On Mon, Jul 9, 2012 at 9:34 AM, Geert Uytterhoeven [off-list ref] wrote:

On Sun, Jul 8, 2012 at 11:56 PM, David Herrmann
[off-list ref] wrote:

quoted

--- a/drivers/video/fbmem.c
+++ b/drivers/video/fbmem.c

@@ -46,7 +46,7 @@ static DEFINE_MUTEX(registration_lock);
 struct fb_info *registered_fb[FB_MAX] __read_mostly;
 int num_registered_fb __read_mostly;

-static struct fb_info *get_fb_info(unsigned int idx)
+struct fb_info *get_fb_info(unsigned int idx)
 {
        struct fb_info *fb_info;

@@ -61,14 +61,16 @@ static struct fb_info *get_fb_info(unsigned int idx)

        return fb_info;
 }
+EXPORT_SYMBOL(get_fb_info);

EXPORT_SYMBOL_GPL?

Thanks, that seems more appropriate here. I've fixed it.

quoted

-static void put_fb_info(struct fb_info *fb_info)
+void put_fb_info(struct fb_info *fb_info)
 {
        if (!atomic_dec_and_test(&fb_info->count))
                return;
        if (fb_info->fbops->fb_destroy)
                fb_info->fbops->fb_destroy(fb_info);
 }
+EXPORT_SYMBOL(put_fb_info);

EXPORT_SYMBOL_GPL?

Thanks
David