From 99d23f60af17d0d6de656fba6a17578d52fadc4a Mon Sep 17 00:00:00 2001 From: Sameer Pujar Date: Fri, 19 Mar 2021 20:45:44 +0530 Subject: [PATCH] ASoC: tegra: Use runtime PM get/put() on SFC device SFC driver performs runtime PM operations on the parent device of SFC. This is not necessary as runtime get/put() calls on the SFC device itself would be sufficient. Because SFC device is child of AHUB device and resuming child would resume parent device as well. This does not fix any functional issue, but just makes code more easy to read. Bug 200698314 Change-Id: Ia4ce4b45b8681d2db15af96f61433643dd4f3916 Signed-off-by: Sameer Pujar --- sound/soc/tegra/tegra210_sfc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/sound/soc/tegra/tegra210_sfc.c b/sound/soc/tegra/tegra210_sfc.c index bc0c01d4..8abad757 100644 --- a/sound/soc/tegra/tegra210_sfc.c +++ b/sound/soc/tegra/tegra210_sfc.c @@ -3094,9 +3094,9 @@ static int tegra210_sfc_init(struct snd_soc_component *cmpnt, int init) dev_dbg(cmpnt->dev, "%s: inrate %d outrate %d\n", __func__, sfc->srate_in, sfc->srate_out); - err = pm_runtime_get_sync(cmpnt->dev->parent); + err = pm_runtime_get_sync(cmpnt->dev); if (err < 0) { - dev_err(cmpnt->dev, "parent get_sync failed: %d\n", err); + dev_err(cmpnt->dev, "RPM get_sync() failed: %d\n", err); return err; } @@ -3158,7 +3158,7 @@ static int tegra210_sfc_init(struct snd_soc_component *cmpnt, int init) regmap_write(sfc->regmap, TEGRA210_SFC_ENABLE, 1); } exit: - pm_runtime_put(cmpnt->dev->parent); + pm_runtime_put(cmpnt->dev); return err; }