On Tue, Oct 31, 2023 at 07:29:34PM +0530, Syed Saba Kareem wrote:
I2S mode register field will be set to 1 when tdm mode is enabled. Update the I2S mode field based on tdm_mode flag check.
This will fix below smatch checker warning.
sound/soc/amd/acp/acp-i2s.c:59 acp_set_i2s_clk() warn: odd binop '0x0 & 0x2'
Fixes: 40f74d5f09d7 ("ASoC: amd: acp: refactor acp i2s clock generation code")
I saw this yesterday but didn't comment because I didn't want you to have to redo the patch. Mark, of course, also saw it and didn't ask you to redo it. And now it's applied.
But for the future, these Fixes tags should be on one line and in the tags section of the commit message. (It's in chronological order).
Fixes: 40f74d5f09d7 ("ASoC: amd: acp: refactor acp i2s clock generation code") Reported-By: Dan Carpenter dan.carpenter@linaro.org Signed-off-by: Syed Saba Kareem Syed.SabaKareem@amd.com Signed-off-by: Mark Brown
regards, dan carpenter