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 -- 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
