Commit 7440ea8b authored by Stephen Rothwell's avatar Stephen Rothwell Committed by Michael Ellerman

drivers/macintosh/smu.c: Mark expected switch fall-through

Mark switch cases where we are expecting to fall through.

This patch fixes the following warning (Building: powerpc):

  drivers/macintosh/smu.c: In function 'smu_queue_i2c':
  drivers/macintosh/smu.c:854:21: warning: this statement may fall through [-Wimplicit-fallthrough=]
     cmd->info.devaddr &= 0xfe;
     ~~~~~~~~~~~~~~~~~~^~~~~~~
  drivers/macintosh/smu.c:855:2: note: here
    case SMU_I2C_TRANSFER_STDSUB:
    ^~~~

Fixes: 0365ba7f ("[PATCH] ppc64: SMU driver update & i2c support")
Signed-off-by: default avatarStephen Rothwell <sfr@canb.auug.org.au>
Reviewed-by: default avatarKees Cook <keescook@chromium.org>
Signed-off-by: default avatarMichael Ellerman <mpe@ellerman.id.au>
Link: https://lore.kernel.org/r/20190730143704.060a2606@canb.auug.org.au
parent 7db57e77
...@@ -852,6 +852,7 @@ int smu_queue_i2c(struct smu_i2c_cmd *cmd) ...@@ -852,6 +852,7 @@ int smu_queue_i2c(struct smu_i2c_cmd *cmd)
break; break;
case SMU_I2C_TRANSFER_COMBINED: case SMU_I2C_TRANSFER_COMBINED:
cmd->info.devaddr &= 0xfe; cmd->info.devaddr &= 0xfe;
/* fall through */
case SMU_I2C_TRANSFER_STDSUB: case SMU_I2C_TRANSFER_STDSUB:
if (cmd->info.sublen > 3) if (cmd->info.sublen > 3)
return -EINVAL; return -EINVAL;
......
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