[PATCH 10/18] mtd: rawnand: factor nand_command_lp() into nand_command()
Arnout Vandecappelle
arnout at mind.be
Mon Apr 23 00:16:23 PDT 2018
Hi Boris,
On 20-04-18 22:34, Boris Brezillon wrote:
> Hi Sam, Arnout,
>
> On Fri, 20 Apr 2018 10:19:38 +0200
> Sam Lefebvre <sam.lefebvre at essensium.com> wrote:
>
>> From: "Arnout Vandecappelle (Essensium/Mind)" <arnout at mind.be>[snip]
>> Note that I don't have access to a small-page device, so only tested on
>> large-page devices. Also only tested on i.MX6Q (gpmi-nand).
>>
>> I only verified the lack of change in behaviour during nand_detect by
>> reading the code, so it's possible that I missed something. Testing on
>> various devices (ONFI, JEDEC, non-ONFI/JEDEC) is needed to be really
>> sure that nothing breaks.
>>
>> Note that this patch can be removed from the series without affecting
>> the rest.
>
> Hm, I don't want to risk any regression, so I'm gonna pass on this
> patch, especially since we're trying to get rid of ->cmdfunc() in favor
> or ->exec_op().
Absolutely. This patch was included more to show how patch 11 was constructed.
I completely agree that it's too risky.
Regards,
Arnout
>
> The same goes for patch 9, sorry.
>
> Regards,
>
> Boris
[snip]
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF
More information about the linux-mtd
mailing list