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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxxxxxxxxxxx]
> 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@xxxxxxxxxxxxxxxxxxx; stable@xxxxxxxxxxxxxxx; 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@xxxxxxxxxxxxxxxxxxxx>
> > Reviewed-by: Joakim Tjernlund <Joakim.Tjernlund@xxxxxxxxxxxx>
> > Cc: Chris Packham <chris.packham@xxxxxxxxxxxxxxxxxxx>
> > Cc: Brian Norris <computersforpeace@xxxxxxxxx>
> > Cc: David Woodhouse <dwmw2@xxxxxxxxxxxxx>
> > Cc: Boris Brezillon <boris.brezillon@xxxxxxxxxxxxxxxxxx>
> > Cc: Marek Vasut <marek.vasut@xxxxxxxxx>
> > Cc: Richard Weinberger <richard@xxxxxx>
> > Cc: Cyrille Pitchen <cyrille.pitchen@xxxxxxxxxx>
> > Cc: linux-mtd@xxxxxxxxxxxxxxxxxxx
> > Cc: stable@xxxxxxxxxxxxxxx
> > Signed-off-by: Boris Brezillon <boris.brezillon@xxxxxxxxxxx>
> > ---
> >  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




[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux