Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <10ce5bf1-bf12-b028-49cb-a3f1999f9a07@gmail.com>
Date:   Wed, 11 May 2022 23:36:18 +0300
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101
 Thunderbird/91.9.0
Subject: Re: [PATCH] mtd: parsers: add support for Sercomm partitions
Content-Language: en-US
To:     Miquel Raynal <miquel.raynal@bootlin.com>
Cc:     Richard Weinberger <richard@nod.at>,
        Vignesh Raghavendra <vigneshr@ti.com>,
        Rob Herring <robh+dt@kernel.org>,
        NOGUCHI Hiroshi <drvlabo@gmail.com>,
        Hauke Mehrtens <hauke@hauke-m.de>,
        Karim <Karimdplay@gmail.com>, M <x1@disroot.org>,
        linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org,
        devicetree@vger.kernel.org
References: <20220329122016.4122899-1-csharper2005@gmail.com>
 <20220330100958.57dc4d61@xps13>
 <ede9e050-3b60-ba0e-c626-044f7825adb7@gmail.com>
 <20220511091643.064f9d13@xps13>
From:   Mikhail Zhilkin <csharper2005@gmail.com>
In-Reply-To: <20220511091643.064f9d13@xps13>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Precedence: bulk

On 5/11/2022 10:16 AM, Miquel Raynal wrote:

>>>> This is essentially the same code as proposed by NOGUCHI Hiroshi
>>>> <drvlabo@gmail.com> here:  
>>> I would credit Hiroshi with a Suggested-by at least  
>> I read submitting patches rules again and thought that Signed-off-by is
>> suitable for this case. Is this ok?
> Either you take his work almost like it is and he must be the author
> *and* the first signed-off-by line, or you take the authorship if you
> think you did enough modifications to the code and in this case you can
> either credit him with a suggested-by before your signed-off, or you
> can credit him with a co-developed-by + his signed-off and then yours.

Thank you for clarification! My modifications were quite small:
 - Remove redundant braces and logical NOT operator
 - Define pr_fmt
 - Replace kcalloc by kzalloc
 - Use of_get_child_count() and alloc big enough array before the
   for_each_child_of_node()

Therefore, I prefer to keep the authorship of the original author (NOGUCHI Hiroshi). 

Patch is really good and was successfully tested on the hundreds of Sercomm devices (5.4, 5.10, 5.15 Kernel). My motivation for upstreaming is work under official support for these devices in OpenWrt.

What are the next steps from my side in this case? Should I wait for review of this patch here (Link: https://lore.kernel.org/linux-mtd/20220510162655.21011-1-csharper2005@gmail.com/T/#u
) or do I need to do something additionally (for example, send updated set v5 with Acked-by from Krzysztof in the first patch)? 

And sorry for the many questions... I haven't fully mastered the Linux change management process and best practices yet.

> Thanks,
> Miquèl

-- 
Best regards,
Mikhail