On Tue, 20 Apr 2010 14:35:10 +0530 "Hiremath, Vaibhav" <hvaibhav@xxxxxx> wrote: > ... > > > --- a/include/linux/fb.h > > +++ b/include/linux/fb.h > > @@ -23,6 +23,8 @@ struct dentry; > > #else > > #define FBIO_CURSOR _IOWR('F', 0x08, struct fb_cursor) > > #endif > > +#define FBIO_WAITFORVSYNC _IOW('F', 0x20, __u32) > > + > [Hiremath, Vaibhav] Do not insert it in middle, lets follow the sequence and append it at last. Otherwise looks ok to me. Confused. That definition appears to be in the appropriate place to me. > We should merge this patch. > > There are some other drivers (like OMAP) which define WAITFORVSYNC ioctl with custom (OMAPFB_WAITFORVSYNC) way needs to be changed for this. > > I will change them and submit it to list. Please cc myself... -- 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