Commit 40e31179 authored by Federico Vaga's avatar Federico Vaga

Revert "kernel: fix memory leakage"

This reverts commit 0598de40.

This was not a memory leakage because the fmc-bus release that
pointer for us. Is it good?

We allocate and it is someone else us free, it lacks of simmetry.
Probably the register function should be called devm_ like all
the others in the kernel who relase for your the memory. In this
way it is clear the behaviour.
parent d56cd47c
......@@ -598,7 +598,6 @@ void spec_fmc_destroy(struct spec_dev *spec)
spec_irq_exit(spec->fmc);
spec_i2c_exit(spec->fmc);
put_device(&spec->fmc->dev);
kfree(spec->fmc);
spec->fmc = NULL;
spec->flags &= ~SPEC_FLAG_FMC_REGISTERED;
}
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment