Skip to content

Commit 7299cc0

Browse files
tobluxbroonie
authored andcommitted
ASoC: SOF: Intel: Use str_yes_no() to improve bdw_dump()
Remove hard-coded strings by using the str_yes_no() helper function. Signed-off-by: Thorsten Blum <[email protected]> Link: https://patch.msgid.link/[email protected] Signed-off-by: Mark Brown <[email protected]>
1 parent abea0cb commit 7299cc0

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

sound/soc/sof/intel/bdw.c

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -266,20 +266,20 @@ static void bdw_dump(struct snd_sof_dev *sdev, u32 flags)
266266
imrd = snd_sof_dsp_read(sdev, BDW_DSP_BAR, SHIM_IMRD);
267267
dev_err(sdev->dev,
268268
"error: ipc host -> DSP: pending %s complete %s raw 0x%8.8x\n",
269-
(panic & SHIM_IPCX_BUSY) ? "yes" : "no",
270-
(panic & SHIM_IPCX_DONE) ? "yes" : "no", panic);
269+
str_yes_no(panic & SHIM_IPCX_BUSY),
270+
str_yes_no(panic & SHIM_IPCX_DONE), panic);
271271
dev_err(sdev->dev,
272272
"error: mask host: pending %s complete %s raw 0x%8.8x\n",
273-
(imrx & SHIM_IMRX_BUSY) ? "yes" : "no",
274-
(imrx & SHIM_IMRX_DONE) ? "yes" : "no", imrx);
273+
str_yes_no(imrx & SHIM_IMRX_BUSY),
274+
str_yes_no(imrx & SHIM_IMRX_DONE), imrx);
275275
dev_err(sdev->dev,
276276
"error: ipc DSP -> host: pending %s complete %s raw 0x%8.8x\n",
277-
(status & SHIM_IPCD_BUSY) ? "yes" : "no",
278-
(status & SHIM_IPCD_DONE) ? "yes" : "no", status);
277+
str_yes_no(status & SHIM_IPCD_BUSY),
278+
str_yes_no(status & SHIM_IPCD_DONE), status);
279279
dev_err(sdev->dev,
280280
"error: mask DSP: pending %s complete %s raw 0x%8.8x\n",
281-
(imrd & SHIM_IMRD_BUSY) ? "yes" : "no",
282-
(imrd & SHIM_IMRD_DONE) ? "yes" : "no", imrd);
281+
str_yes_no(imrd & SHIM_IMRD_BUSY),
282+
str_yes_no(imrd & SHIM_IMRD_DONE), imrd);
283283
}
284284

285285
/*

0 commit comments

Comments
 (0)