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

[Stephen Boyd <sboyd@codeaurora.org>]

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