ethernet: tundra: Drop forward declaration of static functions
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Mon, 19 Sep 2022 13:15:15 +0000 (15:15 +0200)
committerJakub Kicinski <kuba@kernel.org>
Thu, 22 Sep 2022 01:37:57 +0000 (18:37 -0700)
Usually it's not necessary to declare static functions if the symbols are
in the right order. Moving the definition of tsi_eth_driver down in the
compilation unit allows to drop two such declarations.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Link: https://lore.kernel.org/r/20220919131515.885361-1-u.kleine-koenig@pengutronix.de
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/tundra/tsi108_eth.c

index 5251fc3..c0b26b5 100644 (file)
@@ -59,9 +59,6 @@
 /* Check the phy status every half a second. */
 #define CHECK_PHY_INTERVAL (HZ/2)
 
-static int tsi108_init_one(struct platform_device *pdev);
-static int tsi108_ether_remove(struct platform_device *pdev);
-
 struct tsi108_prv_data {
        void  __iomem *regs;    /* Base of normal regs */
        void  __iomem *phyregs; /* Base of register bank used for PHY access */
@@ -144,16 +141,6 @@ struct tsi108_prv_data {
        struct platform_device *pdev;
 };
 
-/* Structure for a device driver */
-
-static struct platform_driver tsi_eth_driver = {
-       .probe = tsi108_init_one,
-       .remove = tsi108_ether_remove,
-       .driver = {
-               .name = "tsi-ethernet",
-       },
-};
-
 static void tsi108_timed_checker(struct timer_list *t);
 
 #ifdef DEBUG
@@ -1683,6 +1670,16 @@ static int tsi108_ether_remove(struct platform_device *pdev)
 
        return 0;
 }
+
+/* Structure for a device driver */
+
+static struct platform_driver tsi_eth_driver = {
+       .probe = tsi108_init_one,
+       .remove = tsi108_ether_remove,
+       .driver = {
+               .name = "tsi-ethernet",
+       },
+};
 module_platform_driver(tsi_eth_driver);
 
 MODULE_AUTHOR("Tundra Semiconductor Corporation");