* [PATCH] can: mcp251x: add some messages to mcp251x_can_probe()
@ 2016-05-29 1:59 Ed Spiridonov
2016-06-17 9:22 ` Marc Kleine-Budde
0 siblings, 1 reply; 6+ messages in thread
From: Ed Spiridonov @ 2016-05-29 1:59 UTC (permalink / raw)
To: mkl; +Cc: linux-can, Ed Spiridonov
Print error message if mcp251x_hw_probe() fails (e.g. MCP2515 isn't connected or other wiring issue).
Print information message if mcp251x_can_probe() is successful.
---
drivers/net/can/spi/mcp251x.c | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/drivers/net/can/spi/mcp251x.c b/drivers/net/can/spi/mcp251x.c
index cf36d26..49453f7 100644
--- a/drivers/net/can/spi/mcp251x.c
+++ b/drivers/net/can/spi/mcp251x.c
@@ -1145,8 +1145,10 @@ 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) {
+ dev_err(&spi->dev, "hw_probe failed, err=%d (wrong wiring?)\n", -ret);
goto error_probe;
+ }
mcp251x_hw_sleep(spi);
@@ -1156,6 +1158,8 @@ static int mcp251x_can_probe(struct spi_device *spi)
devm_can_led_init(net);
+ netdev_info(net, "successfully initialized.\n");
+
return 0;
error_probe:
--
2.7.0
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH] can: mcp251x: add some messages to mcp251x_can_probe()
2016-05-29 1:59 [PATCH] can: mcp251x: add some messages to mcp251x_can_probe() Ed Spiridonov
@ 2016-06-17 9:22 ` Marc Kleine-Budde
2016-06-17 20:41 ` Ed Spiridonov
[not found] ` <CACm0Nn320ry07YOuorrMO=u8Q4+JsOd9i3s7eFAwQ-L8AWGm=Q@mail.gmail.com>
0 siblings, 2 replies; 6+ messages in thread
From: Marc Kleine-Budde @ 2016-06-17 9:22 UTC (permalink / raw)
To: Ed Spiridonov; +Cc: linux-can
[-- Attachment #1.1: Type: text/plain, Size: 1536 bytes --]
On 05/29/2016 03:59 AM, Ed Spiridonov wrote:
> Print error message if mcp251x_hw_probe() fails (e.g. MCP2515 isn't connected or other wiring issue).
> Print information message if mcp251x_can_probe() is successful.
Can I add your S-o-b to the patch?
> ---
> drivers/net/can/spi/mcp251x.c | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/net/can/spi/mcp251x.c b/drivers/net/can/spi/mcp251x.c
> index cf36d26..49453f7 100644
> --- a/drivers/net/can/spi/mcp251x.c
> +++ b/drivers/net/can/spi/mcp251x.c
> @@ -1145,8 +1145,10 @@ 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) {
> + dev_err(&spi->dev, "hw_probe failed, err=%d (wrong wiring?)\n", -ret);
> goto error_probe;
> + }
>
> mcp251x_hw_sleep(spi);
>
> @@ -1156,6 +1158,8 @@ static int mcp251x_can_probe(struct spi_device *spi)
>
> devm_can_led_init(net);
>
> + netdev_info(net, "successfully initialized.\n");
I'll remove this hunk, as we try to make the CAN devices as silent as
possible in case of no errors.
> +
> return 0;
>
> error_probe:
>
regards,
Marc
--
Pengutronix e.K. | Marc Kleine-Budde |
Industrial Linux Solutions | Phone: +49-231-2826-924 |
Vertretung West/Dortmund | Fax: +49-5121-206917-5555 |
Amtsgericht Hildesheim, HRA 2686 | http://www.pengutronix.de |
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 455 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] can: mcp251x: add some messages to mcp251x_can_probe()
2016-06-17 9:22 ` Marc Kleine-Budde
@ 2016-06-17 20:41 ` Ed Spiridonov
2016-06-18 8:41 ` Oliver Hartkopp
[not found] ` <CACm0Nn320ry07YOuorrMO=u8Q4+JsOd9i3s7eFAwQ-L8AWGm=Q@mail.gmail.com>
1 sibling, 1 reply; 6+ messages in thread
From: Ed Spiridonov @ 2016-06-17 20:41 UTC (permalink / raw)
Cc: linux-can
17.06.2016 12:22 пользователь "Marc Kleine-Budde" <mkl@pengutronix.de> написал:
>
> > @@ -1156,6 +1158,8 @@ static int mcp251x_can_probe(struct spi_device *spi)
> >
> > devm_can_led_init(net);
> >
> > + netdev_info(net, "successfully initialized.\n");
>
> I'll remove this hunk, as we try to make the CAN devices as silent as
> possible in case of no errors.
Why it is so important to be totally silent? IMHO one line per CAN bus
isn't too much.
But it could help to debug new configuration, e.g. to find wrong
device tree configuration, especially with several CAN buses.
I made this patch because a lot of complains on Raspberry PI forum.
Current module does not produce any message if module initializes
MCP2510/2515 chip successfully or not. It could be confusing.
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] can: mcp251x: add some messages to mcp251x_can_probe()
2016-06-17 20:41 ` Ed Spiridonov
@ 2016-06-18 8:41 ` Oliver Hartkopp
0 siblings, 0 replies; 6+ messages in thread
From: Oliver Hartkopp @ 2016-06-18 8:41 UTC (permalink / raw)
To: Ed Spiridonov, Marc Kleine-Budde; +Cc: linux-can
Hello Marc,
On 06/17/2016 10:41 PM, Ed Spiridonov wrote:
> 17.06.2016 12:22 пользователь "Marc Kleine-Budde" <mkl@pengutronix.de> написал:
>>
>>> @@ -1156,6 +1158,8 @@ static int mcp251x_can_probe(struct spi_device *spi)
>>>
>>> devm_can_led_init(net);
>>>
>>> + netdev_info(net, "successfully initialized.\n");
>>
>> I'll remove this hunk, as we try to make the CAN devices as silent as
>> possible in case of no errors.
I agree with Ed in this special case.
In opposite to CAN interfaces that reside on local busses or inside of
SoCs the mcp251x is attached indirectly via SPI. This adds additional
options to fail at initialization time.
As USB interfaces also tell about the USB plug-in and about their
successful netdev initialization - the SPI based interfaces should be
able to do it similarly.
Even the mcp251x stuff is not a very high performance CAN interface
implementation many RasPi users do their first steps with SocketCAN
using this tricky hardware setup.
That's why I vote for more information at initialization time too.
Regards,
Oliver
>
>
> Why it is so important to be totally silent? IMHO one line per CAN bus
> isn't too much.
> But it could help to debug new configuration, e.g. to find wrong
> device tree configuration, especially with several CAN buses.
>
> I made this patch because a lot of complains on Raspberry PI forum.
> Current module does not produce any message if module initializes
> MCP2510/2515 chip successfully or not. It could be confusing.
> --
> To unsubscribe from this list: send the line "unsubscribe linux-can" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 6+ messages in thread
[parent not found: <CACm0Nn320ry07YOuorrMO=u8Q4+JsOd9i3s7eFAwQ-L8AWGm=Q@mail.gmail.com>]
* Re: [PATCH] can: mcp251x: add some messages to mcp251x_can_probe()
[not found] ` <CACm0Nn320ry07YOuorrMO=u8Q4+JsOd9i3s7eFAwQ-L8AWGm=Q@mail.gmail.com>
@ 2016-06-20 8:47 ` Marc Kleine-Budde
2016-06-20 18:49 ` Ed Spiridonov
0 siblings, 1 reply; 6+ messages in thread
From: Marc Kleine-Budde @ 2016-06-20 8:47 UTC (permalink / raw)
To: Ed Spiridonov; +Cc: linux-can
[-- Attachment #1.1: Type: text/plain, Size: 1358 bytes --]
On 06/17/2016 10:35 PM, Ed Spiridonov wrote:
> 17.06.2016 12:22 пользователь "Marc Kleine-Budde" <mkl@pengutronix.de
> <mailto:mkl@pengutronix.de>> написал:
>
> > @@ -1156,6 +1158,8 @@ static int mcp251x_can_probe(struct
> spi_device *spi)
> >
> > devm_can_led_init(net);
> >
> > + netdev_info(net, "successfully initialized.\n");
>
> I'll remove this hunk, as we try to make the CAN devices as silent as
> possible in case of no errors.
>
>
> Why it is so important to be totally silent? IMHO one line per CAN bus
> isn't too much.
> But it could help to debug new configuration, e.g. to find wrong device
> tree configuration, especially with several CAN buses.
>
> I made this patch because a lot of complains on Raspberry PI forum.
> Current module does not produce any message if module initializes
> MCP2510/2515 chip successfully or not. It could be confusing.
ok, can I have your S-o-b [1]?
Marc
[1]
http://lxr.free-electrons.com/source/Documentation/SubmittingPatches#L409
--
Pengutronix e.K. | Marc Kleine-Budde |
Industrial Linux Solutions | Phone: +49-231-2826-924 |
Vertretung West/Dortmund | Fax: +49-5121-206917-5555 |
Amtsgericht Hildesheim, HRA 2686 | http://www.pengutronix.de |
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 455 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2016-06-20 18:50 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-05-29 1:59 [PATCH] can: mcp251x: add some messages to mcp251x_can_probe() Ed Spiridonov
2016-06-17 9:22 ` Marc Kleine-Budde
2016-06-17 20:41 ` Ed Spiridonov
2016-06-18 8:41 ` Oliver Hartkopp
[not found] ` <CACm0Nn320ry07YOuorrMO=u8Q4+JsOd9i3s7eFAwQ-L8AWGm=Q@mail.gmail.com>
2016-06-20 8:47 ` Marc Kleine-Budde
2016-06-20 18:49 ` Ed Spiridonov
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).