diff mbox series

[2/3] can: rcar_canfd: Improve printing of global operational state

Message ID 14c8c5ce026e9fec128404706d1c73c8ffa11ced.1716973640.git.geert+renesas@glider.be (mailing list archive)
State Awaiting Upstream
Delegated to: Netdev Maintainers
Headers show
Series can: rcar_canfd: Small improvements and cleanups | expand

Checks

Context Check Description
netdev/tree_selection success Series ignored based on subject

Commit Message

Geert Uytterhoeven May 29, 2024, 9:12 a.m. UTC
Replace the printing of internal numerical values by the printing of
strings reflecting their meaning, to make the message self-explanatory.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 drivers/net/can/rcar/rcar_canfd.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

Comments

Wolfram Sang June 3, 2024, 10:32 a.m. UTC | #1
On Wed, May 29, 2024 at 11:12:14AM +0200, Geert Uytterhoeven wrote:
> Replace the printing of internal numerical values by the printing of
> strings reflecting their meaning, to make the message self-explanatory.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

Yes, useful!

Reviewed-by: Wolfram Sang <wsa@kernel.org>
diff mbox series

Patch

diff --git a/drivers/net/can/rcar/rcar_canfd.c b/drivers/net/can/rcar/rcar_canfd.c
index 474840b58e8f13f1..c2c1c47bcc7a166c 100644
--- a/drivers/net/can/rcar/rcar_canfd.c
+++ b/drivers/net/can/rcar/rcar_canfd.c
@@ -2049,8 +2049,9 @@  static int rcar_canfd_probe(struct platform_device *pdev)
 	}
 
 	platform_set_drvdata(pdev, gpriv);
-	dev_info(dev, "global operational state (clk %d, fdmode %d)\n",
-		 gpriv->extclk, gpriv->fdmode);
+	dev_info(dev, "global operational state (%s clk, %s mode)\n",
+		 gpriv->extclk ? "ext" : "canfd",
+		 gpriv->fdmode ? "fd" : "classical");
 	return 0;
 
 fail_channel: