Messages in this thread | | | From | Linus Walleij <> | Date | Wed, 2 May 2018 13:47:54 +0200 | Subject | Re: [PATCH][next][V2] pinctrl: actions: fix missing break in PIN_CONFIG_DRIVE_STRENGTH case. |
| |
On Mon, Apr 30, 2018 at 3:22 PM, Colin King <colin.king@canonical.com> wrote:
> From: Colin Ian King <colin.king@canonical.com> > > There is a missing break in case PIN_CONFIG_DRIVE_STRENGTH leading to > a fall-through to the PIN_CONFIG_SLEW_RATE case that performs different > checks against *arg. This looks like an unintentional missing break so > add in the break. > > Detected by CoverityScan, CID#1468456, 1468459 ("Missing break in switch") > > Fixes: 513d7a2f7e0f ("pinctrl: actions: Add Actions S900 pinctrl driver") > Signed-off-by: Colin Ian King <colin.king@canonical.com>
This v2 applied with Manivannan's ACK.
Yours, Linus Walleij
| |