Hi Gerd, Thank you for the patch. On Tuesday, 11 September 2018 09:59:12 EEST Gerd Hoffmann wrote: > Reported-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > Signed-off-by: Gerd Hoffmann <kraxel@xxxxxxxxxx> > --- > drivers/dma-buf/udmabuf.c | 17 ++++++++--------- > 1 file changed, 8 insertions(+), 9 deletions(-) > > diff --git a/drivers/dma-buf/udmabuf.c b/drivers/dma-buf/udmabuf.c > index 2e8502250a..e63d301bcb 100644 > --- a/drivers/dma-buf/udmabuf.c > +++ b/drivers/dma-buf/udmabuf.c > @@ -1,17 +1,16 @@ > // SPDX-License-Identifier: GPL-2.0 > -#include <linux/init.h> > -#include <linux/module.h> > +#include <linux/cred.h> > #include <linux/device.h> > -#include <linux/kernel.h> > -#include <linux/slab.h> > -#include <linux/miscdevice.h> > #include <linux/dma-buf.h> > #include <linux/highmem.h> > -#include <linux/cred.h> > -#include <linux/shmem_fs.h> > +#include <linux/init.h> > +#include <linux/kernel.h> > #include <linux/memfd.h> > - > -#include <uapi/linux/udmabuf.h> > +#include <linux/miscdevice.h> > +#include <linux/module.h> > +#include <linux/slab.h> > +#include <linux/shmem_fs.h> Nearly there, l comes after h :-) Apart from that, Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > +#include <linux/udmabuf.h> > > struct udmabuf { > u32 pagecount; -- Regards, Laurent Pinchart _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel