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>
<snip>
Hi,
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 http://lwn.net/Articles/232575/
--
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 http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/