powerpc/ep8248e: Mark driver as non removable
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Wed, 26 Jul 2023 08:14:42 +0000 (10:14 +0200)
committerMichael Ellerman <mpe@ellerman.id.au>
Wed, 2 Aug 2023 12:22:19 +0000 (22:22 +1000)
Instead of resorting to BUG() ensure that the driver isn't unbound by
suppressing its bind and unbind sysfs attributes. As the driver is
built-in there is no way to remove a device once bound.

As a nice side effect this allows to drop the remove function.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://msgid.link/20230726081442.461026-1-u.kleine-koenig@pengutronix.de
arch/powerpc/platforms/82xx/ep8248e.c

index 16808536f7888e697b235fcdd42249a3f631c247..4bfa1a95e15556ed85810598a39b2fe922d9f283 100644 (file)
@@ -141,12 +141,6 @@ err_free_bus:
        return ret;
 }
 
-static int ep8248e_mdio_remove(struct platform_device *ofdev)
-{
-       BUG();
-       return 0;
-}
-
 static const struct of_device_id ep8248e_mdio_match[] = {
        {
                .compatible = "fsl,ep8248e-mdio-bitbang",
@@ -158,9 +152,9 @@ static struct platform_driver ep8248e_mdio_driver = {
        .driver = {
                .name = "ep8248e-mdio-bitbang",
                .of_match_table = ep8248e_mdio_match,
+               .suppress_bind_attrs = true,
        },
        .probe = ep8248e_mdio_probe,
-       .remove = ep8248e_mdio_remove,
 };
 
 struct cpm_pin {