Re: [PATCH 1/1] UIO: Add a write() function to enable/disable interrupts

From: Tom Spink
Date: Thu May 22 2008 - 15:47:33 EST

2008/5/22 Hans J. Koch <hjk@xxxxxxxxxxxxx>:
> Sometimes it is necessary to enable/disable the interrupt of a UIO device
> from the userspace part of the driver. With this patch, the UIO kernel driver
> can implement an "irqcontrol()" function that does this. Userspace can write
> an s32 value to /dev/uioX (usually 0 or 1 to turn the irq off or on). The
> UIO core will then call the driver's irqcontrol function.
> Signed-off-by: Hans J. Koch <hjk@xxxxxxxxxxxxx>



I wonder if it would be better to implement this as an ioctl, rather
than a write to the device. Writing to a device is a pretty generic
thing, and this patch would tie that up to specifically controlling
interrupts. An ioctl would be more appropriate, IMO, as you are
issuing a controlling command, i.e. disable or enable interrupts.

By the way, I have absolutely no idea how the UIO driver works, other
than reading

Tom Spink
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at