On 24/02/2022 15:29, Pierre-Louis Bossart wrote:
On 2/24/22 05:17, Srinivas Kandagatla wrote:
WSA881x codecs can not cope up with clk stop and requires a full reset after suspend. WSA SoundWire Controller connected to this instances do a full soft reset on suspend.
is it the manager or peripheral that cannot cope with clock stop?
@@ -1124,6 +1136,7 @@ static int wsa881x_probe(struct sdw_slave *pdev, pdev->prop.sink_ports = GENMASK(WSA881X_MAX_SWR_PORTS, 0); pdev->prop.sink_dpn_prop = wsa_sink_dpn_prop; pdev->prop.scp_int1_mask = SDW_SCP_INT1_BUS_CLASH | SDW_SCP_INT1_PARITY;
- pdev->prop.simple_clk_stop_capable = true;
here it's pretty clear the peripheral is capable of supporting clock stop, no?
Something's not clear between the commit message and the code.
True, This should be removed along with attempting to call clock stop functions from WSA SoundWire controller suspend that does not support clock stop.
--srini