Re: [PATCH 03/17] ASoC: Intel: Introduce AVS driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2022-02-25 9:23 PM, Pierre-Louis Bossart wrote:

+int avs_dsp_core_disable(struct avs_dev *adev, u32 core_mask)
+{
+    /* Be permissive to allow for full DSP shutdown in disable path. */

that comment isn't very clear, what is permissive here?


There is no error checking below.

+    avs_dsp_op(adev, stall, core_mask, true);
+    avs_dsp_op(adev, reset, core_mask, true);
+
+    return avs_dsp_op(adev, power, core_mask, false);

consider adding a comment then, along the lines of 'we don't prevent
suspend or shutdown with error checks' or something. 'permissive' was
rather unclear to me.

Rewording in v2 as suggested.



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux