[PATCH 1/4] mtd: cfi_cmdset_0002: Change write buffer to check correct value

[ikegami@xxxxxxxxxxxxxxxxxxxx (IKEGAMI Tokunori)]

Hi Greg-san,

The commit IDs on mainline are below.

  1. dfeae1073583 mtd: cfi_cmdset_0002: Change write buffer to check correct value
  2. 85a82e28b023 mtd: cfi_cmdset_0002: Change definition naming to retry write operation
  3. 45f75b8a919a mtd: cfi_cmdset_0002: Change erase functions to retry for error
  4. 79ca484b6130 mtd: cfi_cmdset_0002: Change erase functions to check chip good only

Note:
  The patches just sent today as revised are only existing with commit IDs on my local environment.
  Also the patches are existing on the Linux MTD development patchwork as below but the commit IDs are not assigned.
    <https://patchwork.ozlabs.org/project/linux-mtd/list/?series=53133>

  If those revised patches should be applied anywhere to have commit IDs at first please let me know.
  Probably it should be consulted with the MTD maintainer Boris-san.

Regards,
Ikegami

> -----Original Message-----
> From: Greg Kroah-Hartman [mailto:gregkh at linuxfoundation.org]
> Sent: Monday, July 02, 2018 3:19 PM
> To: IKEGAMI Tokunori
> Cc: PACKHAM Chris; Brian Norris; David Woodhouse; Boris Brezillon; Marek
> Vasut; Richard Weinberger; Cyrille Pitchen;
> linux-mtd at lists.infradead.org; stable at vger.kernel.org; Boris
> Brezillon
> Subject: Re: [PATCH 1/4] mtd: cfi_cmdset_0002: Change write buffer to
> check correct value
> 
> On Mon, Jul 02, 2018 at 11:50:42AM +0900, Tokunori Ikegami wrote:
> > For the word write it is checked if the chip has the correct value.
> > But it is not checked for the write buffer as only checked if ready.
> > To make sure for the write buffer change to check the value.
> >
> > It is enough as this patch is only checking the last written word.
> > Since it is described by data sheets to check the operation status.
> >
> > Signed-off-by: Tokunori Ikegami <ikegami at allied-telesis.co.jp>
> > Reviewed-by: Joakim Tjernlund <Joakim.Tjernlund at infinera.com>
> > Cc: Chris Packham <chris.packham at alliedtelesis.co.nz>
> > Cc: Brian Norris <computersforpeace at gmail.com>
> > Cc: David Woodhouse <dwmw2 at infradead.org>
> > Cc: Boris Brezillon <boris.brezillon at free-electrons.com>
> > Cc: Marek Vasut <marek.vasut at gmail.com>
> > Cc: Richard Weinberger <richard at nod.at>
> > Cc: Cyrille Pitchen <cyrille.pitchen at wedev4u.fr>
> > Cc: linux-mtd at lists.infradead.org
> > Cc: stable at vger.kernel.org
> > Signed-off-by: Boris Brezillon <boris.brezillon at bootlin.com>
> > ---
> >  drivers/mtd/chips/cfi_cmdset_0002.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> What are the git commit ids for all of these patches, and what stable
> tree do they need to go to?
> 
> thanks,
> 
> greg k-h