On Friday, October 08, 2010 23:12:35 Jarod Wilson wrote:
> I've dropped the .compat_ioctl addition from Joris' original patch,
> as I swear the non-compat definition should now work for both 32-bit
> and 64-bit userspace. Technically, I think we still need/want a
> Signed-off-by: from Joris here. Joris? (And sorry for the lengthy
> delay in getting a reply to you).
No. I just tested the patch and it does not work without .compat_ioctl.
I get this error:
[ 313.614017] ioctl32(irrecord:4490): Unknown cmd fd(4) cmd(80046900 {t:'i';sz:4} arg(080705e8) on /dev/lirc0
irrecord: could not get hardware features
irrecord: this device driver does not support the new LIRC interface
irrecord: major number of /dev/lirc0 is 252
irrecord: LIRC major number is 61
irrecord: check if /dev/lirc0 is a LIRC device
irrecord: could not init hardware (lircd running ? --> close it, check permissions)
On the other hand it works fine when .compat_ioctl is added to ir_lirc_codec.c and lirc_dev.c and
lirc_serial.c.
Regards,
Joris.
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
