On Mon, Aug 22, 2022 at 08:50:45PM +0800, xinlei.lee wrote: [..] > Hi Nícolas: > > Thanks for your careful review. > I will correct it in the next version: > 1. Modify key nouns in the description; > 2. Modify the label of jitao to Co-developed-by; To be clear, you shouldn't change jitao's signed-off-by to a co-developed-by, but add a co-developed-by. If you check the link I sent earlier [1], it should be clear that you should have something like Co-developed-by: Jitao Shi <jitao.shi@xxxxxxxxxxxx> Signed-off-by: Jitao Shi <jitao.shi@xxxxxxxxxxxx> Signed-off-by: Xinlei Lee <xinlei.lee@xxxxxxxxxxxx> [1] https://www.kernel.org/doc/html/latest/process/submitting-patches.html#when-to-use-acked-by-cc-and-co-developed-by Thanks, Nícolas > 3. Macro definition address lowercase problem and function naming. > > If there are no other questions, I will send out the second edition in > the near future. > > Best Regards! > xinlei >