All of lore.kernel.org
 help / color / mirror / Atom feed
From: Miquel Raynal <miquel.raynal@bootlin.com>
To: Simon Glass <sjg@chromium.org>
Cc: Mikhail Kshevetskiy <mikhail.kshevetskiy@iopsys.eu>,
	Tom Rini <trini@konsulko.com>,
	Dario Binacchi <dario.binacchi@amarulasolutions.com>,
	Michael Trimarchi <michael@amarulasolutions.com>,
	Frieder Schrempf <frieder.schrempf@kontron.de>,
	Jagan Teki <jagan@amarulasolutions.com>,
	William Zhang <william.zhang@broadcom.com>,
	Dmitry Rokosov <ddrokosov@sberdevices.ru>,
	Igor Prusov <ivprusov@salutedevices.com>,
	Bruce Suen <bruce_suen@163.com>,
	Alexey Romanov <avromanov@salutedevices.com>,
	Chuanhong Guo <gch981213@gmail.com>,
	Martin Kurbanov <mmkurbanov@sberdevices.ru>,
	Max Krummenacher <max.krummenacher@toradex.com>,
	Francesco Dolcini <francesco.dolcini@toradex.com>,
	u-boot@lists.denx.de
Subject: Re: [RESEND PATCH v4 00/10] mtd: spinand: initial support of ecc engines
Date: Wed, 9 Oct 2024 10:11:35 +0200	[thread overview]
Message-ID: <20241009101135.6dfc657f@xps-13> (raw)
In-Reply-To: <CAFLszTiJMO+0+tMVPCM8Ho95TzvfqCgYvSLBJzmNbPJXJk-rYQ@mail.gmail.com>

Hi Simon, Mikhail,

sjg@chromium.org wrote on Tue, 8 Oct 2024 19:50:59 -0600:

> Hi Mikhail,
> 
> On Tue, 1 Oct 2024 at 14:45, Mikhail Kshevetskiy
> <mikhail.kshevetskiy@iopsys.eu> wrote:
> >
> > This patch series:
> >  * sync spinand driver code with linux-6.10
> >  * sync spinand flash support with linux-6.10
> >  * add initial support of ecc engines
> >
> > Up to now only software ecc is supported, but other engines can be add quite easily  
> 
> Is there a uclass for ECC?

Not that I am aware of. Indeed maybe a uclass for these engines might
make sense (maybe UCLASS_ECC_ENGINE). The uclass for them might be
simple enough just to match devices with drivers and drivers and perhaps
expose the few callbacks needed (like init/prepare/finish/cleanup).
Then in the NAND layer when looking for a suitable device you could do
a research by class ID.

Thanks,
Miquèl

  reply	other threads:[~2024-10-09  8:11 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-10-01 20:44 [RESEND PATCH v4 00/10] mtd: spinand: initial support of ecc engines Mikhail Kshevetskiy
2024-10-01 20:44 ` [RESEND PATCH v4 01/10] mtd: spinand: Use the spi-mem dirmap API Mikhail Kshevetskiy
2024-10-01 20:44 ` [RESEND PATCH v4 02/10] mtd: spinand: Add a NAND page I/O request type Mikhail Kshevetskiy
2024-10-01 20:44 ` [RESEND PATCH v4 03/10] mtd: spinand: add missed add missed MODULE_DEVICE_TABLE() Mikhail Kshevetskiy
2024-10-01 20:44 ` [RESEND PATCH v4 04/10] mtd: spinand: simulate behavior of linux's function spinand_wait() Mikhail Kshevetskiy
2024-10-01 20:44 ` [RESEND PATCH v4 05/10] mtd: spinand: more use of spinand_to_{something} helpers Mikhail Kshevetskiy
2024-10-01 20:44 ` [RESEND PATCH v4 06/10] mtd: spinand: replace enable_ecc variable with disable_ecc and update corresponding logic Mikhail Kshevetskiy
2024-10-01 20:44 ` [RESEND PATCH v4 07/10] mtd: spinand: minor refactoring Mikhail Kshevetskiy
2024-10-01 20:44 ` [RESEND PATCH v4 08/10] mtd: spinand: more refactoring Mikhail Kshevetskiy
2024-10-01 20:44 ` [RESEND PATCH v4 09/10] mtd: spinand: sync supported flashes with linux-6.10 Mikhail Kshevetskiy
2024-10-01 20:44 ` [RESEND PATCH v4 10/10] mtd: nand: add initial ecc engine support Mikhail Kshevetskiy
2024-10-09  1:50 ` [RESEND PATCH v4 00/10] mtd: spinand: initial support of ecc engines Simon Glass
2024-10-09  8:11   ` Miquel Raynal [this message]
2024-10-09  8:18   ` Mikhail Kshevetskiy
  -- strict thread matches above, loose matches on Subject: below --
2024-09-09 22:23 Mikhail Kshevetskiy
2024-08-14 10:19 Mikhail Kshevetskiy

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=20241009101135.6dfc657f@xps-13 \
    --to=miquel.raynal@bootlin.com \
    --cc=avromanov@salutedevices.com \
    --cc=bruce_suen@163.com \
    --cc=dario.binacchi@amarulasolutions.com \
    --cc=ddrokosov@sberdevices.ru \
    --cc=francesco.dolcini@toradex.com \
    --cc=frieder.schrempf@kontron.de \
    --cc=gch981213@gmail.com \
    --cc=ivprusov@salutedevices.com \
    --cc=jagan@amarulasolutions.com \
    --cc=max.krummenacher@toradex.com \
    --cc=michael@amarulasolutions.com \
    --cc=mikhail.kshevetskiy@iopsys.eu \
    --cc=mmkurbanov@sberdevices.ru \
    --cc=sjg@chromium.org \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    --cc=william.zhang@broadcom.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.