[3.13.y.z extended stable] Patch "clk: Fix double free due to devm_clk_register()" has been added to staging queue

[Kamal Mostafa]

On Mon, 2014-06-23 at 14:25 -0700, Stephen Boyd wrote:
> On 06/17/14 14:42, Kamal Mostafa wrote:
> > From c7d31257ab0906113e8f3e2bed11bc9b383b14c9 Mon Sep 17 00:00:00 2001
> > From: Stephen Boyd <sboyd at codeaurora.org>
> > Date: Fri, 18 Apr 2014 16:29:42 -0700
> > Subject: clk: Fix double free due to devm_clk_register()
> >
> > commit 293ba3b4a4fd54891b900f2911d1a57e1ed4a843 upstream.
> >
> > Now that clk_unregister() frees the struct clk we're
> > unregistering we'll free memory twice: first we'll call kfree()
> > [...]
> > Fixes: fcb0ee6a3d33 (clk: Implement clk_unregister)

> Is this patch present in the [3.8.y] tree? I can't seem to find it so
> I don't think this patch is necessary.

> Same for this one [3.13.y]. I don't see this bad commit in the stable tree.

Thanks very much for reviewing this, Stephen.  You're right:
fcb0ee6a3d33 (clk: Implement clk_unregister) is not present in either
3.8.y or 3.13.y.

So I'll drop this unnecessary patch (293ba3b4 clk: Fix double free ...)
from the 3.8.y queue.

For 3.13.y though, it has already been released in v3.13.11.4, so my
inclination is to leave it in (but I could easily be convinced
otherwise).  Is it correct that 293ba3b4 is at least harmless for
3.13.y?

 -Kamal