can: mcp251x: add message about sucessful/unsuccessful probe
authorEd Spiridonov <edo.rus@gmail.com>
Mon, 20 Jun 2016 18:40:15 +0000 (21:40 +0300)
committerMarc Kleine-Budde <mkl@pengutronix.de>
Thu, 23 Jun 2016 09:23:49 +0000 (11:23 +0200)
Silent ignorance of errors during probe procedure is a bad thing, this
patch fixes it. Extra message added for hardware initialization
failure. Such common issues are mostly caused by wrong wiring.  Message
about success added as well, it should be useful to debug new hardware
configuration, especially in case of several CAN buses.

Signed-off-by: Ed Spiridonov <edo.rus@gmail.com>
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
drivers/net/can/spi/mcp251x.c

index cf36d26ef002262aa43c37439d17d57069ed8780..f3f05fea8e1f8e0ad3a1ebd7461d2019835e1d47 100644 (file)
@@ -1145,8 +1145,11 @@ static int mcp251x_can_probe(struct spi_device *spi)
 
        /* Here is OK to not lock the MCP, no one knows about it yet */
        ret = mcp251x_hw_probe(spi);
-       if (ret)
+       if (ret) {
+               if (ret == -ENODEV)
+                       dev_err(&spi->dev, "Cannot initialize MCP%x. Wrong wiring?\n", priv->model);
                goto error_probe;
+       }
 
        mcp251x_hw_sleep(spi);
 
@@ -1156,6 +1159,7 @@ static int mcp251x_can_probe(struct spi_device *spi)
 
        devm_can_led_init(net);
 
+       netdev_info(net, "MCP%x successfully initialized.\n", priv->model);
        return 0;
 
 error_probe:
@@ -1168,6 +1172,7 @@ out_clk:
 out_free:
        free_candev(net);
 
+       dev_err(&spi->dev, "Probe failed, err=%d\n", -ret);
        return ret;
 }