drivers: net: smc911x: Fix passing wrong number of parameters to DBG() macro
authorAndrew Lunn <andrew@lunn.ch>
Tue, 10 Nov 2020 03:02:46 +0000 (04:02 +0100)
committerJakub Kicinski <kuba@kernel.org>
Thu, 12 Nov 2020 22:49:40 +0000 (14:49 -0800)
Now that the compiler always sees the parameters passed to the DBG()
macro, it gives an error message about wrong parameters. The comment
says it all:

/* ndev is not valid yet, so avoid passing it in. */
DBG(SMC_DEBUG_FUNC, "--> %s\n",  __func__);

You cannot not just pass a parameter!

The DBG does not seem to have any real value, to just remove it.

Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/smsc/smc911x.c

index 6978050..ac1a764 100644 (file)
@@ -2047,8 +2047,6 @@ static int smc911x_drv_probe(struct platform_device *pdev)
        void __iomem *addr;
        int ret;
 
-       /* ndev is not valid yet, so avoid passing it in. */
-       DBG(SMC_DEBUG_FUNC, "--> %s\n",  __func__);
        res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
        if (!res) {
                ret = -ENODEV;