ISR_RXFO means "Status of Data Overrun interrupt for the RX channel" according to the datasheetSo, the comment should be RX, not TXThanksMyunguk------------Original Message------------Subject : Re: [PATCH] ASoC: dwc: fix typo in commentDate : 2023-09-11 21:46:05From : Bagas Sanjaya <bagasdotme@gmailcom>To : Myunguk Kim <mwkim@gaonchipscom>, alsa-devel@alsa-projectorgCc : Liam Girdwood <lgirdwood@gmailcom>, Mark Brown <broonie@kernelorg>, Jaroslav Kysela <perex@perexcz>, Takashi Iwai <tiwai@susecom>, Maxim Kochetkov <fido_max@inboxru>, Kuninori Morimoto <kuninorimorimotogx@renesascom>, Xingyu Wu <xingyuwu@starfivetechcom>, Uwe Kleine-König <ukleine-koenig@pengutronixde>, linux-kernel@vgerkernelorgOn Mon, Sep 11, 2023 at 03:46:46PM +0900, Myunguk Kim wrote:> This fixes typo in commentOn what? Can you describe the typofix in more detail?>> Signed-off-by: Myunguk Kim <mwkim@gaonchipscom>> ---> sound/soc/dwc/dwc-i2sc | 2 +-> 1 file changed, 1 insertion(+), 1 deletion(-)>> diff --git a/sound/soc/dwc/dwc-i2sc b/sound/soc/dwc/dwc-i2sc> index 71ff894035a445e55e550924 100644> --- a/sound/soc/dwc/dwc-i2sc> +++ b/sound/soc/dwc/dwc-i2sc> @@ -138,7 +138,7 @@ static irqreturn_t i2s_irq_handler(int irq, void *dev_id)> irq_valid = true;> }>> - /* Error Handling: TX */> + /* Error Handling: RX */> if (isr[i] & ISR_RXFO)> { dev_err_ratelimited(dev->dev, "RX overrun (ch_id=%d)\n", i);> irq_valid = true;Oh, you mean the code following the comment checks for RXFO overrunConfused--An old man doll just what I always wanted! - Clara