On 21/08/19 1:33 PM, Joakim Tjernlund wrote: > On Thu, 2019-06-20 at 01:49 +0900, Tokunori Ikegami wrote: >> CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe. >> >> >> The change is based on the fix for flash erase to use chip_good() done in >> the past. And it is fixed as same way in the OpenWrt Project as below. >> <https://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgit.openwrt.org%2F%3Fp%3Dopenwrt%2Fopenwrt.git%3Ba%3Dcommitdiff%3Bh%3Dddc11c3932&data=02%7C01%7CJoakim.Tjernlund%40infinera.com%7Cc576b3ddad2d4f3c09f008d6f4d635b2%7C285643de5f5b4b03a1530ae2dc8aaf77%7C1%7C0%7C636965598194534713&sdata=1XV5w3jD6tbfP%2B2Q2eBqfeHp07kHLDxpiJhW4ZA5Ea8%3D&reserved=0> >> Also includes some refactoring changes. >> > > Did this go anywhere? > I have this series in my queue and will consider for 5.4-rc1. Regards Vignesh > Jocke > >> Signed-off-by: Tokunori Ikegami <ikegami.t@xxxxxxxxx> >> Cc: Fabio Bettoni <fbettoni@xxxxxxxxx> >> Co: Hauke Mehrtens <hauke@xxxxxxxxxx> >> Cc: Chris Packham <chris.packham@xxxxxxxxxxxxxxxxxxx> >> Cc: Joakim Tjernlund <Joakim.Tjernlund@xxxxxxxxxxxx> >> Cc: linux-mtd@xxxxxxxxxxxxxxxxxxx >> >> Tokunori Ikegami (9): >> mtd: cfi_cmdset_0002: Use chip_good() to retry in do_write_oneword() >> mtd: cfi_cmdset_0002: Remove goto statement from do_write_buffer() >> mtd: cfi_cmdset_0002: Split do_write_oneword() to reduce function size >> mtd: cfi_cmdset_0002: Split do_write_oneword() op_done goto statement >> mtd: cfi_cmdset_0002: Remove op_done goto statement from >> do_write_oneword() >> mtd: cfi_cmdset_0002: Split write-to-buffer-reset sequence >> mtd: cfi_cmdset_0002: Split to wait write buffer to check if completed >> mtd: cfi_cmdset_0002: Split do_write_oneword() to reduce exit paths >> mtd: cfi_cmdset_0002: Disable write buffer functions if >> FORCE_WORD_WRITE is 1 >> >> drivers/mtd/chips/cfi_cmdset_0002.c | 290 ++++++++++++++++++++++-------------- >> 1 file changed, 182 insertions(+), 108 deletions(-) >> >> -- >> 2.11.0 >> > -- Regards Vignesh ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/