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

Input: switch back to struct platform_driver::remove()

After commit 0edb555a65d1 ("platform: Make platform_driver::remove()
return void") .remove() is (again) the right callback to implement for
platform drivers.

Convert all platform drivers below drivers/input/ to use .remove(), with
the eventual goal to drop struct platform_driver::remove_new(). As
.remove() and .remove_new() have the same prototypes, conversion is done
by just changing the structure member name in the driver initializer.

While touching these files, make indention of the struct initializer
consistent in a few drivers.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@baylibre.com>
Link: https://lore.kernel.org/r/20241008090009.462836-2-u.kleine-koenig@baylibre.com
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>

authored by

Uwe Kleine-König and committed by
Dmitry Torokhov
2c19d015 1ec46bf0

+56 -56
+1 -1
drivers/input/keyboard/adp5520-keys.c
··· 181 181 .name = "adp5520-keys", 182 182 }, 183 183 .probe = adp5520_keys_probe, 184 - .remove_new = adp5520_keys_remove, 184 + .remove = adp5520_keys_remove, 185 185 }; 186 186 module_platform_driver(adp5520_keys_driver); 187 187
+1 -1
drivers/input/keyboard/cros_ec_keyb.c
··· 770 770 771 771 static struct platform_driver cros_ec_keyb_driver = { 772 772 .probe = cros_ec_keyb_probe, 773 - .remove_new = cros_ec_keyb_remove, 773 + .remove = cros_ec_keyb_remove, 774 774 .driver = { 775 775 .name = "cros-ec-keyb", 776 776 .dev_groups = cros_ec_keyb_groups,
+1 -1
drivers/input/keyboard/ep93xx_keypad.c
··· 315 315 .pm = pm_sleep_ptr(&ep93xx_keypad_pm_ops), 316 316 }, 317 317 .probe = ep93xx_keypad_probe, 318 - .remove_new = ep93xx_keypad_remove, 318 + .remove = ep93xx_keypad_remove, 319 319 }; 320 320 module_platform_driver(ep93xx_keypad_driver); 321 321
+1 -1
drivers/input/keyboard/iqs62x-keys.c
··· 323 323 .name = "iqs62x-keys", 324 324 }, 325 325 .probe = iqs62x_keys_probe, 326 - .remove_new = iqs62x_keys_remove, 326 + .remove = iqs62x_keys_remove, 327 327 }; 328 328 module_platform_driver(iqs62x_keys_platform_driver); 329 329
+1 -1
drivers/input/keyboard/omap-keypad.c
··· 290 290 291 291 static struct platform_driver omap_kp_driver = { 292 292 .probe = omap_kp_probe, 293 - .remove_new = omap_kp_remove, 293 + .remove = omap_kp_remove, 294 294 .driver = { 295 295 .name = "omap-keypad", 296 296 .dev_groups = omap_kp_groups,
+1 -1
drivers/input/keyboard/omap4-keypad.c
··· 485 485 486 486 static struct platform_driver omap4_keypad_driver = { 487 487 .probe = omap4_keypad_probe, 488 - .remove_new = omap4_keypad_remove, 488 + .remove = omap4_keypad_remove, 489 489 .driver = { 490 490 .name = "omap4-keypad", 491 491 .of_match_table = omap_keypad_dt_match,
+1 -1
drivers/input/keyboard/samsung-keypad.c
··· 587 587 588 588 static struct platform_driver samsung_keypad_driver = { 589 589 .probe = samsung_keypad_probe, 590 - .remove_new = samsung_keypad_remove, 590 + .remove = samsung_keypad_remove, 591 591 .driver = { 592 592 .name = "samsung-keypad", 593 593 .of_match_table = of_match_ptr(samsung_keypad_dt_match),
+1 -1
drivers/input/keyboard/sh_keysc.c
··· 319 319 320 320 static struct platform_driver sh_keysc_device_driver = { 321 321 .probe = sh_keysc_probe, 322 - .remove_new = sh_keysc_remove, 322 + .remove = sh_keysc_remove, 323 323 .driver = { 324 324 .name = "sh_keysc", 325 325 .pm = pm_sleep_ptr(&sh_keysc_dev_pm_ops),
+1 -1
drivers/input/keyboard/stmpe-keypad.c
··· 414 414 static struct platform_driver stmpe_keypad_driver = { 415 415 .driver.name = "stmpe-keypad", 416 416 .probe = stmpe_keypad_probe, 417 - .remove_new = stmpe_keypad_remove, 417 + .remove = stmpe_keypad_remove, 418 418 }; 419 419 module_platform_driver(stmpe_keypad_driver); 420 420
+1 -1
drivers/input/misc/88pm80x_onkey.c
··· 141 141 .pm = &pm80x_onkey_pm_ops, 142 142 }, 143 143 .probe = pm80x_onkey_probe, 144 - .remove_new = pm80x_onkey_remove, 144 + .remove = pm80x_onkey_remove, 145 145 }; 146 146 147 147 module_platform_driver(pm80x_onkey_driver);
+2 -2
drivers/input/misc/da9052_onkey.c
··· 140 140 141 141 static struct platform_driver da9052_onkey_driver = { 142 142 .probe = da9052_onkey_probe, 143 - .remove_new = da9052_onkey_remove, 144 - .driver = { 143 + .remove = da9052_onkey_remove, 144 + .driver = { 145 145 .name = "da9052-onkey", 146 146 }, 147 147 };
+2 -2
drivers/input/misc/da9055_onkey.c
··· 145 145 146 146 static struct platform_driver da9055_onkey_driver = { 147 147 .probe = da9055_onkey_probe, 148 - .remove_new = da9055_onkey_remove, 149 - .driver = { 148 + .remove = da9055_onkey_remove, 149 + .driver = { 150 150 .name = "da9055-onkey", 151 151 }, 152 152 };
+1 -1
drivers/input/misc/ideapad_slidebar.c
··· 255 255 .driver = { 256 256 .name = "ideapad_slidebar", 257 257 }, 258 - .remove_new = ideapad_remove, 258 + .remove = ideapad_remove, 259 259 }; 260 260 261 261 static int __init ideapad_dmi_check(const struct dmi_system_id *id)
+1 -1
drivers/input/misc/m68kspkr.c
··· 95 95 .name = "m68kspkr", 96 96 }, 97 97 .probe = m68kspkr_probe, 98 - .remove_new = m68kspkr_remove, 98 + .remove = m68kspkr_remove, 99 99 .shutdown = m68kspkr_shutdown, 100 100 }; 101 101
+1 -1
drivers/input/misc/max8997_haptic.c
··· 384 384 .pm = pm_sleep_ptr(&max8997_haptic_pm_ops), 385 385 }, 386 386 .probe = max8997_haptic_probe, 387 - .remove_new = max8997_haptic_remove, 387 + .remove = max8997_haptic_remove, 388 388 .id_table = max8997_haptic_id, 389 389 }; 390 390 module_platform_driver(max8997_haptic_driver);
+1 -1
drivers/input/misc/mc13783-pwrbutton.c
··· 253 253 254 254 static struct platform_driver mc13783_pwrbutton_driver = { 255 255 .probe = mc13783_pwrbutton_probe, 256 - .remove_new = mc13783_pwrbutton_remove, 256 + .remove = mc13783_pwrbutton_remove, 257 257 .driver = { 258 258 .name = "mc13783-pwrbutton", 259 259 },
+1 -1
drivers/input/misc/palmas-pwrbutton.c
··· 310 310 311 311 static struct platform_driver palmas_pwron_driver = { 312 312 .probe = palmas_pwron_probe, 313 - .remove_new = palmas_pwron_remove, 313 + .remove = palmas_pwron_remove, 314 314 .driver = { 315 315 .name = "palmas_pwrbutton", 316 316 .of_match_table = of_match_ptr(of_palmas_pwr_match),
+1 -1
drivers/input/misc/pcap_keys.c
··· 112 112 113 113 static struct platform_driver pcap_keys_device_driver = { 114 114 .probe = pcap_keys_probe, 115 - .remove_new = pcap_keys_remove, 115 + .remove = pcap_keys_remove, 116 116 .driver = { 117 117 .name = "pcap-keys", 118 118 }
+1 -1
drivers/input/misc/pcf50633-input.c
··· 103 103 .name = "pcf50633-input", 104 104 }, 105 105 .probe = pcf50633_input_probe, 106 - .remove_new = pcf50633_input_remove, 106 + .remove = pcf50633_input_remove, 107 107 }; 108 108 module_platform_driver(pcf50633_input_driver); 109 109
+1 -1
drivers/input/misc/pcspkr.c
··· 127 127 .pm = &pcspkr_pm_ops, 128 128 }, 129 129 .probe = pcspkr_probe, 130 - .remove_new = pcspkr_remove, 130 + .remove = pcspkr_remove, 131 131 .shutdown = pcspkr_shutdown, 132 132 }; 133 133 module_platform_driver(pcspkr_platform_driver);
+1 -1
drivers/input/misc/pm8941-pwrkey.c
··· 465 465 466 466 static struct platform_driver pm8941_pwrkey_driver = { 467 467 .probe = pm8941_pwrkey_probe, 468 - .remove_new = pm8941_pwrkey_remove, 468 + .remove = pm8941_pwrkey_remove, 469 469 .driver = { 470 470 .name = "pm8941-pwrkey", 471 471 .pm = pm_sleep_ptr(&pm8941_pwr_key_pm_ops),
+1 -1
drivers/input/misc/soc_button_array.c
··· 612 612 613 613 static struct platform_driver soc_button_driver = { 614 614 .probe = soc_button_probe, 615 - .remove_new = soc_button_remove, 615 + .remove = soc_button_remove, 616 616 .driver = { 617 617 .name = KBUILD_MODNAME, 618 618 .acpi_match_table = ACPI_PTR(soc_button_acpi_match),
+2 -2
drivers/input/misc/sparcspkr.c
··· 256 256 .of_match_table = bbc_beep_match, 257 257 }, 258 258 .probe = bbc_beep_probe, 259 - .remove_new = bbc_remove, 259 + .remove = bbc_remove, 260 260 .shutdown = sparcspkr_shutdown, 261 261 }; 262 262 ··· 334 334 .of_match_table = grover_beep_match, 335 335 }, 336 336 .probe = grover_beep_probe, 337 - .remove_new = grover_remove, 337 + .remove = grover_remove, 338 338 .shutdown = sparcspkr_shutdown, 339 339 }; 340 340
+1 -1
drivers/input/misc/tps65219-pwrbutton.c
··· 137 137 138 138 static struct platform_driver tps65219_pb_driver = { 139 139 .probe = tps65219_pb_probe, 140 - .remove_new = tps65219_pb_remove, 140 + .remove = tps65219_pb_remove, 141 141 .driver = { 142 142 .name = "tps65219_pwrbutton", 143 143 },
+1 -1
drivers/input/misc/wistron_btns.c
··· 1334 1334 .pm = pm_sleep_ptr(&wistron_pm_ops), 1335 1335 }, 1336 1336 .probe = wistron_probe, 1337 - .remove_new = wistron_remove, 1337 + .remove = wistron_remove, 1338 1338 }; 1339 1339 1340 1340 static int __init wb_module_init(void)
+1 -1
drivers/input/misc/wm831x-on.c
··· 134 134 135 135 static struct platform_driver wm831x_on_driver = { 136 136 .probe = wm831x_on_probe, 137 - .remove_new = wm831x_on_remove, 137 + .remove = wm831x_on_remove, 138 138 .driver = { 139 139 .name = "wm831x-on", 140 140 },
+1 -1
drivers/input/mouse/amimouse.c
··· 139 139 * triggering a section mismatch warning. 140 140 */ 141 141 static struct platform_driver amimouse_driver __refdata = { 142 - .remove_new = __exit_p(amimouse_remove), 142 + .remove = __exit_p(amimouse_remove), 143 143 .driver = { 144 144 .name = "amiga-mouse", 145 145 },
+1 -1
drivers/input/serio/altera_ps2.c
··· 146 146 */ 147 147 static struct platform_driver altera_ps2_driver = { 148 148 .probe = altera_ps2_probe, 149 - .remove_new = altera_ps2_remove, 149 + .remove = altera_ps2_remove, 150 150 .driver = { 151 151 .name = DRV_NAME, 152 152 .of_match_table = of_match_ptr(altera_ps2_match),
+1 -1
drivers/input/serio/ams_delta_serio.c
··· 182 182 183 183 static struct platform_driver ams_delta_serio_driver = { 184 184 .probe = ams_delta_serio_init, 185 - .remove_new = ams_delta_serio_exit, 185 + .remove = ams_delta_serio_exit, 186 186 .driver = { 187 187 .name = DRIVER_NAME 188 188 },
+1 -1
drivers/input/serio/apbps2.c
··· 208 208 .of_match_table = apbps2_of_match, 209 209 }, 210 210 .probe = apbps2_of_probe, 211 - .remove_new = apbps2_of_remove, 211 + .remove = apbps2_of_remove, 212 212 }; 213 213 214 214 module_platform_driver(apbps2_of_driver);
+1 -1
drivers/input/serio/arc_ps2.c
··· 260 260 .of_match_table = of_match_ptr(arc_ps2_match), 261 261 }, 262 262 .probe = arc_ps2_probe, 263 - .remove_new = arc_ps2_remove, 263 + .remove = arc_ps2_remove, 264 264 }; 265 265 266 266 module_platform_driver(arc_ps2_driver);
+1 -1
drivers/input/serio/ct82c710.c
··· 190 190 .name = "ct82c710", 191 191 }, 192 192 .probe = ct82c710_probe, 193 - .remove_new = ct82c710_remove, 193 + .remove = ct82c710_remove, 194 194 }; 195 195 196 196
+1 -1
drivers/input/serio/i8042-sparcio.h
··· 101 101 .of_match_table = sparc_i8042_match, 102 102 }, 103 103 .probe = sparc_i8042_probe, 104 - .remove_new = sparc_i8042_remove, 104 + .remove = sparc_i8042_remove, 105 105 }; 106 106 107 107 static bool i8042_is_mr_coffee(void)
+1 -1
drivers/input/serio/i8042.c
··· 1590 1590 #endif 1591 1591 }, 1592 1592 .probe = i8042_probe, 1593 - .remove_new = i8042_remove, 1593 + .remove = i8042_remove, 1594 1594 .shutdown = i8042_shutdown, 1595 1595 }; 1596 1596
+1 -1
drivers/input/serio/ioc3kbd.c
··· 208 208 209 209 static struct platform_driver ioc3kbd_driver = { 210 210 .probe = ioc3kbd_probe, 211 - .remove_new = ioc3kbd_remove, 211 + .remove = ioc3kbd_remove, 212 212 .id_table = ioc3kbd_id_table, 213 213 .driver = { 214 214 .name = "ioc3-kbd",
+1 -1
drivers/input/serio/maceps2.c
··· 159 159 .name = "maceps2", 160 160 }, 161 161 .probe = maceps2_probe, 162 - .remove_new = maceps2_remove, 162 + .remove = maceps2_remove, 163 163 }; 164 164 165 165 static int __init maceps2_init(void)
+1 -1
drivers/input/serio/olpc_apsp.c
··· 256 256 257 257 static struct platform_driver olpc_apsp_driver = { 258 258 .probe = olpc_apsp_probe, 259 - .remove_new = olpc_apsp_remove, 259 + .remove = olpc_apsp_remove, 260 260 .driver = { 261 261 .name = "olpc-apsp", 262 262 .of_match_table = olpc_apsp_dt_ids,
+1 -1
drivers/input/serio/ps2-gpio.c
··· 491 491 492 492 static struct platform_driver ps2_gpio_driver = { 493 493 .probe = ps2_gpio_probe, 494 - .remove_new = ps2_gpio_remove, 494 + .remove = ps2_gpio_remove, 495 495 .driver = { 496 496 .name = DRIVER_NAME, 497 497 .of_match_table = of_match_ptr(ps2_gpio_match),
+1 -1
drivers/input/serio/q40kbd.c
··· 160 160 .driver = { 161 161 .name = "q40kbd", 162 162 }, 163 - .remove_new = q40kbd_remove, 163 + .remove = q40kbd_remove, 164 164 }; 165 165 166 166 module_platform_driver_probe(q40kbd_driver, q40kbd_probe);
+1 -1
drivers/input/serio/rpckbd.c
··· 144 144 145 145 static struct platform_driver rpckbd_driver = { 146 146 .probe = rpckbd_probe, 147 - .remove_new = rpckbd_remove, 147 + .remove = rpckbd_remove, 148 148 .driver = { 149 149 .name = "kart", 150 150 },
+1 -1
drivers/input/serio/sun4i-ps2.c
··· 318 318 319 319 static struct platform_driver sun4i_ps2_driver = { 320 320 .probe = sun4i_ps2_probe, 321 - .remove_new = sun4i_ps2_remove, 321 + .remove = sun4i_ps2_remove, 322 322 .driver = { 323 323 .name = DRIVER_NAME, 324 324 .of_match_table = sun4i_ps2_match,
+1 -1
drivers/input/serio/xilinx_ps2.c
··· 353 353 .of_match_table = xps2_of_match, 354 354 }, 355 355 .probe = xps2_of_probe, 356 - .remove_new = xps2_of_remove, 356 + .remove = xps2_of_remove, 357 357 }; 358 358 module_platform_driver(xps2_of_driver); 359 359
+1 -1
drivers/input/touchscreen/da9052_tsi.c
··· 326 326 327 327 static struct platform_driver da9052_tsi_driver = { 328 328 .probe = da9052_ts_probe, 329 - .remove_new = da9052_ts_remove, 329 + .remove = da9052_ts_remove, 330 330 .driver = { 331 331 .name = "da9052-tsi", 332 332 },
+1 -1
drivers/input/touchscreen/mainstone-wm97xx.c
··· 261 261 262 262 static struct platform_driver mainstone_wm97xx_driver = { 263 263 .probe = mainstone_wm97xx_probe, 264 - .remove_new = mainstone_wm97xx_remove, 264 + .remove = mainstone_wm97xx_remove, 265 265 .driver = { 266 266 .name = "wm97xx-touch", 267 267 },
+1 -1
drivers/input/touchscreen/mc13783_ts.c
··· 226 226 } 227 227 228 228 static struct platform_driver mc13783_ts_driver = { 229 - .remove_new = mc13783_ts_remove, 229 + .remove = mc13783_ts_remove, 230 230 .driver = { 231 231 .name = MC13783_TS_NAME, 232 232 },
+1 -1
drivers/input/touchscreen/pcap_ts.c
··· 238 238 239 239 static struct platform_driver pcap_ts_driver = { 240 240 .probe = pcap_ts_probe, 241 - .remove_new = pcap_ts_remove, 241 + .remove = pcap_ts_remove, 242 242 .driver = { 243 243 .name = "pcap-ts", 244 244 .pm = PCAP_TS_PM_OPS,
+1 -1
drivers/input/touchscreen/stmpe-ts.c
··· 362 362 .name = STMPE_TS_NAME, 363 363 }, 364 364 .probe = stmpe_input_probe, 365 - .remove_new = stmpe_ts_remove, 365 + .remove = stmpe_ts_remove, 366 366 }; 367 367 module_platform_driver(stmpe_ts_driver); 368 368
+2 -2
drivers/input/touchscreen/sun4i-ts.c
··· 396 396 MODULE_DEVICE_TABLE(of, sun4i_ts_of_match); 397 397 398 398 static struct platform_driver sun4i_ts_driver = { 399 - .driver = { 399 + .driver = { 400 400 .name = "sun4i-ts", 401 401 .of_match_table = sun4i_ts_of_match, 402 402 }, 403 403 .probe = sun4i_ts_probe, 404 - .remove_new = sun4i_ts_remove, 404 + .remove = sun4i_ts_remove, 405 405 }; 406 406 407 407 module_platform_driver(sun4i_ts_driver);
+2 -2
drivers/input/touchscreen/ti_am335x_tsc.c
··· 550 550 551 551 static struct platform_driver ti_tsc_driver = { 552 552 .probe = titsc_probe, 553 - .remove_new = titsc_remove, 553 + .remove = titsc_remove, 554 554 .driver = { 555 - .name = "TI-am335x-tsc", 555 + .name = "TI-am335x-tsc", 556 556 .pm = pm_sleep_ptr(&titsc_pm_ops), 557 557 .of_match_table = ti_tsc_dt_ids, 558 558 },
+1 -1
drivers/input/touchscreen/wm831x-ts.c
··· 387 387 .name = "wm831x-touch", 388 388 }, 389 389 .probe = wm831x_ts_probe, 390 - .remove_new = wm831x_ts_remove, 390 + .remove = wm831x_ts_remove, 391 391 }; 392 392 module_platform_driver(wm831x_ts_driver); 393 393
+1 -1
drivers/input/touchscreen/wm97xx-core.c
··· 876 876 .pm = pm_sleep_ptr(&wm97xx_pm_ops), 877 877 }, 878 878 .probe = wm97xx_mfd_probe, 879 - .remove_new = wm97xx_mfd_remove, 879 + .remove = wm97xx_mfd_remove, 880 880 }; 881 881 882 882 static int __init wm97xx_init(void)