RE: Re: [PATCH] ASoC: dwc: fix typo in comment

RE: Re: [PATCH] ASoC: dwc: fix typo in comment

[김명욱]

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

Re: [PATCH] ASoC: dwc: fix typo in comment

[Bagas Sanjaya]

On 12/09/2023 08:23, 김명욱 wrote:
> ISR_RXFO means "Status of Data Overrun interrupt for the RX channel" according to the datasheet.
> 
> So, the comment should be RX, not TX
> 
=

OK, but next time don't reply like:

> A: http://en.wikipedia.org/wiki/Top_post
> Q: Were do I find info about this thing called top-posting?
> A: Because it messes up the order in which people normally read text.
> Q: Why is top-posting such a bad thing?
> A: Top-posting.
> Q: What is the most annoying thing in e-mail?
> 
> A: No.
> Q: Should I include quotations after my reply?
> 
> http://daringfireball.net/2007/07/on_top

because I can't see the context you're replying to.

And don't send HTML emails as mailing lists hate and
block them.

Regardless, send v2 with reviews (especially on patch description like this)
addressed.

Thanks!

-- 
An old man doll... just what I always wanted! - Clara