From: Cezary Rojewski <cezary.rojewski@intel.com>
Date: Mon, 3 Feb 2025 14:10:43 +0000 (+0100)
Subject: ASoC: codecs: pcm3168a: Allow for 24-bit in provider mode
X-Git-Tag: v6.15-rc1~173^2~4^2~135^2~8
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=7d92a38d67e5d937b64b20aa4fd14451ee1772f3;p=thirdparty%2Fkernel%2Flinux.git

ASoC: codecs: pcm3168a: Allow for 24-bit in provider mode

As per codec device specification, 24-bit is allowed in provider mode.
Update the code to reflect that.

Signed-off-by: Cezary Rojewski <cezary.rojewski@intel.com>
Link: https://patch.msgid.link/20250203141051.2361323-4-cezary.rojewski@intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
---

diff --git a/sound/soc/codecs/pcm3168a.c b/sound/soc/codecs/pcm3168a.c
index 43c0cb5e538e8..df6836a652ef2 100644
--- a/sound/soc/codecs/pcm3168a.c
+++ b/sound/soc/codecs/pcm3168a.c
@@ -493,9 +493,9 @@ static int pcm3168a_hw_params(struct snd_pcm_substream *substream,
 		}
 		break;
 	case 24:
-		if (provider_mode || (format == SND_SOC_DAIFMT_DSP_A) ||
-		    		     (format == SND_SOC_DAIFMT_DSP_B)) {
-			dev_err(component->dev, "24-bit slots not supported in provider mode, or consumer mode using DSP\n");
+		if (!provider_mode && ((format == SND_SOC_DAIFMT_DSP_A) ||
+				       (format == SND_SOC_DAIFMT_DSP_B))) {
+			dev_err(component->dev, "24-bit slots not supported in consumer mode using DSP\n");
 			return -EINVAL;
 		}
 		break;