Unverified Commit b3598fe6 authored by Minghao Chi's avatar Minghao Chi Committed by Mark Brown
Browse files

ASoC: SOF: using pm_runtime_resume_and_get to simplify the code 



Using pm_runtime_resume_and_get() to replace pm_runtime_get_sync and
pm_runtime_put_noidle. This change is just to simplify the code, no
actual functional changes.

Reported-by: default avatarZeal Robot <zealci@zte.com.cn>
Signed-off-by: default avatarMinghao Chi <chi.minghao@zte.com.cn>
Acked-by: default avatarPierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Link: https://lore.kernel.org/r/20220420030315.2575691-1-chi.minghao@zte.com.cn


Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent fbae863d

sound/soc/sof/sof-client-probes.c

+3 −6
Original line number Diff line number Diff line
@@ -503,10 +503,9 @@ static ssize_t sof_probes_dfs_points_read(struct file *file, char __user *to, 
	if (!buf)

		return -ENOMEM;



	ret = pm_runtime_get_sync(dev);

	ret = pm_runtime_resume_and_get(dev);

	if (ret < 0 && ret != -EACCES) {

		dev_err_ratelimited(dev, "debugfs read failed to resume %d\n", ret);

		pm_runtime_put_noidle(dev);

		goto exit;

	}
@@ -568,10 +567,9 @@ sof_probes_dfs_points_write(struct file *file, const char __user *from, 


	desc = (struct sof_probe_point_desc *)tkns;



	ret = pm_runtime_get_sync(dev);

	ret = pm_runtime_resume_and_get(dev);

	if (ret < 0 && ret != -EACCES) {

		dev_err_ratelimited(dev, "debugfs write failed to resume %d\n", ret);

		pm_runtime_put_noidle(dev);

		goto exit;

	}
@@ -621,10 +619,9 @@ sof_probes_dfs_points_remove_write(struct file *file, const char __user *from, 
		goto exit;

	}



	ret = pm_runtime_get_sync(dev);

	ret = pm_runtime_resume_and_get(dev);

	if (ret < 0) {

		dev_err_ratelimited(dev, "debugfs write failed to resume %d\n", ret);

		pm_runtime_put_noidle(dev);

		goto exit;

	}