On 06/15, Tero Kristo wrote: > On 14/06/17 20:49, Stephen Boyd wrote: > >On 06/14, Tero Kristo wrote: > >>On 14/06/17 20:32, Stephen Boyd wrote: > >>>On 06/08, Tero Kristo wrote: > >>>>Hi Stephen, Mike, > >>>> > >>>>This pull contains the v3 of the clkctrl base driver support posted > >>>>couple of weeks back, with Tony's acks added to patch #2 and #4. > >>>> > >>>>-Tero > >>>> > >>> > >>>drivers/clk/ti/clkctrl.c:252 > >>>_ti_clkctrl_clk_register() warn: possible memory leak of 'clkctrl_clk' > >>>drivers/clk/ti/clkctrl.c:264 > >>>_ti_clkctrl_clk_register() warn: possible memory leak of 'clkctrl_clk' > >>>drivers/clk/ti/clkctrl.c:466 > >>>_ti_omap4_clkctrl_setup() warn: possible memory leak of 'clkctrl_clk' > >>>drivers/clk/ti/clkctrl.c:473 > >>>_ti_omap4_clkctrl_setup() warn: possible memory leak of 'clkctrl_clk' > >>> > >>>Can these be silenced with some follow up patch? I'll hold off on > >>>the pull until you can take a look. > >>> > >> > >>What is the tool you use to get these? > >> > >>They seem like valid warnings though, some cleanup could be used for > >>the dynamic memory handling. I'll fix these tomorrow. > >> > > > >I use smatch[1] although I don't always have the most up to date > >version. > > > >[1] git://repo.or.cz/smatch.git > > > > Ok thanks, fixed and posted a new version of patch #2 of the series. > Also force pushed this branch with the updated version in case you > want to use this. > Thanks. Pulled into clk-next now. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
