ASoC: cs35l56: Wait for control port ready during system-resume
authorRichard Fitzgerald <rf@opensource.cirrus.com>
Tue, 8 Aug 2023 16:47:00 +0000 (17:47 +0100)
committerMark Brown <broonie@kernel.org>
Tue, 8 Aug 2023 21:34:02 +0000 (22:34 +0100)
The CS35L56 could be hard-reset during a system suspend-resume cycle,
either by the codec driver, in cs35l56_system_resume_early(), or by ACPI.
After a hard reset the driver must wait for the control port to be ready
(datasheet tIRS time) before attempting to access the CS35L56.

Signed-off-by: Richard Fitzgerald <rf@opensource.cirrus.com>
Link: https://lore.kernel.org/r/20230808164702.21272-4-rf@opensource.cirrus.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/cs35l56.c

index 9560059..094bcbd 100644 (file)
@@ -952,6 +952,12 @@ int cs35l56_system_resume(struct device *dev)
 
        dev_dbg(dev, "system_resume\n");
 
+       /*
+        * We might have done a hard reset or the CS35L56 was power-cycled
+        * so wait for control port to be ready.
+        */
+       cs35l56_wait_control_port_ready();
+
        /* Undo pm_runtime_force_suspend() before re-enabling the irq */
        ret = pm_runtime_force_resume(dev);
        if (cs35l56->base.irq)