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
