Thanks for having a look at this patch, Pekka! On Friday, October 8th, 2021 at 09:29, Pekka Paalanen <ppaalanen@xxxxxxxxx> wrote: > > +#define DRM_IOCTL_MODE_CLOSEFB DRM_IOWR(0xCF, unsigned int) > > Should it have a structure with 'flags' for future-proofing? > > ISTR some rule of thumb that everything new must have 'flags' field > enforced to be zero just in case. I only now saw that RMFB cannot have > flags. I wonder. I don't mind adding a struct with flags or padding, not sure what the policy is for the DRM subsystem.