[PATCH 11/18] mtd: rawnand: gpmi: instantiate cmdfunc
Boris Brezillon
boris.brezillon at bootlin.com
Fri Apr 20 13:38:45 PDT 2018
Hi Sam, Arnout,
On Fri, 20 Apr 2018 10:19:39 +0200
Sam Lefebvre <sam.lefebvre at essensium.com> wrote:
> From: "Arnout Vandecappelle (Essensium/Mind)" <arnout at mind.be>
>
> Later patches will optimize the command handling for gpmi. This
> requires a gpmmi-specific implementation of cmdfunc. As a first step,
> nand_command() is copied literally from nand_base.c (after merging
> nand_command() and nand_command_lp()).
NACK. As said in my review of patch 10, we're trying to move a many
drivers as possible to the ->exec_op() approach in order to ease
maintenance of the NAND subsystem. What you're doing here is going in
the wrong direction.
Please work on a solution based on ->exec_op().
Regards,
Boris
More information about the linux-mtd
mailing list