linux-mtd.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* Does this patch set affect your driver(drivers/mtd/nand/alauda.c)?
@ 2013-08-18 16:07 Huang Shijie
  2013-08-20  3:54 ` Brian Norris
  0 siblings, 1 reply; 3+ messages in thread
From: Huang Shijie @ 2013-08-18 16:07 UTC (permalink / raw)
  To: joern, linux-mtd, Brian Norris, dedekind1

Hi Joern:

I have submitted a patch set to expose the ecc step size to the user space.

see:
http://lists.infradead.org/pipermail/linux-mtd/2013-August/048114.html

Since your driver does not set the ecc.size, nor calls the
nand_scan/nand_scan_ident/nand_scan_tail.

I am not sure if this patch set affect your driver or not.
Please let me know if it does affect your driver.

thanks
Huang Shijie

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Does this patch set affect your driver(drivers/mtd/nand/alauda.c)?
  2013-08-18 16:07 Does this patch set affect your driver(drivers/mtd/nand/alauda.c)? Huang Shijie
@ 2013-08-20  3:54 ` Brian Norris
  2013-08-20 15:12   ` Jörn Engel
  0 siblings, 1 reply; 3+ messages in thread
From: Brian Norris @ 2013-08-20  3:54 UTC (permalink / raw)
  To: Huang Shijie; +Cc: joern, linux-mtd, dedekind1

On Sun, Aug 18, 2013 at 12:07:14PM -0400, Huang Shijie wrote:
> Hi Joern:
> 
> I have submitted a patch set to expose the ecc step size to the user space.
> 
> see:
> http://lists.infradead.org/pipermail/linux-mtd/2013-August/048114.html
> 
> Since your driver does not set the ecc.size, nor calls the
> nand_scan/nand_scan_ident/nand_scan_tail.

The real problem was that by default, we would expose ecc_step_size = 0
for alauda.

> I am not sure if this patch set affect your driver or not.
> Please let me know if it does affect your driver.

I see Joern submitted a patch for this driver's removal, so this
point is now moot.

Thanks,
Brian

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Does this patch set affect your driver(drivers/mtd/nand/alauda.c)?
  2013-08-20  3:54 ` Brian Norris
@ 2013-08-20 15:12   ` Jörn Engel
  0 siblings, 0 replies; 3+ messages in thread
From: Jörn Engel @ 2013-08-20 15:12 UTC (permalink / raw)
  To: Brian Norris; +Cc: linux-mtd, Huang Shijie, dedekind1

On Mon, 19 August 2013 20:54:58 -0700, Brian Norris wrote:
> 
> I see Joern submitted a patch for this driver's removal, so this
> point is now moot.

Yes.  I've had two independent people/groups look into this driver,
which exceeds the entire user base.  That's just a waste of time for
everyone involved.

Jörn

--
Any headline which ends in a question mark can be answered by the word 'no'.
-- Betteridge's Law

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2013-08-20 16:46 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-08-18 16:07 Does this patch set affect your driver(drivers/mtd/nand/alauda.c)? Huang Shijie
2013-08-20  3:54 ` Brian Norris
2013-08-20 15:12   ` Jörn Engel

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).