diff options
| author | Linus Torvalds <torvalds@linux-foundation.org> | 2012-10-02 20:16:10 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-10-02 20:16:10 -0400 |
| commit | fc47912d9cda50ae6bd9ca30e97e8c03de5b7b60 (patch) | |
| tree | d7da6ce3b23c1524e681ee33aa6e21d3c0586108 /drivers/input/misc | |
| parent | 06fe918e9f177dc2a0592b0ad40a6ce4920b2033 (diff) | |
| parent | dde3ada3d0069855eeb353707b2b0f946191cfd6 (diff) | |
Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/dtor/input
Pull input updates from Dmitry Torokhov:
"A few drivers were updated with device tree bindings and others got a
few small cleanups and fixes."
Fix trivial conflict in drivers/input/keyboard/omap-keypad.c due to
changes clashing with a whitespace cleanup.
* 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/dtor/input: (28 commits)
Input: wacom - mark Intuos5 pad as in-prox when touching buttons
Input: synaptics - adjust threshold for treating position values as negative
Input: hgpk - use %*ph to dump small buffer
Input: gpio_keys_polled - fix dt pdata->nbuttons
Input: Add KD[GS]KBDIACRUC ioctls to the compatible list
Input: omap-keypad - fixed formatting
Input: tegra - move platform data header
Input: wacom - add support for EMR on Cintiq 24HD touch
Input: s3c2410_ts - make s3c_ts_pmops const
Input: samsung-keypad - use of_get_child_count() helper
Input: samsung-keypad - use of_match_ptr()
Input: uinput - fix formatting
Input: uinput - specify exact bit sizes on userspace APIs
Input: uinput - mark failed submission requests as free
Input: uinput - fix race that can block nonblocking read
Input: uinput - return -EINVAL when read buffer size is too small
Input: uinput - take event lock when fetching events from buffer
Input: get rid of MATCH_BIT() macro
Input: rotary-encoder - add DT bindings
Input: rotary-encoder - constify platform data pointers
...
Diffstat (limited to 'drivers/input/misc')
| -rw-r--r-- | drivers/input/misc/rotary_encoder.c | 140 | ||||
| -rw-r--r-- | drivers/input/misc/twl4030-pwrbutton.c | 1 | ||||
| -rw-r--r-- | drivers/input/misc/uinput.c | 197 |
3 files changed, 218 insertions, 120 deletions
diff --git a/drivers/input/misc/rotary_encoder.c b/drivers/input/misc/rotary_encoder.c index f07f784198b..99a49e4968d 100644 --- a/drivers/input/misc/rotary_encoder.c +++ b/drivers/input/misc/rotary_encoder.c | |||
| @@ -24,12 +24,14 @@ | |||
| 24 | #include <linux/gpio.h> | 24 | #include <linux/gpio.h> |
| 25 | #include <linux/rotary_encoder.h> | 25 | #include <linux/rotary_encoder.h> |
| 26 | #include <linux/slab.h> | 26 | #include <linux/slab.h> |
| 27 | #include <linux/of_platform.h> | ||
| 28 | #include <linux/of_gpio.h> | ||
| 27 | 29 | ||
| 28 | #define DRV_NAME "rotary-encoder" | 30 | #define DRV_NAME "rotary-encoder" |
| 29 | 31 | ||
| 30 | struct rotary_encoder { | 32 | struct rotary_encoder { |
| 31 | struct input_dev *input; | 33 | struct input_dev *input; |
| 32 | struct rotary_encoder_platform_data *pdata; | 34 | const struct rotary_encoder_platform_data *pdata; |
| 33 | 35 | ||
| 34 | unsigned int axis; | 36 | unsigned int axis; |
| 35 | unsigned int pos; | 37 | unsigned int pos; |
| @@ -43,7 +45,7 @@ struct rotary_encoder { | |||
| 43 | char last_stable; | 45 | char last_stable; |
| 44 | }; | 46 | }; |
| 45 | 47 | ||
| 46 | static int rotary_encoder_get_state(struct rotary_encoder_platform_data *pdata) | 48 | static int rotary_encoder_get_state(const struct rotary_encoder_platform_data *pdata) |
| 47 | { | 49 | { |
| 48 | int a = !!gpio_get_value(pdata->gpio_a); | 50 | int a = !!gpio_get_value(pdata->gpio_a); |
| 49 | int b = !!gpio_get_value(pdata->gpio_b); | 51 | int b = !!gpio_get_value(pdata->gpio_b); |
| @@ -56,7 +58,7 @@ static int rotary_encoder_get_state(struct rotary_encoder_platform_data *pdata) | |||
| 56 | 58 | ||
| 57 | static void rotary_encoder_report_event(struct rotary_encoder *encoder) | 59 | static void rotary_encoder_report_event(struct rotary_encoder *encoder) |
| 58 | { | 60 | { |
| 59 | struct rotary_encoder_platform_data *pdata = encoder->pdata; | 61 | const struct rotary_encoder_platform_data *pdata = encoder->pdata; |
| 60 | 62 | ||
| 61 | if (pdata->relative_axis) { | 63 | if (pdata->relative_axis) { |
| 62 | input_report_rel(encoder->input, | 64 | input_report_rel(encoder->input, |
| @@ -140,36 +142,89 @@ static irqreturn_t rotary_encoder_half_period_irq(int irq, void *dev_id) | |||
| 140 | return IRQ_HANDLED; | 142 | return IRQ_HANDLED; |
| 141 | } | 143 | } |
| 142 | 144 | ||
| 145 | #ifdef CONFIG_OF | ||
| 146 | static struct of_device_id rotary_encoder_of_match[] = { | ||
| 147 | { .compatible = "rotary-encoder", }, | ||
| 148 | { }, | ||
| 149 | }; | ||
| 150 | MODULE_DEVICE_TABLE(of, rotary_encoder_of_match); | ||
| 151 | |||
| 152 | static struct rotary_encoder_platform_data * __devinit | ||
| 153 | rotary_encoder_parse_dt(struct device *dev) | ||
| 154 | { | ||
| 155 | const struct of_device_id *of_id = | ||
| 156 | of_match_device(rotary_encoder_of_match, dev); | ||
| 157 | struct device_node *np = dev->of_node; | ||
| 158 | struct rotary_encoder_platform_data *pdata; | ||
| 159 | enum of_gpio_flags flags; | ||
| 160 | |||
| 161 | if (!of_id || !np) | ||
| 162 | return NULL; | ||
| 163 | |||
| 164 | pdata = kzalloc(sizeof(struct rotary_encoder_platform_data), | ||
| 165 | GFP_KERNEL); | ||
| 166 | if (!pdata) | ||
| 167 | return ERR_PTR(-ENOMEM); | ||
| 168 | |||
| 169 | of_property_read_u32(np, "rotary-encoder,steps", &pdata->steps); | ||
| 170 | of_property_read_u32(np, "linux,axis", &pdata->axis); | ||
| 171 | |||
| 172 | pdata->gpio_a = of_get_gpio_flags(np, 0, &flags); | ||
| 173 | pdata->inverted_a = flags & OF_GPIO_ACTIVE_LOW; | ||
| 174 | |||
| 175 | pdata->gpio_b = of_get_gpio_flags(np, 1, &flags); | ||
| 176 | pdata->inverted_b = flags & OF_GPIO_ACTIVE_LOW; | ||
| 177 | |||
| 178 | pdata->relative_axis = !!of_get_property(np, | ||
| 179 | "rotary-encoder,relative-axis", NULL); | ||
| 180 | pdata->rollover = !!of_get_property(np, | ||
| 181 | "rotary-encoder,rollover", NULL); | ||
| 182 | pdata->half_period = !!of_get_property(np, | ||
| 183 | "rotary-encoder,half-period", NULL); | ||
| 184 | |||
| 185 | return pdata; | ||
| 186 | } | ||
| 187 | #else | ||
| 188 | static inline struct rotary_encoder_platform_data * | ||
| 189 | rotary_encoder_parse_dt(struct device *dev) | ||
| 190 | { | ||
| 191 | return NULL; | ||
| 192 | } | ||
| 193 | #endif | ||
| 194 | |||
| 143 | static int __devinit rotary_encoder_probe(struct platform_device *pdev) | 195 | static int __devinit rotary_encoder_probe(struct platform_device *pdev) |
| 144 | { | 196 | { |
| 145 | struct rotary_encoder_platform_data *pdata = pdev->dev.platform_data; | 197 | struct device *dev = &pdev->dev; |
| 198 | const struct rotary_encoder_platform_data *pdata = dev_get_platdata(dev); | ||
| 146 | struct rotary_encoder *encoder; | 199 | struct rotary_encoder *encoder; |
| 147 | struct input_dev *input; | 200 | struct input_dev *input; |
| 148 | irq_handler_t handler; | 201 | irq_handler_t handler; |
| 149 | int err; | 202 | int err; |
| 150 | 203 | ||
| 151 | if (!pdata) { | 204 | if (!pdata) { |
| 152 | dev_err(&pdev->dev, "missing platform data\n"); | 205 | pdata = rotary_encoder_parse_dt(dev); |
| 153 | return -ENOENT; | 206 | if (IS_ERR(pdata)) |
| 207 | return PTR_ERR(pdata); | ||
| 208 | |||
| 209 | if (!pdata) { | ||
| 210 | dev_err(dev, "missing platform data\n"); | ||
| 211 | return -EINVAL; | ||
| 212 | } | ||
| 154 | } | 213 | } |
| 155 | 214 | ||
| 156 | encoder = kzalloc(sizeof(struct rotary_encoder), GFP_KERNEL); | 215 | encoder = kzalloc(sizeof(struct rotary_encoder), GFP_KERNEL); |
| 157 | input = input_allocate_device(); | 216 | input = input_allocate_device(); |
| 158 | if (!encoder || !input) { | 217 | if (!encoder || !input) { |
| 159 | dev_err(&pdev->dev, "failed to allocate memory for device\n"); | ||
| 160 | err = -ENOMEM; | 218 | err = -ENOMEM; |
| 161 | goto exit_free_mem; | 219 | goto exit_free_mem; |
| 162 | } | 220 | } |
| 163 | 221 | ||
| 164 | encoder->input = input; | 222 | encoder->input = input; |
| 165 | encoder->pdata = pdata; | 223 | encoder->pdata = pdata; |
| 166 | encoder->irq_a = gpio_to_irq(pdata->gpio_a); | ||
| 167 | encoder->irq_b = gpio_to_irq(pdata->gpio_b); | ||
| 168 | 224 | ||
| 169 | /* create and register the input driver */ | ||
| 170 | input->name = pdev->name; | 225 | input->name = pdev->name; |
| 171 | input->id.bustype = BUS_HOST; | 226 | input->id.bustype = BUS_HOST; |
| 172 | input->dev.parent = &pdev->dev; | 227 | input->dev.parent = dev; |
| 173 | 228 | ||
| 174 | if (pdata->relative_axis) { | 229 | if (pdata->relative_axis) { |
| 175 | input->evbit[0] = BIT_MASK(EV_REL); | 230 | input->evbit[0] = BIT_MASK(EV_REL); |
| @@ -180,40 +235,21 @@ static int __devinit rotary_encoder_probe(struct platform_device *pdev) | |||
| 180 | pdata->axis, 0, pdata->steps, 0, 1); | 235 | pdata->axis, 0, pdata->steps, 0, 1); |
| 181 | } | 236 | } |
| 182 | 237 | ||
| 183 | err = input_register_device(input); | ||
| 184 | if (err) { | ||
| 185 | dev_err(&pdev->dev, "failed to register input device\n"); | ||
| 186 | goto exit_free_mem; | ||
| 187 | } | ||
| 188 | |||
| 189 | /* request the GPIOs */ | 238 | /* request the GPIOs */ |
| 190 | err = gpio_request(pdata->gpio_a, DRV_NAME); | 239 | err = gpio_request_one(pdata->gpio_a, GPIOF_IN, dev_name(dev)); |
| 191 | if (err) { | ||
| 192 | dev_err(&pdev->dev, "unable to request GPIO %d\n", | ||
| 193 | pdata->gpio_a); | ||
| 194 | goto exit_unregister_input; | ||
| 195 | } | ||
| 196 | |||
| 197 | err = gpio_direction_input(pdata->gpio_a); | ||
| 198 | if (err) { | 240 | if (err) { |
| 199 | dev_err(&pdev->dev, "unable to set GPIO %d for input\n", | 241 | dev_err(dev, "unable to request GPIO %d\n", pdata->gpio_a); |
| 200 | pdata->gpio_a); | 242 | goto exit_free_mem; |
| 201 | goto exit_unregister_input; | ||
| 202 | } | 243 | } |
| 203 | 244 | ||
| 204 | err = gpio_request(pdata->gpio_b, DRV_NAME); | 245 | err = gpio_request_one(pdata->gpio_b, GPIOF_IN, dev_name(dev)); |
| 205 | if (err) { | 246 | if (err) { |
| 206 | dev_err(&pdev->dev, "unable to request GPIO %d\n", | 247 | dev_err(dev, "unable to request GPIO %d\n", pdata->gpio_b); |
| 207 | pdata->gpio_b); | ||
| 208 | goto exit_free_gpio_a; | 248 | goto exit_free_gpio_a; |
| 209 | } | 249 | } |
| 210 | 250 | ||
| 211 | err = gpio_direction_input(pdata->gpio_b); | 251 | encoder->irq_a = gpio_to_irq(pdata->gpio_a); |
| 212 | if (err) { | 252 | encoder->irq_b = gpio_to_irq(pdata->gpio_b); |
| 213 | dev_err(&pdev->dev, "unable to set GPIO %d for input\n", | ||
| 214 | pdata->gpio_b); | ||
| 215 | goto exit_free_gpio_a; | ||
| 216 | } | ||
| 217 | 253 | ||
| 218 | /* request the IRQs */ | 254 | /* request the IRQs */ |
| 219 | if (pdata->half_period) { | 255 | if (pdata->half_period) { |
| @@ -227,8 +263,7 @@ static int __devinit rotary_encoder_probe(struct platform_device *pdev) | |||
| 227 | IRQF_TRIGGER_RISING | IRQF_TRIGGER_FALLING, | 263 | IRQF_TRIGGER_RISING | IRQF_TRIGGER_FALLING, |
| 228 | DRV_NAME, encoder); | 264 | DRV_NAME, encoder); |
| 229 | if (err) { | 265 | if (err) { |
| 230 | dev_err(&pdev->dev, "unable to request IRQ %d\n", | 266 | dev_err(dev, "unable to request IRQ %d\n", encoder->irq_a); |
| 231 | encoder->irq_a); | ||
| 232 | goto exit_free_gpio_b; | 267 | goto exit_free_gpio_b; |
| 233 | } | 268 | } |
| 234 | 269 | ||
| @@ -236,43 +271,55 @@ static int __devinit rotary_encoder_probe(struct platform_device *pdev) | |||
| 236 | IRQF_TRIGGER_RISING | IRQF_TRIGGER_FALLING, | 271 | IRQF_TRIGGER_RISING | IRQF_TRIGGER_FALLING, |
| 237 | DRV_NAME, encoder); | 272 | DRV_NAME, encoder); |
| 238 | if (err) { | 273 | if (err) { |
| 239 | dev_err(&pdev->dev, "unable to request IRQ %d\n", | 274 | dev_err(dev, "unable to request IRQ %d\n", encoder->irq_b); |
| 240 | encoder->irq_b); | ||
| 241 | goto exit_free_irq_a; | 275 | goto exit_free_irq_a; |
| 242 | } | 276 | } |
| 243 | 277 | ||
| 278 | err = input_register_device(input); | ||
| 279 | if (err) { | ||
| 280 | dev_err(dev, "failed to register input device\n"); | ||
| 281 | goto exit_free_irq_b; | ||
| 282 | } | ||
| 283 | |||
| 244 | platform_set_drvdata(pdev, encoder); | 284 | platform_set_drvdata(pdev, encoder); |
| 245 | 285 | ||
| 246 | return 0; | 286 | return 0; |
| 247 | 287 | ||
| 288 | exit_free_irq_b: | ||
| 289 | free_irq(encoder->irq_b, encoder); | ||
| 248 | exit_free_irq_a: | 290 | exit_free_irq_a: |
| 249 | free_irq(encoder->irq_a, encoder); | 291 | free_irq(encoder->irq_a, encoder); |
| 250 | exit_free_gpio_b: | 292 | exit_free_gpio_b: |
| 251 | gpio_free(pdata->gpio_b); | 293 | gpio_free(pdata->gpio_b); |
| 252 | exit_free_gpio_a: | 294 | exit_free_gpio_a: |
| 253 | gpio_free(pdata->gpio_a); | 295 | gpio_free(pdata->gpio_a); |
| 254 | exit_unregister_input: | ||
| 255 | input_unregister_device(input); | ||
| 256 | input = NULL; /* so we don't try to free it */ | ||
| 257 | exit_free_mem: | 296 | exit_free_mem: |
| 258 | input_free_device(input); | 297 | input_free_device(input); |
| 259 | kfree(encoder); | 298 | kfree(encoder); |
| 299 | if (!dev_get_platdata(&pdev->dev)) | ||
| 300 | kfree(pdata); | ||
| 301 | |||
| 260 | return err; | 302 | return err; |
| 261 | } | 303 | } |
| 262 | 304 | ||
| 263 | static int __devexit rotary_encoder_remove(struct platform_device *pdev) | 305 | static int __devexit rotary_encoder_remove(struct platform_device *pdev) |
| 264 | { | 306 | { |
| 265 | struct rotary_encoder *encoder = platform_get_drvdata(pdev); | 307 | struct rotary_encoder *encoder = platform_get_drvdata(pdev); |
| 266 | struct rotary_encoder_platform_data *pdata = pdev->dev.platform_data; | 308 | const struct rotary_encoder_platform_data *pdata = encoder->pdata; |
| 267 | 309 | ||
| 268 | free_irq(encoder->irq_a, encoder); | 310 | free_irq(encoder->irq_a, encoder); |
| 269 | free_irq(encoder->irq_b, encoder); | 311 | free_irq(encoder->irq_b, encoder); |
| 270 | gpio_free(pdata->gpio_a); | 312 | gpio_free(pdata->gpio_a); |
| 271 | gpio_free(pdata->gpio_b); | 313 | gpio_free(pdata->gpio_b); |
| 314 | |||
| 272 | input_unregister_device(encoder->input); | 315 | input_unregister_device(encoder->input); |
| 273 | platform_set_drvdata(pdev, NULL); | ||
| 274 | kfree(encoder); | 316 | kfree(encoder); |
| 275 | 317 | ||
| 318 | if (!dev_get_platdata(&pdev->dev)) | ||
| 319 | kfree(pdata); | ||
| 320 | |||
| 321 | platform_set_drvdata(pdev, NULL); | ||
| 322 | |||
| 276 | return 0; | 323 | return 0; |
| 277 | } | 324 | } |
| 278 | 325 | ||
| @@ -282,6 +329,7 @@ static struct platform_driver rotary_encoder_driver = { | |||
| 282 | .driver = { | 329 | .driver = { |
| 283 | .name = DRV_NAME, | 330 | .name = DRV_NAME, |
| 284 | .owner = THIS_MODULE, | 331 | .owner = THIS_MODULE, |
| 332 | .of_match_table = of_match_ptr(rotary_encoder_of_match), | ||
| 285 | } | 333 | } |
| 286 | }; | 334 | }; |
| 287 | module_platform_driver(rotary_encoder_driver); | 335 | module_platform_driver(rotary_encoder_driver); |
diff --git a/drivers/input/misc/twl4030-pwrbutton.c b/drivers/input/misc/twl4030-pwrbutton.c index 38e4b507b94..b3dd96d6448 100644 --- a/drivers/input/misc/twl4030-pwrbutton.c +++ b/drivers/input/misc/twl4030-pwrbutton.c | |||
| @@ -42,6 +42,7 @@ static irqreturn_t powerbutton_irq(int irq, void *_pwr) | |||
| 42 | err = twl_i2c_read_u8(TWL4030_MODULE_PM_MASTER, &value, | 42 | err = twl_i2c_read_u8(TWL4030_MODULE_PM_MASTER, &value, |
| 43 | STS_HW_CONDITIONS); | 43 | STS_HW_CONDITIONS); |
| 44 | if (!err) { | 44 | if (!err) { |
| 45 | pm_wakeup_event(pwr->dev.parent, 0); | ||
| 45 | input_report_key(pwr, KEY_POWER, value & PWR_PWRON_IRQ); | 46 | input_report_key(pwr, KEY_POWER, value & PWR_PWRON_IRQ); |
| 46 | input_sync(pwr); | 47 | input_sync(pwr); |
| 47 | } else { | 48 | } else { |
diff --git a/drivers/input/misc/uinput.c b/drivers/input/misc/uinput.c index 6b1797503e3..a0a4bbaef02 100644 --- a/drivers/input/misc/uinput.c +++ b/drivers/input/misc/uinput.c | |||
| @@ -40,7 +40,8 @@ | |||
| 40 | #include <linux/input/mt.h> | 40 | #include <linux/input/mt.h> |
| 41 | #include "../input-compat.h" | 41 | #include "../input-compat.h" |
| 42 | 42 | ||
| 43 | static int uinput_dev_event(struct input_dev *dev, unsigned int type, unsigned int code, int value) | 43 | static int uinput_dev_event(struct input_dev *dev, |
| 44 | unsigned int type, unsigned int code, int value) | ||
| 44 | { | 45 | { |
| 45 | struct uinput_device *udev = input_get_drvdata(dev); | 46 | struct uinput_device *udev = input_get_drvdata(dev); |
| 46 | 47 | ||
| @@ -56,10 +57,11 @@ static int uinput_dev_event(struct input_dev *dev, unsigned int type, unsigned i | |||
| 56 | } | 57 | } |
| 57 | 58 | ||
| 58 | /* Atomically allocate an ID for the given request. Returns 0 on success. */ | 59 | /* Atomically allocate an ID for the given request. Returns 0 on success. */ |
| 59 | static int uinput_request_alloc_id(struct uinput_device *udev, struct uinput_request *request) | 60 | static bool uinput_request_alloc_id(struct uinput_device *udev, |
| 61 | struct uinput_request *request) | ||
| 60 | { | 62 | { |
| 61 | int id; | 63 | unsigned int id; |
| 62 | int err = -1; | 64 | bool reserved = false; |
| 63 | 65 | ||
| 64 | spin_lock(&udev->requests_lock); | 66 | spin_lock(&udev->requests_lock); |
| 65 | 67 | ||
| @@ -67,32 +69,35 @@ static int uinput_request_alloc_id(struct uinput_device *udev, struct uinput_req | |||
| 67 | if (!udev->requests[id]) { | 69 | if (!udev->requests[id]) { |
| 68 | request->id = id; | 70 | request->id = id; |
| 69 | udev->requests[id] = request; | 71 | udev->requests[id] = request; |
| 70 | err = 0; | 72 | reserved = true; |
| 71 | break; | 73 | break; |
| 72 | } | 74 | } |
| 73 | } | 75 | } |
| 74 | 76 | ||
| 75 | spin_unlock(&udev->requests_lock); | 77 | spin_unlock(&udev->requests_lock); |
| 76 | return err; | 78 | return reserved; |
| 77 | } | 79 | } |
| 78 | 80 | ||
| 79 | static struct uinput_request *uinput_request_find(struct uinput_device *udev, int id) | 81 | static struct uinput_request *uinput_request_find(struct uinput_device *udev, |
| 82 | unsigned int id) | ||
| 80 | { | 83 | { |
| 81 | /* Find an input request, by ID. Returns NULL if the ID isn't valid. */ | 84 | /* Find an input request, by ID. Returns NULL if the ID isn't valid. */ |
| 82 | if (id >= UINPUT_NUM_REQUESTS || id < 0) | 85 | if (id >= UINPUT_NUM_REQUESTS) |
| 83 | return NULL; | 86 | return NULL; |
| 84 | 87 | ||
| 85 | return udev->requests[id]; | 88 | return udev->requests[id]; |
| 86 | } | 89 | } |
| 87 | 90 | ||
| 88 | static inline int uinput_request_reserve_slot(struct uinput_device *udev, struct uinput_request *request) | 91 | static int uinput_request_reserve_slot(struct uinput_device *udev, |
| 92 | struct uinput_request *request) | ||
| 89 | { | 93 | { |
| 90 | /* Allocate slot. If none are available right away, wait. */ | 94 | /* Allocate slot. If none are available right away, wait. */ |
| 91 | return wait_event_interruptible(udev->requests_waitq, | 95 | return wait_event_interruptible(udev->requests_waitq, |
| 92 | !uinput_request_alloc_id(udev, request)); | 96 | uinput_request_alloc_id(udev, request)); |
| 93 | } | 97 | } |
| 94 | 98 | ||
| 95 | static void uinput_request_done(struct uinput_device *udev, struct uinput_request *request) | 99 | static void uinput_request_done(struct uinput_device *udev, |
| 100 | struct uinput_request *request) | ||
| 96 | { | 101 | { |
| 97 | /* Mark slot as available */ | 102 | /* Mark slot as available */ |
| 98 | udev->requests[request->id] = NULL; | 103 | udev->requests[request->id] = NULL; |
| @@ -101,14 +106,11 @@ static void uinput_request_done(struct uinput_device *udev, struct uinput_reques | |||
| 101 | complete(&request->done); | 106 | complete(&request->done); |
| 102 | } | 107 | } |
| 103 | 108 | ||
| 104 | static int uinput_request_submit(struct uinput_device *udev, struct uinput_request *request) | 109 | static int uinput_request_send(struct uinput_device *udev, |
| 110 | struct uinput_request *request) | ||
| 105 | { | 111 | { |
| 106 | int retval; | 112 | int retval; |
| 107 | 113 | ||
| 108 | retval = uinput_request_reserve_slot(udev, request); | ||
| 109 | if (retval) | ||
| 110 | return retval; | ||
| 111 | |||
| 112 | retval = mutex_lock_interruptible(&udev->mutex); | 114 | retval = mutex_lock_interruptible(&udev->mutex); |
| 113 | if (retval) | 115 | if (retval) |
| 114 | return retval; | 116 | return retval; |
| @@ -118,7 +120,12 @@ static int uinput_request_submit(struct uinput_device *udev, struct uinput_reque | |||
| 118 | goto out; | 120 | goto out; |
| 119 | } | 121 | } |
| 120 | 122 | ||
| 121 | /* Tell our userspace app about this new request by queueing an input event */ | 123 | init_completion(&request->done); |
| 124 | |||
| 125 | /* | ||
| 126 | * Tell our userspace application about this new request | ||
| 127 | * by queueing an input event. | ||
| 128 | */ | ||
| 122 | uinput_dev_event(udev->dev, EV_UINPUT, request->code, request->id); | 129 | uinput_dev_event(udev->dev, EV_UINPUT, request->code, request->id); |
| 123 | 130 | ||
| 124 | out: | 131 | out: |
| @@ -126,8 +133,27 @@ static int uinput_request_submit(struct uinput_device *udev, struct uinput_reque | |||
| 126 | return retval; | 133 | return retval; |
| 127 | } | 134 | } |
| 128 | 135 | ||
| 136 | static int uinput_request_submit(struct uinput_device *udev, | ||
| 137 | struct uinput_request *request) | ||
| 138 | { | ||
| 139 | int error; | ||
| 140 | |||
| 141 | error = uinput_request_reserve_slot(udev, request); | ||
| 142 | if (error) | ||
| 143 | return error; | ||
| 144 | |||
| 145 | error = uinput_request_send(udev, request); | ||
| 146 | if (error) { | ||
| 147 | uinput_request_done(udev, request); | ||
| 148 | return error; | ||
| 149 | } | ||
| 150 | |||
| 151 | wait_for_completion(&request->done); | ||
| 152 | return request->retval; | ||
| 153 | } | ||
| 154 | |||
| 129 | /* | 155 | /* |
| 130 | * Fail all ouitstanding requests so handlers don't wait for the userspace | 156 | * Fail all outstanding requests so handlers don't wait for the userspace |
| 131 | * to finish processing them. | 157 | * to finish processing them. |
| 132 | */ | 158 | */ |
| 133 | static void uinput_flush_requests(struct uinput_device *udev) | 159 | static void uinput_flush_requests(struct uinput_device *udev) |
| @@ -163,11 +189,12 @@ static int uinput_dev_playback(struct input_dev *dev, int effect_id, int value) | |||
| 163 | return uinput_dev_event(dev, EV_FF, effect_id, value); | 189 | return uinput_dev_event(dev, EV_FF, effect_id, value); |
| 164 | } | 190 | } |
| 165 | 191 | ||
| 166 | static int uinput_dev_upload_effect(struct input_dev *dev, struct ff_effect *effect, struct ff_effect *old) | 192 | static int uinput_dev_upload_effect(struct input_dev *dev, |
| 193 | struct ff_effect *effect, | ||
| 194 | struct ff_effect *old) | ||
| 167 | { | 195 | { |
| 168 | struct uinput_device *udev = input_get_drvdata(dev); | 196 | struct uinput_device *udev = input_get_drvdata(dev); |
| 169 | struct uinput_request request; | 197 | struct uinput_request request; |
| 170 | int retval; | ||
| 171 | 198 | ||
| 172 | /* | 199 | /* |
| 173 | * uinput driver does not currently support periodic effects with | 200 | * uinput driver does not currently support periodic effects with |
| @@ -180,42 +207,25 @@ static int uinput_dev_upload_effect(struct input_dev *dev, struct ff_effect *eff | |||
| 180 | effect->u.periodic.waveform == FF_CUSTOM) | 207 | effect->u.periodic.waveform == FF_CUSTOM) |
| 181 | return -EINVAL; | 208 | return -EINVAL; |
| 182 | 209 | ||
| 183 | request.id = -1; | ||
| 184 | init_completion(&request.done); | ||
| 185 | request.code = UI_FF_UPLOAD; | 210 | request.code = UI_FF_UPLOAD; |
| 186 | request.u.upload.effect = effect; | 211 | request.u.upload.effect = effect; |
| 187 | request.u.upload.old = old; | 212 | request.u.upload.old = old; |
| 188 | 213 | ||
| 189 | retval = uinput_request_submit(udev, &request); | 214 | return uinput_request_submit(udev, &request); |
| 190 | if (!retval) { | ||
| 191 | wait_for_completion(&request.done); | ||
| 192 | retval = request.retval; | ||
| 193 | } | ||
| 194 | |||
| 195 | return retval; | ||
| 196 | } | 215 | } |
| 197 | 216 | ||
| 198 | static int uinput_dev_erase_effect(struct input_dev *dev, int effect_id) | 217 | static int uinput_dev_erase_effect(struct input_dev *dev, int effect_id) |
| 199 | { | 218 | { |
| 200 | struct uinput_device *udev = input_get_drvdata(dev); | 219 | struct uinput_device *udev = input_get_drvdata(dev); |
| 201 | struct uinput_request request; | 220 | struct uinput_request request; |
| 202 | int retval; | ||
| 203 | 221 | ||
| 204 | if (!test_bit(EV_FF, dev->evbit)) | 222 | if (!test_bit(EV_FF, dev->evbit)) |
| 205 | return -ENOSYS; | 223 | return -ENOSYS; |
| 206 | 224 | ||
| 207 | request.id = -1; | ||
| 208 | init_completion(&request.done); | ||
| 209 | request.code = UI_FF_ERASE; | 225 | request.code = UI_FF_ERASE; |
| 210 | request.u.effect_id = effect_id; | 226 | request.u.effect_id = effect_id; |
| 211 | 227 | ||
| 212 | retval = uinput_request_submit(udev, &request); | 228 | return uinput_request_submit(udev, &request); |
| 213 | if (!retval) { | ||
| 214 | wait_for_completion(&request.done); | ||
| 215 | retval = request.retval; | ||
| 216 | } | ||
| 217 | |||
| 218 | return retval; | ||
| 219 | } | 229 | } |
| 220 | 230 | ||
| 221 | static void uinput_destroy_device(struct uinput_device *udev) | 231 | static void uinput_destroy_device(struct uinput_device *udev) |
| @@ -347,7 +357,8 @@ static int uinput_allocate_device(struct uinput_device *udev) | |||
| 347 | return 0; | 357 | return 0; |
| 348 | } | 358 | } |
| 349 | 359 | ||
| 350 | static int uinput_setup_device(struct uinput_device *udev, const char __user *buffer, size_t count) | 360 | static int uinput_setup_device(struct uinput_device *udev, |
| 361 | const char __user *buffer, size_t count) | ||
| 351 | { | 362 | { |
| 352 | struct uinput_user_dev *user_dev; | 363 | struct uinput_user_dev *user_dev; |
| 353 | struct input_dev *dev; | 364 | struct input_dev *dev; |
| @@ -419,7 +430,8 @@ static int uinput_setup_device(struct uinput_device *udev, const char __user *bu | |||
| 419 | return retval; | 430 | return retval; |
| 420 | } | 431 | } |
| 421 | 432 | ||
| 422 | static inline ssize_t uinput_inject_event(struct uinput_device *udev, const char __user *buffer, size_t count) | 433 | static ssize_t uinput_inject_event(struct uinput_device *udev, |
| 434 | const char __user *buffer, size_t count) | ||
| 423 | { | 435 | { |
| 424 | struct input_event ev; | 436 | struct input_event ev; |
| 425 | 437 | ||
| @@ -434,11 +446,15 @@ static inline ssize_t uinput_inject_event(struct uinput_device *udev, const char | |||
| 434 | return input_event_size(); | 446 | return input_event_size(); |
| 435 | } | 447 | } |
| 436 | 448 | ||
| 437 | static ssize_t uinput_write(struct file *file, const char __user *buffer, size_t count, loff_t *ppos) | 449 | static ssize_t uinput_write(struct file *file, const char __user *buffer, |
| 450 | size_t count, loff_t *ppos) | ||
| 438 | { | 451 | { |
| 439 | struct uinput_device *udev = file->private_data; | 452 | struct uinput_device *udev = file->private_data; |
| 440 | int retval; | 453 | int retval; |
| 441 | 454 | ||
| 455 | if (count == 0) | ||
| 456 | return 0; | ||
| 457 | |||
| 442 | retval = mutex_lock_interruptible(&udev->mutex); | 458 | retval = mutex_lock_interruptible(&udev->mutex); |
| 443 | if (retval) | 459 | if (retval) |
| 444 | return retval; | 460 | return retval; |
| @@ -452,42 +468,74 @@ static ssize_t uinput_write(struct file *file, const char __user *buffer, size_t | |||
| 452 | return retval; | 468 | return retval; |
| 453 | } | 469 | } |
| 454 | 470 | ||
| 455 | static ssize_t uinput_read(struct file *file, char __user *buffer, size_t count, loff_t *ppos) | 471 | static bool uinput_fetch_next_event(struct uinput_device *udev, |
| 472 | struct input_event *event) | ||
| 456 | { | 473 | { |
| 457 | struct uinput_device *udev = file->private_data; | 474 | bool have_event; |
| 458 | int retval = 0; | ||
| 459 | 475 | ||
| 460 | if (udev->state != UIST_CREATED) | 476 | spin_lock_irq(&udev->dev->event_lock); |
| 461 | return -ENODEV; | ||
| 462 | 477 | ||
| 463 | if (udev->head == udev->tail && (file->f_flags & O_NONBLOCK)) | 478 | have_event = udev->head != udev->tail; |
| 464 | return -EAGAIN; | 479 | if (have_event) { |
| 480 | *event = udev->buff[udev->tail]; | ||
| 481 | udev->tail = (udev->tail + 1) % UINPUT_BUFFER_SIZE; | ||
| 482 | } | ||
| 465 | 483 | ||
| 466 | retval = wait_event_interruptible(udev->waitq, | 484 | spin_unlock_irq(&udev->dev->event_lock); |
| 467 | udev->head != udev->tail || udev->state != UIST_CREATED); | ||
| 468 | if (retval) | ||
| 469 | return retval; | ||
| 470 | 485 | ||
| 471 | retval = mutex_lock_interruptible(&udev->mutex); | 486 | return have_event; |
| 472 | if (retval) | 487 | } |
| 473 | return retval; | ||
| 474 | 488 | ||
| 475 | if (udev->state != UIST_CREATED) { | 489 | static ssize_t uinput_events_to_user(struct uinput_device *udev, |
| 476 | retval = -ENODEV; | 490 | char __user *buffer, size_t count) |
| 477 | goto out; | 491 | { |
| 478 | } | 492 | struct input_event event; |
| 493 | size_t read = 0; | ||
| 479 | 494 | ||
| 480 | while (udev->head != udev->tail && retval + input_event_size() <= count) { | 495 | while (read + input_event_size() <= count && |
| 481 | if (input_event_to_user(buffer + retval, &udev->buff[udev->tail])) { | 496 | uinput_fetch_next_event(udev, &event)) { |
| 482 | retval = -EFAULT; | 497 | |
| 483 | goto out; | 498 | if (input_event_to_user(buffer + read, &event)) |
| 484 | } | 499 | return -EFAULT; |
| 485 | udev->tail = (udev->tail + 1) % UINPUT_BUFFER_SIZE; | 500 | |
| 486 | retval += input_event_size(); | 501 | read += input_event_size(); |
| 487 | } | 502 | } |
| 488 | 503 | ||
| 489 | out: | 504 | return read; |
| 490 | mutex_unlock(&udev->mutex); | 505 | } |
| 506 | |||
| 507 | static ssize_t uinput_read(struct file *file, char __user *buffer, | ||
| 508 | size_t count, loff_t *ppos) | ||
| 509 | { | ||
| 510 | struct uinput_device *udev = file->private_data; | ||
| 511 | ssize_t retval; | ||
| 512 | |||
| 513 | if (count != 0 && count < input_event_size()) | ||
| 514 | return -EINVAL; | ||
| 515 | |||
| 516 | do { | ||
| 517 | retval = mutex_lock_interruptible(&udev->mutex); | ||
| 518 | if (retval) | ||
| 519 | return retval; | ||
| 520 | |||
| 521 | if (udev->state != UIST_CREATED) | ||
| 522 | retval = -ENODEV; | ||
| 523 | else if (udev->head == udev->tail && | ||
| 524 | (file->f_flags & O_NONBLOCK)) | ||
| 525 | retval = -EAGAIN; | ||
| 526 | else | ||
| 527 | retval = uinput_events_to_user(udev, buffer, count); | ||
| 528 | |||
| 529 | mutex_unlock(&udev->mutex); | ||
| 530 | |||
| 531 | if (retval || count == 0) | ||
| 532 | break; | ||
| 533 | |||
| 534 | if (!(file->f_flags & O_NONBLOCK)) | ||
| 535 | retval = wait_event_interruptible(udev->waitq, | ||
| 536 | udev->head != udev->tail || | ||
| 537 | udev->state != UIST_CREATED); | ||
| 538 | } while (retval == 0); | ||
| 491 | 539 | ||
| 492 | return retval; | 540 | return retval; |
| 493 | } | 541 | } |
| @@ -516,8 +564,8 @@ static int uinput_release(struct inode *inode, struct file *file) | |||
| 516 | 564 | ||
| 517 | #ifdef CONFIG_COMPAT | 565 | #ifdef CONFIG_COMPAT |
| 518 | struct uinput_ff_upload_compat { | 566 | struct uinput_ff_upload_compat { |
| 519 | int request_id; | 567 | __u32 request_id; |
| 520 | int retval; | 568 | __s32 retval; |
| 521 | struct ff_effect_compat effect; | 569 | struct ff_effect_compat effect; |
| 522 | struct ff_effect_compat old; | 570 | struct ff_effect_compat old; |
| 523 | }; | 571 | }; |
| @@ -703,7 +751,8 @@ static long uinput_ioctl_handler(struct file *file, unsigned int cmd, | |||
| 703 | break; | 751 | break; |
| 704 | 752 | ||
| 705 | req = uinput_request_find(udev, ff_up.request_id); | 753 | req = uinput_request_find(udev, ff_up.request_id); |
| 706 | if (!req || req->code != UI_FF_UPLOAD || !req->u.upload.effect) { | 754 | if (!req || req->code != UI_FF_UPLOAD || |
| 755 | !req->u.upload.effect) { | ||
| 707 | retval = -EINVAL; | 756 | retval = -EINVAL; |
| 708 | break; | 757 | break; |
| 709 | } | 758 | } |
| @@ -786,7 +835,8 @@ static long uinput_ioctl(struct file *file, unsigned int cmd, unsigned long arg) | |||
| 786 | } | 835 | } |
| 787 | 836 | ||
| 788 | #ifdef CONFIG_COMPAT | 837 | #ifdef CONFIG_COMPAT |
| 789 | static long uinput_compat_ioctl(struct file *file, unsigned int cmd, unsigned long arg) | 838 | static long uinput_compat_ioctl(struct file *file, |
| 839 | unsigned int cmd, unsigned long arg) | ||
| 790 | { | 840 | { |
| 791 | return uinput_ioctl_handler(file, cmd, arg, compat_ptr(arg)); | 841 | return uinput_ioctl_handler(file, cmd, arg, compat_ptr(arg)); |
| 792 | } | 842 | } |
| @@ -831,4 +881,3 @@ MODULE_VERSION("0.3"); | |||
| 831 | 881 | ||
| 832 | module_init(uinput_init); | 882 | module_init(uinput_init); |
| 833 | module_exit(uinput_exit); | 883 | module_exit(uinput_exit); |
| 834 | |||
