Commit c24bfa8f authored by Gustavo A. R. Silva's avatar Gustavo A. R. Silva Committed by Mark Brown

spi: slave: Fix missing break in switch

Apparently, this code does not actually fall through to the next case
because the machine restarts before it has a chance. However, for the
sake of maintenance and readability, we better add the missing break
statement.

Addresses-Coverity-ID: 1437892 ("Missing break in switch")
Signed-off-by: default avatarGustavo A. R. Silva <gustavo@embeddedor.com>
Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent aeb8a43f
...@@ -60,6 +60,7 @@ static void spi_slave_system_control_complete(void *arg) ...@@ -60,6 +60,7 @@ static void spi_slave_system_control_complete(void *arg)
case CMD_REBOOT: case CMD_REBOOT:
dev_info(&priv->spi->dev, "Rebooting system...\n"); dev_info(&priv->spi->dev, "Rebooting system...\n");
kernel_restart(NULL); kernel_restart(NULL);
break;
case CMD_POWEROFF: case CMD_POWEROFF:
dev_info(&priv->spi->dev, "Powering off system...\n"); dev_info(&priv->spi->dev, "Powering off system...\n");
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment