* 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).