Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

Input: keyboard - use local variables consistently

If a function declares a variable to access a structure element,
use it consistently.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>

authored by

Guenter Roeck and committed by
Dmitry Torokhov
b4e66e7d 8300445c

+25 -27
+2 -2
drivers/input/keyboard/davinci_keyscan.c
··· 172 172 struct input_dev *key_dev; 173 173 struct resource *res, *mem; 174 174 struct device *dev = &pdev->dev; 175 - struct davinci_ks_platform_data *pdata = dev_get_platdata(&pdev->dev); 175 + struct davinci_ks_platform_data *pdata = dev_get_platdata(dev); 176 176 int error, i; 177 177 178 178 if (pdata->device_enable) { ··· 255 255 256 256 key_dev->name = "davinci_keyscan"; 257 257 key_dev->phys = "davinci_keyscan/input0"; 258 - key_dev->dev.parent = &pdev->dev; 258 + key_dev->dev.parent = dev; 259 259 key_dev->id.bustype = BUS_HOST; 260 260 key_dev->id.vendor = 0x0001; 261 261 key_dev->id.product = 0x0001;
+11 -13
drivers/input/keyboard/gpio_keys.c
··· 518 518 if (button->active_low) 519 519 flags |= GPIOF_ACTIVE_LOW; 520 520 521 - error = devm_gpio_request_one(&pdev->dev, button->gpio, flags, 522 - desc); 521 + error = devm_gpio_request_one(dev, button->gpio, flags, desc); 523 522 if (error < 0) { 524 523 dev_err(dev, "Failed to request GPIO %d, error %d\n", 525 524 button->gpio, error); ··· 588 589 * Install custom action to cancel release timer and 589 590 * workqueue item. 590 591 */ 591 - error = devm_add_action(&pdev->dev, gpio_keys_quiesce_key, bdata); 592 + error = devm_add_action(dev, gpio_keys_quiesce_key, bdata); 592 593 if (error) { 593 - dev_err(&pdev->dev, 594 - "failed to register quiesce action, error: %d\n", 594 + dev_err(dev, "failed to register quiesce action, error: %d\n", 595 595 error); 596 596 return error; 597 597 } ··· 602 604 if (!button->can_disable) 603 605 irqflags |= IRQF_SHARED; 604 606 605 - error = devm_request_any_context_irq(&pdev->dev, bdata->irq, 606 - isr, irqflags, desc, bdata); 607 + error = devm_request_any_context_irq(dev, bdata->irq, isr, irqflags, 608 + desc, bdata); 607 609 if (error < 0) { 608 610 dev_err(dev, "Unable to claim irq %d; error %d\n", 609 611 bdata->irq, error); ··· 775 777 776 778 input->name = pdata->name ? : pdev->name; 777 779 input->phys = "gpio-keys/input0"; 778 - input->dev.parent = &pdev->dev; 780 + input->dev.parent = dev; 779 781 input->open = gpio_keys_open; 780 782 input->close = gpio_keys_close; 781 783 ··· 796 798 const struct gpio_keys_button *button = &pdata->buttons[i]; 797 799 798 800 if (!dev_get_platdata(dev)) { 799 - child = device_get_next_child_node(&pdev->dev, child); 801 + child = device_get_next_child_node(dev, child); 800 802 if (!child) { 801 - dev_err(&pdev->dev, 803 + dev_err(dev, 802 804 "missing child device node for entry %d\n", 803 805 i); 804 806 return -EINVAL; ··· 818 820 819 821 fwnode_handle_put(child); 820 822 821 - error = sysfs_create_group(&pdev->dev.kobj, &gpio_keys_attr_group); 823 + error = sysfs_create_group(&dev->kobj, &gpio_keys_attr_group); 822 824 if (error) { 823 825 dev_err(dev, "Unable to export keys/switches, error: %d\n", 824 826 error); ··· 832 834 goto err_remove_group; 833 835 } 834 836 835 - device_init_wakeup(&pdev->dev, wakeup); 837 + device_init_wakeup(dev, wakeup); 836 838 837 839 return 0; 838 840 839 841 err_remove_group: 840 - sysfs_remove_group(&pdev->dev.kobj, &gpio_keys_attr_group); 842 + sysfs_remove_group(&dev->kobj, &gpio_keys_attr_group); 841 843 return error; 842 844 } 843 845
+3 -3
drivers/input/keyboard/gpio_keys_polled.c
··· 252 252 253 253 size = sizeof(struct gpio_keys_polled_dev) + 254 254 pdata->nbuttons * sizeof(struct gpio_keys_button_data); 255 - bdev = devm_kzalloc(&pdev->dev, size, GFP_KERNEL); 255 + bdev = devm_kzalloc(dev, size, GFP_KERNEL); 256 256 if (!bdev) { 257 257 dev_err(dev, "no memory for private data\n"); 258 258 return -ENOMEM; 259 259 } 260 260 261 - poll_dev = devm_input_allocate_polled_device(&pdev->dev); 261 + poll_dev = devm_input_allocate_polled_device(dev); 262 262 if (!poll_dev) { 263 263 dev_err(dev, "no memory for polled device\n"); 264 264 return -ENOMEM; ··· 332 332 if (button->active_low) 333 333 flags |= GPIOF_ACTIVE_LOW; 334 334 335 - error = devm_gpio_request_one(&pdev->dev, button->gpio, 335 + error = devm_gpio_request_one(dev, button->gpio, 336 336 flags, button->desc ? : DRV_NAME); 337 337 if (error) { 338 338 dev_err(dev,
+9 -9
drivers/input/keyboard/mpr121_touchkey.c
··· 237 237 int i; 238 238 239 239 if (!client->irq) { 240 - dev_err(&client->dev, "irq number should not be zero\n"); 240 + dev_err(dev, "irq number should not be zero\n"); 241 241 return -EINVAL; 242 242 } 243 243 ··· 247 247 248 248 vdd_uv = regulator_get_voltage(vdd_supply); 249 249 250 - mpr121 = devm_kzalloc(&client->dev, sizeof(*mpr121), GFP_KERNEL); 250 + mpr121 = devm_kzalloc(dev, sizeof(*mpr121), GFP_KERNEL); 251 251 if (!mpr121) 252 252 return -ENOMEM; 253 253 254 - input_dev = devm_input_allocate_device(&client->dev); 254 + input_dev = devm_input_allocate_device(dev); 255 255 if (!input_dev) 256 256 return -ENOMEM; 257 257 ··· 275 275 276 276 input_dev->name = "Freescale MPR121 Touchkey"; 277 277 input_dev->id.bustype = BUS_I2C; 278 - input_dev->dev.parent = &client->dev; 278 + input_dev->dev.parent = dev; 279 279 if (device_property_read_bool(dev, "autorepeat")) 280 280 __set_bit(EV_REP, input_dev->evbit); 281 281 input_set_capability(input_dev, EV_MSC, MSC_SCAN); ··· 289 289 290 290 error = mpr121_phys_init(mpr121, client, vdd_uv); 291 291 if (error) { 292 - dev_err(&client->dev, "Failed to init register\n"); 292 + dev_err(dev, "Failed to init register\n"); 293 293 return error; 294 294 } 295 295 296 - error = devm_request_threaded_irq(&client->dev, client->irq, 297 - NULL, mpr_touchkey_interrupt, 296 + error = devm_request_threaded_irq(dev, client->irq, NULL, 297 + mpr_touchkey_interrupt, 298 298 IRQF_TRIGGER_FALLING | IRQF_ONESHOT, 299 - client->dev.driver->name, mpr121); 299 + dev->driver->name, mpr121); 300 300 if (error) { 301 - dev_err(&client->dev, "Failed to register interrupt\n"); 301 + dev_err(dev, "Failed to register interrupt\n"); 302 302 return error; 303 303 } 304 304