can: mcp251xfd: remove unneeded break
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>
This commit is contained in:
parent
da623840d8
commit
78db1aa8b5
@ -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>";
|
||||
@ -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>";
|
||||
|
Loading…
x
Reference in New Issue
Block a user