RE: [PATCH] fbdev: move FBIO_WAITFORVSYNC to linux/fb.h

[James Simmons <jsimmons@infradead.org>]

> > -----Original Message-----
> > From: Andrew Morton [mailto:akpm@linux-foundation.org]
> > Sent: Wednesday, April 21, 2010 4:25 AM
> > To: Hiremath, Vaibhav
> > Cc: Grazvydas Ignotas; linux-fbdev@vger.kernel.org
> > Subject: Re: [PATCH] fbdev: move FBIO_WAITFORVSYNC to linux/fb.h
> > 
> > On Tue, 20 Apr 2010 14:35:10 +0530
> > "Hiremath, Vaibhav" <hvaibhav@ti.com> 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.
> > 
> [Hiremath, Vaibhav] Andrew, 
> If I understand correctly, the previous patch was not following the IOCTL numbering sequence.
> 
> Logically this IOCTL should get added as per their number sequence that is at the end after FBIOGET_DISPINFO (0x4618).

Changing the IOCTL number for WAITFORSYNC would break several apps like 
egl and directfb. I'm afraid we are stuff with that number.