Re: [PATCH] Input: input.h: Fix EVIOCSFF macro inconsistency by using _IOW()
From: Elias Vanderstuyft <hidden>
Date: 2015-09-26 09:45:56
Also in:
linux-api, lkml
On Thu, Sep 17, 2015 at 7:26 PM, Elias Vanderstuyft [off-list ref] wrote:
quoted hunk ↗ jump to hunk
Just like the EVIOCSABS(abs) macro, use the more compact _IOW(..., type) instead of _IOC(_IOC_WRITE, ..., sizeof(type)) for the EVIOCSFF macro. Signed-off-by: Elias Vanderstuyft <redacted> --- include/uapi/linux/input.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)diff --git a/include/uapi/linux/input.h b/include/uapi/linux/input.h index 731417c..63915a7 100644 --- a/include/uapi/linux/input.h +++ b/include/uapi/linux/input.h@@ -147,7 +147,7 @@ struct input_keymap_entry { #define EVIOCGABS(abs) _IOR('E', 0x40 + (abs), struct input_absinfo) /* get abs value/limits */ #define EVIOCSABS(abs) _IOW('E', 0xc0 + (abs), struct input_absinfo) /* set abs value/limits */ -#define EVIOCSFF _IOC(_IOC_WRITE, 'E', 0x80, sizeof(struct ff_effect)) /* send a force effect to a force feedback device */ +#define EVIOCSFF _IOW('E', 0x80, struct ff_effect) /* send a force effect to a force feedback device */ #define EVIOCRMFF _IOW('E', 0x81, int) /* Erase a force effect */ #define EVIOCGEFFECTS _IOR('E', 0x84, int) /* Report number of effects playable at the same time */ --1.9.3
Hi Dmitry, Is there anything that prevents this patch to be applied? About a year ago, we discussed this by message: https://www.marc.info/?l=linux-input&m=141020112415578&w=1 Thanks, Elias