Messages in this thread | | | Date | Thu, 15 May 2014 10:34:10 +0530 | Subject | Re: [PATCH] PM / OPP: discard duplicate OPP additions | From | Viresh Kumar <> |
| |
On 15 May 2014 10:22, Inderpal Singh <inderpal.s@samsung.com> wrote:
> What i am saying that "what if we are not going to re-use again ? "
That's what I said:
Yes, it will keep occupying some space but there is only one instance of that per 'cluster' and is very much affordable instead of building it again..
So, we may not need to free it.
> Also, I feel the driver who created the opp table at its registration > time should free it at its unregistration. Isn't it true in general?
Probably case to case basis. You must free resources for two reasons: - They are not lost, like memory leak .. - They can be used by others ..
And both these doesn't happen in this case. OPP tables can be used by any other framework and is more or less a core thing..
Now, with this discussion I have another idea here..
Why don't we build these tables automatically on boot from some core code, rather than asking drivers to do it ? That will get rid of duplication from all drivers and would also reduce confusion
@Rafael/Nishanth ??
If things look right, then I can write a patch for fixing it quickly...
| |