Thread (1 message) 1 message, 1 author, 2021-09-13

Re: [PATCH 06/13] ASoC: tegra: Add Tegra210 based MVC driver

From: Mark Brown <broonie@kernel.org>
Date: 2021-09-13 14:52:46
Also in: alsa-devel, linux-arm-kernel, linux-tegra, lkml

Possibly related (same subject, not in this thread)

On Mon, Sep 13, 2021 at 10:32:32AM +0530, Sameer Pujar wrote:
On 9/3/2021 11:43 PM, Mark Brown wrote:
quoted
quoted
+	err |= regmap_update_bits(mvc->regmap, TEGRA210_MVC_SWITCH,
+			TEGRA210_MVC_VOLUME_SWITCH_MASK,
+			TEGRA210_MVC_VOLUME_SWITCH_TRIGGER);
+
+end:
+	pm_runtime_put(cmpnt->dev);
+	return err;
+}
quoted
_put() should return 0 if there's no change or 1 for a change.
So this means, no need to return any error value and just return 0 in such
cases?
If there's an error you should return that instead.

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help