On Thu, Mar 12, 2015 at 2:00 AM, Laurent Pinchart <[email protected]> wrote: >> @@ -322,13 +319,15 @@ int sh_pfc_config_mux(struct sh_pfc *pfc, unsigned >> mark, int pinmux_type) return -EINVAL; >> } >> >> - pos = 0; >> - enum_id = 0; >> - field = 0; >> - value = 0; >> - >> /* Iterate over all the configuration fields we need to update. */ >> while (1) { >> + const struct pinmux_cfg_reg *cr = NULL; >> + unsigned int field = 0; >> + u16 enum_id = 0; >> + u32 value = 0; > > There's no need to initialize field, enum_id and value to 0 here.
And cr. > With that fixed, Will update and resend. > Acked-by: Laurent Pinchart <[email protected]> Thanks! Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected] In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
