[RESEND PATCH v4 0/4] mtd: cfi_cmdset_0002: Add OTP support for Micron M29EW NOR flashes

[RESEND PATCH v4 0/4] mtd: cfi_cmdset_0002: Add OTP support for Micron M29EW NOR flashes

[Christian Riesch]

Hello Brian,

The first version of this patchset was posted to the list in February 2013.
There were a few comments to these patches (from Artem), which I have
addressed in separate patches (merged by you on March 9).

Since there are no further objections against these patches (after a review
time of more than a year), I would like to ask you to merge them as well.

Best regards,
Christian 


This patchset adds support for the reading, writing, and locking of the
one time programmable (OTP) memory of Micron M29EW NOR flashes.

These flashes have a 256 byte OTP memory, which is called extended
memory region in its datasheet. The flash is compatible to Spansion S29GL
and Macronix MX29GL devices, so support for the OTP memory of these
flashes should also be possible.

Tests with this patchset were done with a single Micron M29EW device
(512 Mbit, 16 bit) on a custom board with a Texas Instruments AM1808
system on chip.

Prerequisites:
- Applies on linux-mtd.git

Changes for v2:
- dropped one patch that changes the detection of M29EW devices,
  this patch is currently not required
- added support for locking
- fixed a few bugs in the loop that walks through the OTP memory

Changes for v3:
- Added retlet parameter to cfi_amdstd_get_{fact,user}_prot_info
- Fixed partial write of words

Changes for v4:
- Updated prerequisites in cover letter

Christian Riesch (4):
  mtd: cfi_cmdset_0002: Add support for reading OTP
  mtd: cfi_cmdset_0002: Invalidate cache after entering/exiting OTP
    memory
  mtd: cfi_cmdset_0002: Add support for writing OTP memory
  mtd: cfi_cmdset_0002: Add support for locking OTP memory

 drivers/mtd/chips/cfi_cmdset_0002.c |  350 +++++++++++++++++++++++++++++++++--
 1 file changed, 331 insertions(+), 19 deletions(-)

-- 
1.7.9.5

Re: [RESEND PATCH v4 0/4] mtd: cfi_cmdset_0002: Add OTP support for Micron M29EW NOR flashes

[Christian Riesch]

Hello Brian,

On Mon, May 5, 2014 at 8:14 AM, Christian Riesch
<christian.riesch@omicron.at> wrote:
> Hello Brian,
>
> The first version of this patchset was posted to the list in February 2013.
> There were a few comments to these patches (from Artem), which I have
> addressed in separate patches (merged by you on March 9).
>
> Since there are no further objections against these patches (after a review
> time of more than a year), I would like to ask you to merge them as well.

Since you didn't take this patchset for the 3.16 merge window, I'd
like to know if there is anything I should change here. Otherwise I
would like to ask you to apply this patchset.
Thanks, Christian

>
> Best regards,
> Christian
>
>
> This patchset adds support for the reading, writing, and locking of the
> one time programmable (OTP) memory of Micron M29EW NOR flashes.
>
> These flashes have a 256 byte OTP memory, which is called extended
> memory region in its datasheet. The flash is compatible to Spansion S29GL
> and Macronix MX29GL devices, so support for the OTP memory of these
> flashes should also be possible.
>
> Tests with this patchset were done with a single Micron M29EW device
> (512 Mbit, 16 bit) on a custom board with a Texas Instruments AM1808
> system on chip.
>
> Prerequisites:
> - Applies on linux-mtd.git
>
> Changes for v2:
> - dropped one patch that changes the detection of M29EW devices,
>   this patch is currently not required
> - added support for locking
> - fixed a few bugs in the loop that walks through the OTP memory
>
> Changes for v3:
> - Added retlet parameter to cfi_amdstd_get_{fact,user}_prot_info
> - Fixed partial write of words
>
> Changes for v4:
> - Updated prerequisites in cover letter
>
> Christian Riesch (4):
>   mtd: cfi_cmdset_0002: Add support for reading OTP
>   mtd: cfi_cmdset_0002: Invalidate cache after entering/exiting OTP
>     memory
>   mtd: cfi_cmdset_0002: Add support for writing OTP memory
>   mtd: cfi_cmdset_0002: Add support for locking OTP memory
>
>  drivers/mtd/chips/cfi_cmdset_0002.c |  350 +++++++++++++++++++++++++++++++++--
>  1 file changed, 331 insertions(+), 19 deletions(-)
>
> --
> 1.7.9.5
>
>
> ______________________________________________________
> Linux MTD discussion mailing list
> http://lists.infradead.org/mailman/listinfo/linux-mtd/

Re: [RESEND PATCH v4 0/4] mtd: cfi_cmdset_0002: Add OTP support for Micron M29EW NOR flashes

[Brian Norris]

Hi Christian,

On Mon, Jul 07, 2014 at 02:22:51PM +0200, Christian Riesch wrote:
> On Mon, May 5, 2014 at 8:14 AM, Christian Riesch <christian.riesch@omicron.at> wrote:
> > The first version of this patchset was posted to the list in February 2013.
> > There were a few comments to these patches (from Artem), which I have
> > addressed in separate patches (merged by you on March 9).
> >
> > Since there are no further objections against these patches (after a review
> > time of more than a year), I would like to ask you to merge them as well.
> 
> Since you didn't take this patchset for the 3.16 merge window, I'd
> like to know if there is anything I should change here. Otherwise I
> would like to ask you to apply this patchset.

Overlooked again, sorry.

> >
> > Best regards,
> > Christian
> >
> >
> > This patchset adds support for the reading, writing, and locking of the
> > one time programmable (OTP) memory of Micron M29EW NOR flashes.
> >
> > These flashes have a 256 byte OTP memory, which is called extended
> > memory region in its datasheet. The flash is compatible to Spansion S29GL
> > and Macronix MX29GL devices, so support for the OTP memory of these
> > flashes should also be possible.
> >
> > Tests with this patchset were done with a single Micron M29EW device
> > (512 Mbit, 16 bit) on a custom board with a Texas Instruments AM1808
> > system on chip.
> >
> > Prerequisites:
> > - Applies on linux-mtd.git
> >
> > Changes for v2:
> > - dropped one patch that changes the detection of M29EW devices,
> >   this patch is currently not required
> > - added support for locking
> > - fixed a few bugs in the loop that walks through the OTP memory
> >
> > Changes for v3:
> > - Added retlet parameter to cfi_amdstd_get_{fact,user}_prot_info
> > - Fixed partial write of words
> >
> > Changes for v4:
> > - Updated prerequisites in cover letter
> >
> > Christian Riesch (4):
> >   mtd: cfi_cmdset_0002: Add support for reading OTP
> >   mtd: cfi_cmdset_0002: Invalidate cache after entering/exiting OTP
> >     memory
> >   mtd: cfi_cmdset_0002: Add support for writing OTP memory
> >   mtd: cfi_cmdset_0002: Add support for locking OTP memory
> >
> >  drivers/mtd/chips/cfi_cmdset_0002.c |  350 +++++++++++++++++++++++++++++++++--
> >  1 file changed, 331 insertions(+), 19 deletions(-)

Pushed to l2-mtd.git. Thanks!

Brian