RE: [PATCH v16 RESEND 0/7] of: add display helper
From: Mohammed, Afzal <hidden>
Date: 2013-01-24 08:47:02
Also in:
dri-devel, linux-fbdev, linux-media
Subsystem:
framebuffer layer, the rest · Maintainers:
Helge Deller, Linus Torvalds
Possibly related (same subject, not in this thread)
- 2013-02-01 · Re: [PATCH v16 RESEND 0/7] of: add display helper · Jingoo Han <hidden>
- 2013-01-24 · Re: [PATCH v16 RESEND 0/7] of: add display helper · Steffen Trumtrar <hidden>
- 2013-01-24 · Re: [PATCH v16 RESEND 0/7] of: add display helper · Dave Airlie <airlied@gmail.com>
- 2013-01-23 · Re: [PATCH v16 RESEND 0/7] of: add display helper · Steffen Trumtrar <hidden>
- 2013-01-22 · Re: [PATCH v16 RESEND 0/7] of: add display helper · Rob Clark <hidden>
Hi Steffen, On Thu, Jan 24, 2013 at 13:49:58, Steffen Trumtrar wrote:
Thanks. I'll use that opportunity for a v17 that is rebased onto 3.8-rc4.
As you are going to have a v17, if you can fold the diff[1] (that I mentioned earlier) into the patch, "fbmon: add of_videomode helpers", it would be helpful. Regards Afzal [1]
diff --git a/include/linux/fb.h b/include/linux/fb.h
index 58b9860..0ce30d1 100644
--- a/include/linux/fb.h
+++ b/include/linux/fb.h@@ -716,9 +716,19 @@ extern void fb_destroy_modedb(struct fb_videomode *modedb); extern int fb_find_mode_cvt(struct fb_videomode *mode, int margins, int rb); extern unsigned char *fb_ddc_read(struct i2c_adapter *adapter); +#if defined(CONFIG_OF_VIDEOMODE) && defined(CONFIG_FB_MODE_HELPERS) extern int of_get_fb_videomode(struct device_node *np, struct fb_videomode *fb, int index); +#else +static inline int of_get_fb_videomode(struct device_node *np, + struct fb_videomode *fb, + int index) +{ + return -EINVAL; +} +#endif + extern int fb_videomode_from_videomode(const struct videomode *vm, struct fb_videomode *fbmode);