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

ata: 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/ata/ 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 several files.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@baylibre.com>
Reviewed-by: Damien Le Moal <dlemoal@kernel.org>
Link: https://lore.kernel.org/r/20241016141609.27329-2-u.kleine-koenig@baylibre.com
Signed-off-by: Niklas Cassel <cassel@kernel.org>

authored by

Uwe Kleine-König and committed by
Niklas Cassel
2d910fe1 f7a870d0

+41 -41
+1 -1
drivers/ata/ahci_brcm.c
··· 571 571 572 572 static struct platform_driver brcm_ahci_driver = { 573 573 .probe = brcm_ahci_probe, 574 - .remove_new = brcm_ahci_remove, 574 + .remove = brcm_ahci_remove, 575 575 .shutdown = brcm_ahci_shutdown, 576 576 .driver = { 577 577 .name = DRV_NAME,
+1 -1
drivers/ata/ahci_ceva.c
··· 402 402 403 403 static struct platform_driver ceva_ahci_driver = { 404 404 .probe = ceva_ahci_probe, 405 - .remove_new = ata_platform_remove_one, 405 + .remove = ata_platform_remove_one, 406 406 .driver = { 407 407 .name = DRV_NAME, 408 408 .of_match_table = ceva_ahci_of_match,
+1 -1
drivers/ata/ahci_da850.c
··· 238 238 239 239 static struct platform_driver ahci_da850_driver = { 240 240 .probe = ahci_da850_probe, 241 - .remove_new = ata_platform_remove_one, 241 + .remove = ata_platform_remove_one, 242 242 .driver = { 243 243 .name = DRV_NAME, 244 244 .of_match_table = ahci_da850_of_match,
+1 -1
drivers/ata/ahci_dm816.c
··· 182 182 183 183 static struct platform_driver ahci_dm816_driver = { 184 184 .probe = ahci_dm816_probe, 185 - .remove_new = ata_platform_remove_one, 185 + .remove = ata_platform_remove_one, 186 186 .driver = { 187 187 .name = AHCI_DM816_DRV_NAME, 188 188 .of_match_table = ahci_dm816_of_match,
+1 -1
drivers/ata/ahci_dwc.c
··· 478 478 479 479 static struct platform_driver ahci_dwc_driver = { 480 480 .probe = ahci_dwc_probe, 481 - .remove_new = ata_platform_remove_one, 481 + .remove = ata_platform_remove_one, 482 482 .shutdown = ahci_platform_shutdown, 483 483 .driver = { 484 484 .name = DRV_NAME,
+1 -1
drivers/ata/ahci_imx.c
··· 1027 1027 1028 1028 static struct platform_driver imx_ahci_driver = { 1029 1029 .probe = imx_ahci_probe, 1030 - .remove_new = ata_platform_remove_one, 1030 + .remove = ata_platform_remove_one, 1031 1031 .driver = { 1032 1032 .name = DRV_NAME, 1033 1033 .of_match_table = imx_ahci_of_match,
+1 -1
drivers/ata/ahci_mtk.c
··· 174 174 175 175 static struct platform_driver mtk_ahci_driver = { 176 176 .probe = mtk_ahci_probe, 177 - .remove_new = ata_platform_remove_one, 177 + .remove = ata_platform_remove_one, 178 178 .driver = { 179 179 .name = DRV_NAME, 180 180 .of_match_table = ahci_of_match,
+1 -1
drivers/ata/ahci_mvebu.c
··· 245 245 246 246 static struct platform_driver ahci_mvebu_driver = { 247 247 .probe = ahci_mvebu_probe, 248 - .remove_new = ata_platform_remove_one, 248 + .remove = ata_platform_remove_one, 249 249 .suspend = ahci_mvebu_suspend, 250 250 .resume = ahci_mvebu_resume, 251 251 .driver = {
+1 -1
drivers/ata/ahci_platform.c
··· 96 96 97 97 static struct platform_driver ahci_driver = { 98 98 .probe = ahci_probe, 99 - .remove_new = ata_platform_remove_one, 99 + .remove = ata_platform_remove_one, 100 100 .shutdown = ahci_platform_shutdown, 101 101 .driver = { 102 102 .name = DRV_NAME,
+1 -1
drivers/ata/ahci_qoriq.c
··· 357 357 358 358 static struct platform_driver ahci_qoriq_driver = { 359 359 .probe = ahci_qoriq_probe, 360 - .remove_new = ata_platform_remove_one, 360 + .remove = ata_platform_remove_one, 361 361 .driver = { 362 362 .name = DRV_NAME, 363 363 .of_match_table = ahci_qoriq_of_match,
+1 -1
drivers/ata/ahci_seattle.c
··· 185 185 186 186 static struct platform_driver ahci_seattle_driver = { 187 187 .probe = ahci_seattle_probe, 188 - .remove_new = ata_platform_remove_one, 188 + .remove = ata_platform_remove_one, 189 189 .driver = { 190 190 .name = DRV_NAME, 191 191 .acpi_match_table = ahci_acpi_match,
+1 -1
drivers/ata/ahci_st.c
··· 238 238 .of_match_table = st_ahci_match, 239 239 }, 240 240 .probe = st_ahci_probe, 241 - .remove_new = ata_platform_remove_one, 241 + .remove = ata_platform_remove_one, 242 242 }; 243 243 module_platform_driver(st_ahci_driver); 244 244
+1 -1
drivers/ata/ahci_sunxi.c
··· 292 292 293 293 static struct platform_driver ahci_sunxi_driver = { 294 294 .probe = ahci_sunxi_probe, 295 - .remove_new = ata_platform_remove_one, 295 + .remove = ata_platform_remove_one, 296 296 .driver = { 297 297 .name = DRV_NAME, 298 298 .of_match_table = ahci_sunxi_of_match,
+1 -1
drivers/ata/ahci_tegra.c
··· 608 608 609 609 static struct platform_driver tegra_ahci_driver = { 610 610 .probe = tegra_ahci_probe, 611 - .remove_new = ata_platform_remove_one, 611 + .remove = ata_platform_remove_one, 612 612 .driver = { 613 613 .name = DRV_NAME, 614 614 .of_match_table = tegra_ahci_of_match,
+1 -1
drivers/ata/ahci_xgene.c
··· 859 859 860 860 static struct platform_driver xgene_ahci_driver = { 861 861 .probe = xgene_ahci_probe, 862 - .remove_new = ata_platform_remove_one, 862 + .remove = ata_platform_remove_one, 863 863 .driver = { 864 864 .name = DRV_NAME, 865 865 .of_match_table = xgene_ahci_of_match,
+1 -1
drivers/ata/pata_arasan_cf.c
··· 964 964 965 965 static struct platform_driver arasan_cf_driver = { 966 966 .probe = arasan_cf_probe, 967 - .remove_new = arasan_cf_remove, 967 + .remove = arasan_cf_remove, 968 968 .driver = { 969 969 .name = DRIVER_NAME, 970 970 .pm = &arasan_cf_pm_ops,
+1 -1
drivers/ata/pata_ep93xx.c
··· 1015 1015 .of_match_table = ep93xx_pata_of_ids, 1016 1016 }, 1017 1017 .probe = ep93xx_pata_probe, 1018 - .remove_new = ep93xx_pata_remove, 1018 + .remove = ep93xx_pata_remove, 1019 1019 }; 1020 1020 1021 1021 module_platform_driver(ep93xx_pata_platform_driver);
+2 -2
drivers/ata/pata_falcon.c
··· 225 225 226 226 static struct platform_driver pata_falcon_driver = { 227 227 .probe = pata_falcon_init_one, 228 - .remove_new = pata_falcon_remove_one, 229 - .driver = { 228 + .remove = pata_falcon_remove_one, 229 + .driver = { 230 230 .name = "atari-falcon-ide", 231 231 }, 232 232 };
+1 -1
drivers/ata/pata_ftide010.c
··· 557 557 .of_match_table = pata_ftide010_of_match, 558 558 }, 559 559 .probe = pata_ftide010_probe, 560 - .remove_new = pata_ftide010_remove, 560 + .remove = pata_ftide010_remove, 561 561 }; 562 562 module_platform_driver(pata_ftide010_driver); 563 563
+3 -3
drivers/ata/pata_gayle.c
··· 202 202 203 203 static struct platform_driver pata_gayle_driver = { 204 204 .probe = pata_gayle_init_one, 205 - .remove_new = pata_gayle_remove_one, 206 - .driver = { 207 - .name = "amiga-gayle-ide", 205 + .remove = pata_gayle_remove_one, 206 + .driver = { 207 + .name = "amiga-gayle-ide", 208 208 }, 209 209 }; 210 210
+1 -1
drivers/ata/pata_imx.c
··· 249 249 250 250 static struct platform_driver pata_imx_driver = { 251 251 .probe = pata_imx_probe, 252 - .remove_new = pata_imx_remove, 252 + .remove = pata_imx_remove, 253 253 .driver = { 254 254 .name = DRV_NAME, 255 255 .of_match_table = imx_pata_dt_ids,
+1 -1
drivers/ata/pata_ixp4xx_cf.c
··· 298 298 .of_match_table = ixp4xx_pata_of_match, 299 299 }, 300 300 .probe = ixp4xx_pata_probe, 301 - .remove_new = ata_platform_remove_one, 301 + .remove = ata_platform_remove_one, 302 302 }; 303 303 304 304 module_platform_driver(ixp4xx_pata_platform_driver);
+1 -1
drivers/ata/pata_mpc52xx.c
··· 854 854 855 855 static struct platform_driver mpc52xx_ata_of_platform_driver = { 856 856 .probe = mpc52xx_ata_probe, 857 - .remove_new = mpc52xx_ata_remove, 857 + .remove = mpc52xx_ata_remove, 858 858 #ifdef CONFIG_PM_SLEEP 859 859 .suspend = mpc52xx_ata_suspend, 860 860 .resume = mpc52xx_ata_resume,
+1 -1
drivers/ata/pata_of_platform.c
··· 89 89 .of_match_table = pata_of_platform_match, 90 90 }, 91 91 .probe = pata_of_platform_probe, 92 - .remove_new = ata_platform_remove_one, 92 + .remove = ata_platform_remove_one, 93 93 }; 94 94 95 95 module_platform_driver(pata_of_platform_driver);
+1 -1
drivers/ata/pata_platform.c
··· 223 223 224 224 static struct platform_driver pata_platform_driver = { 225 225 .probe = pata_platform_probe, 226 - .remove_new = ata_platform_remove_one, 226 + .remove = ata_platform_remove_one, 227 227 .driver = { 228 228 .name = DRV_NAME, 229 229 },
+1 -1
drivers/ata/pata_pxa.c
··· 306 306 307 307 static struct platform_driver pxa_ata_driver = { 308 308 .probe = pxa_ata_probe, 309 - .remove_new = pxa_ata_remove, 309 + .remove = pxa_ata_remove, 310 310 .driver = { 311 311 .name = DRV_NAME, 312 312 },
+1 -1
drivers/ata/pata_rb532_cf.c
··· 164 164 165 165 static struct platform_driver rb532_pata_platform_driver = { 166 166 .probe = rb532_pata_driver_probe, 167 - .remove_new = rb532_pata_driver_remove, 167 + .remove = rb532_pata_driver_remove, 168 168 .driver = { 169 169 .name = DRV_NAME, 170 170 },
+1 -1
drivers/ata/sata_dwc_460ex.c
··· 1240 1240 .of_match_table = sata_dwc_match, 1241 1241 }, 1242 1242 .probe = sata_dwc_probe, 1243 - .remove_new = sata_dwc_remove, 1243 + .remove = sata_dwc_remove, 1244 1244 }; 1245 1245 1246 1246 module_platform_driver(sata_dwc_driver);
+1 -1
drivers/ata/sata_fsl.c
··· 1589 1589 .of_match_table = fsl_sata_match, 1590 1590 }, 1591 1591 .probe = sata_fsl_probe, 1592 - .remove_new = sata_fsl_remove, 1592 + .remove = sata_fsl_remove, 1593 1593 #ifdef CONFIG_PM_SLEEP 1594 1594 .suspend = sata_fsl_suspend, 1595 1595 .resume = sata_fsl_resume,
+1 -1
drivers/ata/sata_gemini.c
··· 425 425 .of_match_table = gemini_sata_of_match, 426 426 }, 427 427 .probe = gemini_sata_probe, 428 - .remove_new = gemini_sata_remove, 428 + .remove = gemini_sata_remove, 429 429 }; 430 430 module_platform_driver(gemini_sata_driver); 431 431
+6 -6
drivers/ata/sata_highbank.c
··· 614 614 ahci_highbank_suspend, ahci_highbank_resume); 615 615 616 616 static struct platform_driver ahci_highbank_driver = { 617 - .remove_new = ata_platform_remove_one, 618 - .driver = { 619 - .name = "highbank-ahci", 620 - .of_match_table = ahci_of_match, 621 - .pm = &ahci_highbank_pm_ops, 622 - }, 617 + .remove = ata_platform_remove_one, 618 + .driver = { 619 + .name = "highbank-ahci", 620 + .of_match_table = ahci_of_match, 621 + .pm = &ahci_highbank_pm_ops, 622 + }, 623 623 .probe = ahci_highbank_probe, 624 624 }; 625 625
+1 -1
drivers/ata/sata_mv.c
··· 4255 4255 4256 4256 static struct platform_driver mv_platform_driver = { 4257 4257 .probe = mv_platform_probe, 4258 - .remove_new = mv_platform_remove, 4258 + .remove = mv_platform_remove, 4259 4259 .suspend = mv_platform_suspend, 4260 4260 .resume = mv_platform_resume, 4261 4261 .driver = {
+1 -1
drivers/ata/sata_rcar.c
··· 1009 1009 1010 1010 static struct platform_driver sata_rcar_driver = { 1011 1011 .probe = sata_rcar_probe, 1012 - .remove_new = sata_rcar_remove, 1012 + .remove = sata_rcar_remove, 1013 1013 .driver = { 1014 1014 .name = DRV_NAME, 1015 1015 .of_match_table = sata_rcar_match,