All of lore.kernel.org
 help / color / mirror / Atom feed
From: Miquel Raynal <miquel.raynal@bootlin.com>
To: Pengpeng Hou <pengpeng@iscas.ac.cn>
Cc: Kyungmin Park <kyungmin.park@samsung.com>,
	 Richard Weinberger <richard@nod.at>,
	 Vignesh Raghavendra <vigneshr@ti.com>,
	linux-mtd@lists.infradead.org,  linux-kernel@vger.kernel.org
Subject: Re: [PATCH] mtd: onenand: samsung: report DMA completion timeouts
Date: Mon, 29 Jun 2026 17:00:16 +0200	[thread overview]
Message-ID: <871pdptmrj.fsf@bootlin.com> (raw)
In-Reply-To: <20260623061233.40188-1-pengpeng@iscas.ac.cn> (Pengpeng Hou's message of "Tue, 23 Jun 2026 14:12:33 +0800")

Hi Pengpeng,

On 23/06/2026 at 14:12:33 +08, Pengpeng Hou <pengpeng@iscas.ac.cn> wrote:

> The S5PC110 OneNAND DMA helpers have bounded waits for transfer
> completion. The polling helper falls out of its timeout loop and returns
> success, and the IRQ helper ignores wait_for_completion_timeout().
>
> Return -ETIMEDOUT when the DMA transfer-done bit or completion does not
> arrive before the timeout so callers can treat the buffer transfer as
> failed.
>
> Signed-off-by: Pengpeng Hou <pengpeng@iscas.ac.cn>

There is a list a commits for which you forgot to Cc: stable and more
importantly set a Fixes tag. Please add the missing tags and send a v2
for the following commits:

[PATCH] mtd: rawnand: lpc32xx_mlc: fail DMA transfers on timeout
[PATCH] mtd: rawnand: lpc32xx_slc: fail DMA transfer on completion timeout
[PATCH] mtd: mchp23k256: Use SPI match data for chip caps
[PATCH] mtd: rawnand: fsl_ifc: return errors for failed page reads
[PATCH] mtd: onenand: samsung: report DMA completion timeouts

Thank you,
Miquèl

WARNING: multiple messages have this Message-ID (diff)
From: Miquel Raynal <miquel.raynal@bootlin.com>
To: Pengpeng Hou <pengpeng@iscas.ac.cn>
Cc: Kyungmin Park <kyungmin.park@samsung.com>,
	 Richard Weinberger <richard@nod.at>,
	 Vignesh Raghavendra <vigneshr@ti.com>,
	linux-mtd@lists.infradead.org,  linux-kernel@vger.kernel.org
Subject: Re: [PATCH] mtd: onenand: samsung: report DMA completion timeouts
Date: Mon, 29 Jun 2026 17:00:16 +0200	[thread overview]
Message-ID: <871pdptmrj.fsf@bootlin.com> (raw)
In-Reply-To: <20260623061233.40188-1-pengpeng@iscas.ac.cn> (Pengpeng Hou's message of "Tue, 23 Jun 2026 14:12:33 +0800")

Hi Pengpeng,

On 23/06/2026 at 14:12:33 +08, Pengpeng Hou <pengpeng@iscas.ac.cn> wrote:

> The S5PC110 OneNAND DMA helpers have bounded waits for transfer
> completion. The polling helper falls out of its timeout loop and returns
> success, and the IRQ helper ignores wait_for_completion_timeout().
>
> Return -ETIMEDOUT when the DMA transfer-done bit or completion does not
> arrive before the timeout so callers can treat the buffer transfer as
> failed.
>
> Signed-off-by: Pengpeng Hou <pengpeng@iscas.ac.cn>

There is a list a commits for which you forgot to Cc: stable and more
importantly set a Fixes tag. Please add the missing tags and send a v2
for the following commits:

[PATCH] mtd: rawnand: lpc32xx_mlc: fail DMA transfers on timeout
[PATCH] mtd: rawnand: lpc32xx_slc: fail DMA transfer on completion timeout
[PATCH] mtd: mchp23k256: Use SPI match data for chip caps
[PATCH] mtd: rawnand: fsl_ifc: return errors for failed page reads
[PATCH] mtd: onenand: samsung: report DMA completion timeouts

Thank you,
Miquèl

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/

  reply	other threads:[~2026-06-29 15:00 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-23  6:12 [PATCH] mtd: onenand: samsung: report DMA completion timeouts Pengpeng Hou
2026-06-23  6:12 ` Pengpeng Hou
2026-06-29 15:00 ` Miquel Raynal [this message]
2026-06-29 15:00   ` Miquel Raynal

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=871pdptmrj.fsf@bootlin.com \
    --to=miquel.raynal@bootlin.com \
    --cc=kyungmin.park@samsung.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mtd@lists.infradead.org \
    --cc=pengpeng@iscas.ac.cn \
    --cc=richard@nod.at \
    --cc=vigneshr@ti.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.