On Wed, 2017-01-18 at 18:13 +0100, Greg KH wrote: > On Wed, Jan 18, 2017 at 07:04:45AM -0800, Michael Zoran wrote: > > This change adds the compatibility data structures for > > compatibility > > ioctls and defines compatibility ioctl definitions for the ioctls > > that have changed. > > > > Signed-off-by: Michael Zoran <mzoran@xxxxxxxxxxxx> > > --- > > .../interface/vchiq_arm/vchiq_ioctl.h | 96 > > ++++++++++++++++++++++ > > 1 file changed, 96 insertions(+) > > > > diff --git > > a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_ioctl.h > > b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_ioctl.h > > index 6137ae9de1c1..1681b7792ef2 100644 > > --- > > a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_ioctl.h > > +++ > > b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_ioctl.h > > @@ -35,6 +35,9 @@ > > #define VCHIQ_IOCTLS_H > > > > #include <linux/ioctl.h> > > +#if defined(CONFIG_COMPAT) > > +#include <linux/compat.h> > > +#endif > > Are you sure this ifdef is needed? > > thanks, > > greg k-h You're correct, that specific ifdef isn't necessary. I think the one around the structure definitions is necessary through. I'm more then willing to incorporate removing it into a V2 version of this series, but I would like to get as much feedback as possible on the set so that I can make all the needed changes at once. _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
