tree 854a1c79ce94a268127a5e0efbf6a5ba77d77b41
parent 85e16fdd2f25ef6b08b4275acd5f60fdccfe0973
author Jon Hunter <jonathanh@nvidia.com> 1534520143 +0100
committer Mark Brown <broonie@kernel.org> 1535984323 +0100
gpgsig -----BEGIN PGP SIGNATURE-----
 
 iQFHBAABCgAxFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAluNQsMTHGJyb29uaWVA
 a2VybmVsLm9yZwAKCRAk1otyXVSH0CpmB/9PCBgyhQBcDPPFcssxm0CV63pq3fNZ
 VavlVj2mg4KHHSp/ojQKXSoF3mzNuE//CekXT+5rCK76lDMX5He6VNzCWZz9kJMe
 1NSydYdT8x/xnzer2GLEP/+S9f8TfgNzFbXlltkQCogaHBPT6aj++qFFtRxBVTSY
 2O1iIxgfmXaGIS4xm82zQhFR4sBzvc9T2wqRhfen6aW08g/FseHBcZf510tt4Dax
 kQQnU3SSfEAdBsXDcQaMDrebXKnHtoXhhdRjqWzqM+mOc45NO/rZ6EPJQbwPxl4K
 Rk1gjE0jbYfrgBXvHnD/6bYJwHl9R3SzEtvIiiFW7ywoE5RQgoDZJAOX
 =h9HB
 -----END PGP SIGNATURE-----

ASoC: core: Don't schedule DAPM work if already in target state

When dapm_power_widgets() is called, the dapm_pre_sequence_async() and
dapm_post_sequence_async() functions are scheduled for all DAPM contexts
(apart from the card DAPM context) regardless of whether the DAPM
context is already in the desired state. The overhead of this is not
insignificant and the more DAPM contexts there are the more overhead
there is.

For example, on the Tegra124 Jetson TK1, when profiling the time taken
to execute the dapm_power_widgets() the following times were observed.

  Times for function dapm_power_widgets() are (us):
     Min 23, Ave 190, Max 434, Count 39

Here 'Count' is the number of times that dapm_power_widgets() has been
called. Please note that the above time were measured using ktime_get()
to log the time on entry and exit from dapm_power_widgets(). So it
should be noted that these times may not be purely the time take to
execute this function if it is preempted. However, after applying this
patch and measuring the time taken to execute dapm_power_widgets() again
a significant improvement is seen as shown below.

  Times for function dapm_power_widgets() are (us):
     Min 4, Ave 16, Max 82, Count 39

Therefore, optimise the dapm_power_widgets() function by only scheduling
the dapm_pre/post_sequence_async() work if the DAPM context is not in
the desired state.

Signed-off-by: Jon Hunter <jonathanh@nvidia.com>
Reviewed-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Signed-off-by: Mark Brown <broonie@kernel.org>