Re: [GIT PULL] clk: ti: clkctrl support for 4.13 merge window

[Stephen Boyd <sboyd@codeaurora.org>]

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