[PATCH] opp: convert dev_warn() to dev_dbg() for duplicate OPPs
stefan.wahren at i2se.com
Mon Nov 17 11:32:54 PST 2014
> Viresh Kumar <viresh.kumar at linaro.org> hat am 17. November 2014 um 09:08
> Now the question is: Should we destroy these OPPs?
> Logically kernel drivers *must* free resources they acquired. But in this
> particular case, the OPPs are created using a static list present in device
> tree. Destroying and then allocating them again isn't of much benefit. The
> benefit of removing OPPs is to save some space if the driver isn't loaded
> This has its own complications. OPPs can be created either from DT (static) or
> platform code (dynamic). Driver should only remove static OPPs and not the
> dynamic ones as they are controlled from platform code. But there is no field
> 'struct dev_pm_opp' which has this information to distinguish between
> kind of OPPs.
> Because of all this, I wasn't sure if drivers should remove static OPPs during
> their removal. And so just fixing the reported issue by issuing a dev_dbg()
> instead of dev_warn().
how about adding some kind of FIXME?
Tested-by: Stefan Wahren <stefan.wahren at i2se.com>
More information about the linux-arm-kernel