On Fri, 2017-10-06 at 11:24 +0200, Greg Kroah-Hartman wrote: > 3.18-stable review patch. If anyone has any objections, please let me know.
Same deal, trivial cpu savings only, no place in stable unless another patch depends on this one. > ------------------ > > From: Markus Elfring <[email protected]> > > > [ Upstream commit 420dc61642920849d824a0de2aa853db59f5244f ] > > Strings which did not contain data format specifications should be put > into a sequence. Thus use the corresponding function "seq_puts". > > This issue was detected by using the Coccinelle software. > > Signed-off-by: Markus Elfring <[email protected]> > Signed-off-by: Linus Walleij <[email protected]> > Signed-off-by: Sasha Levin <[email protected]> > Signed-off-by: Greg Kroah-Hartman <[email protected]> > --- > drivers/pinctrl/mvebu/pinctrl-mvebu.c | 9 +++++---- > 1 file changed, 5 insertions(+), 4 deletions(-) > > --- a/drivers/pinctrl/mvebu/pinctrl-mvebu.c > +++ b/drivers/pinctrl/mvebu/pinctrl-mvebu.c > @@ -195,11 +195,12 @@ static void mvebu_pinconf_group_dbg_show > seq_printf(s, "o"); > seq_printf(s, ")"); > } > - } else > - seq_printf(s, "current: UNKNOWN"); > + } else { > + seq_puts(s, "current: UNKNOWN"); > + } > > if (grp->num_settings > 1) { > - seq_printf(s, ", available = ["); > + seq_puts(s, ", available = ["); > for (n = 0; n < grp->num_settings; n++) { > if (curr == &grp->settings[n]) > continue; > @@ -222,7 +223,7 @@ static void mvebu_pinconf_group_dbg_show > seq_printf(s, ")"); > } > } > - seq_printf(s, " ]"); > + seq_puts(s, " ]"); > } > return; > } > >
