[PATCH RFC] mtd/nand: implement user otp for Micron chips
Matthieu CASTET
matthieu.castet at parrot.com
Thu Feb 21 04:47:28 EST 2013
Uwe Kleine-König a écrit :
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig at pengutronix.de>
> ---
> Hello,
>
> there are some things to clean up before this patch can be applied. See
> the respective comments in the code, most of them are marked with XXX,
> but I'm sure you'll find things to criticize that I didn't mark :-)
>
> About the unsuitablity of nand_do_read_ops and nand_do_write_ops I'd like to
> hear your thoughts. Would you prefer a new function that does only the
> necessary stuff, or should I add another parameter to make them do the
> right thing for me?
>
> BTW, to make use of this patch you'd need
> http://patchwork.ozlabs.org/patch/221191/
>
> Best regards
> Uwe
>
> drivers/mtd/nand/nand_base.c | 126 ++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 126 insertions(+)
>
> diff --git a/drivers/mtd/nand/nand_base.c b/drivers/mtd/nand/nand_base.c
> index 3766682..10e51f3 100644
> --- a/drivers/mtd/nand/nand_base.c
> +++ b/drivers/mtd/nand/nand_base.c
> @@ -2753,6 +2753,116 @@ static int nand_onfi_get_features(struct mtd_info *mtd, struct nand_chip *chip,
> return 0;
Shouldn't vendor specific code go in another file ?
Matthieu
More information about the linux-mtd
mailing list