ASoC: Intel: avs: Do not treat unsupported IPCs as invalid

Utilize NOT_SUPPORTED status code to differentiate between unsupported
and invalid requests. Skip over error paths if it is the former that is
communicated by the base firmware.

Signed-off-by: Cezary Rojewski <cezary.rojewski@intel.com>
Link: https://lore.kernel.org/r/20221010121955.718168-12-cezary.rojewski@intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>

diff --git a/sound/soc/intel/avs/avs.h b/sound/soc/intel/avs/avs.h
index 92e37722d280b5820ff48b7206943c28f9eb068b..91f78eb11bc1c321f83865e5a4dfdafb6b102d0c 100644 (file)
--- a/sound/soc/intel/avs/avs.h
+++ b/sound/soc/intel/avs/avs.h
@@ -220,8 +220,10 @@ static inline void avs_ipc_err(struct avs_dev *adev, struct avs_ipc_msg *tx,
        /*
         * If IPC channel is blocked e.g.: due to ongoing recovery,
         * -EPERM error code is expected and thus it's not an actual error.
+        *
+        * Unsupported IPCs are of no harm either.
         */
-       if (error == -EPERM)
+       if (error == -EPERM || error == AVS_IPC_NOT_SUPPORTED)
                dev_dbg(adev->dev, "%s 0x%08x 0x%08x failed: %d\n", name,
                        tx->glb.primary, tx->glb.ext.val, error);
        else

diff --git a/sound/soc/intel/avs/messages.h b/sound/soc/intel/avs/messages.h
index c0f90dba9af89fde9e2846c2ce05d7ad354518f6..02b3b7a74783a0c7574bffc337228d76da016694 100644 (file)
--- a/sound/soc/intel/avs/messages.h
+++ b/sound/soc/intel/avs/messages.h
@@ -150,6 +150,8 @@ union avs_module_msg {
        };
 } __packed;
 
+#define AVS_IPC_NOT_SUPPORTED 15
+
 union avs_reply_msg {
        u64 val;
        struct {