Skip to content

Commit

Permalink
can: mcp251xfd: remove unneeded break
Browse files Browse the repository at this point in the history
A break is not needed if it is preceded by a return.

Signed-off-by: Tom Rix <trix@redhat.com>
Link: https://lore.kernel.org/r/20201019172412.31143-1-trix@redhat.com
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
  • Loading branch information
Tom Rix authored and Marc Kleine-Budde committed Nov 3, 2020
1 parent da62384 commit 78db1aa
Showing 1 changed file with 11 additions and 11 deletions.
22 changes: 11 additions & 11 deletions drivers/net/can/spi/mcp251xfd/mcp251xfd-core.c
Original file line number Diff line number Diff line change
Expand Up @@ -75,11 +75,11 @@ static const char *__mcp251xfd_get_model_str(enum mcp251xfd_model model)
{
switch (model) {
case MCP251XFD_MODEL_MCP2517FD:
return "MCP2517FD"; break;
return "MCP2517FD";
case MCP251XFD_MODEL_MCP2518FD:
return "MCP2518FD"; break;
return "MCP2518FD";
case MCP251XFD_MODEL_MCP251XFD:
return "MCP251xFD"; break;
return "MCP251xFD";
}

return "<unknown>";
Expand All @@ -95,21 +95,21 @@ static const char *mcp251xfd_get_mode_str(const u8 mode)
{
switch (mode) {
case MCP251XFD_REG_CON_MODE_MIXED:
return "Mixed (CAN FD/CAN 2.0)"; break;
return "Mixed (CAN FD/CAN 2.0)";
case MCP251XFD_REG_CON_MODE_SLEEP:
return "Sleep"; break;
return "Sleep";
case MCP251XFD_REG_CON_MODE_INT_LOOPBACK:
return "Internal Loopback"; break;
return "Internal Loopback";
case MCP251XFD_REG_CON_MODE_LISTENONLY:
return "Listen Only"; break;
return "Listen Only";
case MCP251XFD_REG_CON_MODE_CONFIG:
return "Configuration"; break;
return "Configuration";
case MCP251XFD_REG_CON_MODE_EXT_LOOPBACK:
return "External Loopback"; break;
return "External Loopback";
case MCP251XFD_REG_CON_MODE_CAN2_0:
return "CAN 2.0"; break;
return "CAN 2.0";
case MCP251XFD_REG_CON_MODE_RESTRICTED:
return "Restricted Operation"; break;
return "Restricted Operation";
}

return "<unknown>";
Expand Down

0 comments on commit 78db1aa

Please sign in to comment.