Re: [PATCH v3 3/4] mtd: rawnand: Add support manufacturer specific suspend/resume operation

From: masonccyang
Date: Wed Mar 11 2020 - 01:41:10 EST



Hi Boris,

> > diff --git a/include/linux/mtd/rawnand.h b/include/linux/mtd/rawnand.h
> > index bc2fa3c..c0055ed 100644
> > --- a/include/linux/mtd/rawnand.h
> > +++ b/include/linux/mtd/rawnand.h
> > @@ -1064,6 +1064,8 @@ struct nand_legacy {
> > * @lock: lock protecting the suspended field. Also used to
> > * serialize accesses to the NAND device.
> > * @suspended: set to 1 when the device is suspended, 0 when
it's not.
> > + * @_suspend: [REPLACEABLE] specific NAND device suspend
operation
> > + * @_resume: [REPLACEABLE] specific NAND device resume operation
> > * @bbt: [INTERN] bad block table pointer
> > * @bbt_td: [REPLACEABLE] bad block table descriptor for flash
> > * lookup.
> > @@ -1119,6 +1121,8 @@ struct nand_chip {
> >
> > struct mutex lock;
> > unsigned int suspended : 1;
> > + int (*_suspend)(struct nand_chip *chip);
> > + void (*_resume)(struct nand_chip *chip);
>
> I thought we agreed on not prefixing new hooks with _ ?

For [PATCH v2] series, you mentioned to drop the _ prefix
of _lock/_unlock only and we finally patched to lock_area/unlock_area.

thanks for your time & review.
Mason

CONFIDENTIALITY NOTE:

This e-mail and any attachments may contain confidential information
and/or personal data, which is protected by applicable laws. Please be
reminded that duplication, disclosure, distribution, or use of this e-mail
(and/or its attachments) or any part thereof is prohibited. If you receive
this e-mail in error, please notify us immediately and delete this mail as
well as its attachment(s) from your system. In addition, please be
informed that collection, processing, and/or use of personal data is
prohibited unless expressly permitted by personal data protection laws.
Thank you for your attention and cooperation.

Macronix International Co., Ltd.

=====================================================================



============================================================================

CONFIDENTIALITY NOTE:

This e-mail and any attachments may contain confidential information and/or personal data, which is protected by applicable laws. Please be reminded that duplication, disclosure, distribution, or use of this e-mail (and/or its attachments) or any part thereof is prohibited. If you receive this e-mail in error, please notify us immediately and delete this mail as well as its attachment(s) from your system. In addition, please be informed that collection, processing, and/or use of personal data is prohibited unless expressly permitted by personal data protection laws. Thank you for your attention and cooperation.

Macronix International Co., Ltd.

=====================================================================