RE: [PATCH 0/3] mtd: cfi_cmdset_0002: Fix flash write issue for OpenWrt Project

[IKEGAMI Tokunori <ikegami@xxxxxxxxxxxxxxxxxxxx>]

Sorry let me resend the mail as text mail format since error returned.

From: IKEGAMI Tokunori 
Sent: Friday, October 19, 2018 6:20 PM
To: 'FabioB'; boris.brezillon@xxxxxxxxxxx; 'Hauke Mehrtens'; 'Koen Vandeputte'
Cc: linux-mtd@xxxxxxxxxxxxxxxxxxx; PACKHAM Chris; Joakim.Tjernlund@xxxxxxxxxxxx
Subject: RE: [PATCH 0/3] mtd: cfi_cmdset_0002: Fix flash write issue for OpenWrt Project

Thank you so much for your comments.

The original patch in OpenWRT is below.
　<https://github.com/openwrt/openwrt/blob/v18.06.0/target/linux/ar71xx/patches-4.9/403-mtd_fix_cfi_cmdset_0002_status_check.patch>

So I will do add Hauke-san and Koen-san also in this patch mail.

Hauke-san and Koen-san,

　Can you also agree to fix the above OpenWRT patch in the upstream by me?
　Of course if you like please do it by yourself but I think that you are very busy so I will do this if okay.

By the way how can I add the originator name in the commit message with tag?
So I have just checked the Documentation and I think that the Co-Developed-by tag can be used for the originator name.
Is this correct?
Anyway I will do update the patches.


From: FabioB [mailto:fbettoni@xxxxxxxxx] 
Sent: Friday, October 19, 2018 5:37 PM
To: boris.brezillon@xxxxxxxxxxx
Cc: IKEGAMI Tokunori; linux-mtd@xxxxxxxxxxxxxxxxxxx; PACKHAM Chris; Joakim.Tjernlund@xxxxxxxxxxxx
Subject: Re: [PATCH 0/3] mtd: cfi_cmdset_0002: Fix flash write issue for OpenWrt Project

Hi Boris,
sure it's OK to handover autorship to Tokunori,
I wrote to him to ask for help and he told me to restore removed patch.
So I only had to edit by hand because patching phase was not working during build.
Tokunori must have all authorship and credits to the solution.

Fabio

Il giorno ven 19 ott 2018 alle ore 10:27 Boris Brezillon <boris.brezillon@xxxxxxxxxxx> ha scritto:
+mtd ML and all people CC-ed on other patches

On Fri, 19 Oct 2018 17:13:17 +0900
Tokunori Ikegami <ikegami@xxxxxxxxxxxxxxxxxxxx> wrote:

> 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://git.openwrt.org/?p=openwrt/openwrt.git;a=commitdiff;h=ddc11c3932c7b7b7df7d5fbd48f207e77619eaa7>
> Also includes some refactoring changes.
> 
> Tokunori Ikegami (3):
>   mtd: cfi_cmdset_0002: Change do_write_oneword() to use chip_good()
>   mtd: cfi_cmdset_0002: Remove chip_ready() from do_write_buffer()
>   mtd: cfi_cmdset_0002: Remove goto statemnet from do_write_buffer()

I see that Fabio was the original author of the OpenWRT patch. We
usually keep authorship when porting patching from one project to
another, so I just want to check with Fabio if he's okay to handover
this authorship to you.

> 
>  drivers/mtd/chips/cfi_cmdset_0002.c | 68 ++++++++++++++++-------------
>  1 file changed, 38 insertions(+), 30 deletions(-)
> 
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/