Received-SPF: pass (google.com: domain of linux-kernel+bounces-5415-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Date: Tue, 19 Dec 2023 17:52:33 +0300
From: Serge Semin <fancer.lancer@gmail.com>
To: Andy Shevchenko <andy.shevchenko@gmail.com>
Cc: xiongxin <xiongxin@kylinos.cn>, hoan@os.amperecomputing.com, 
	linus.walleij@linaro.org, brgl@bgdev.pl, linux-gpio@vger.kernel.org, 
	linux-kernel@vger.kernel.org, stable@kernel.org, Riwen Lu <luriwen@kylinos.cn>
Subject: Re: [PATCH v3] gpio: dwapb: mask/unmask IRQ when disable/enale it
Message-ID: <ee3fwqa5r3jablflueewun7wzu3g6uezoniloouxfebox5qnyf@e7n63v7teaw2>
References: <20231219013751.20386-1-xiongxin@kylinos.cn>
 <7zdg5ujizncarxvdyahnusojiq44rzxx2zybqj4kzsonzr27gq@fm5wj7npqsk3>
 <CAHp75VceVAZYTNsJaYYRN+EMExFZSQARsJowd-CvDLRtuOPKSg@mail.gmail.com>
 <euhbczna4hk5sacb23i2xwqh2jewlek7cfceprfslpsiijhwk3@3d6vtybmgag5>
 <ZYGsRXJUcrLKEzUn@smile.fi.intel.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <ZYGsRXJUcrLKEzUn@smile.fi.intel.com>

On Tue, Dec 19, 2023 at 04:44:21PM +0200, Andy Shevchenko wrote:
> On Tue, Dec 19, 2023 at 05:31:38PM +0300, Serge Semin wrote:
> > On Tue, Dec 19, 2023 at 04:17:16PM +0200, Andy Shevchenko wrote:
> > > On Tue, Dec 19, 2023 at 11:14 AM Serge Semin <fancer.lancer@gmail.com> wrote:
> > > > On Tue, Dec 19, 2023 at 09:37:51AM +0800, xiongxin wrote:
> > > 
> > > ...
> > > 
> > > > Also note all the tags you've already got must be preserved on the
> > > > next patch revisions. One more time:
> > > 
> > > > Acked-by: Serge Semin <fancer.lancer@gmail.com>
> > > 
> > > I recommend using `b4` for that.
> > > 
> > > it harvests tags from the email thread, so no need to care about
> > > possible misses.
> > 
> > AFAICS it doesn't pick up the tags from the previous revisions at
> > least if the new patch wasn't submitted as in-reply-to the prev one.
> 
> ???
> 
> > Just tested it on v3. b4 found my new ab-tag only and no yours rb-tag.
> > Did you mean something other than I thought you did?
> 

> Grabbing thread from lore.kernel.org/all/mdogxxro42ymeaykrgqpld2kqbppopbywcm76osskuf3df72sl@5jalt26vzcv4/t.mbox.gz

It's _v2_. I was talking about _v3_:

[user@pc] $ b4 am 20231219013751.20386-1-xiongxin@kylinos.cn
Grabbing thread from lore.kernel.org/all/20231219013751.20386-1-xiongxin%40kylinos.cn/t.mbox.gz
Analyzing 5 messages in the thread
Checking attestation on all messages, may take a moment...
---
  [PATCH v3] gpio: dwapb: mask/unmask IRQ when disable/enale it
    + Acked-by: Serge Semin <fancer.lancer@gmail.com> (✓ DKIM/gmail.com)
---
Total patches: 1
---
 Link: https://lore.kernel.org/r/20231219013751.20386-1-xiongxin@kylinos.cn
 Base: applies clean to current tree
       git checkout -b v3_20231219_xiongxin_kylinos_cn HEAD
       git am ./v3_20231219_xiongxin_gpio_dwapb_mask_unmask_irq_when_disable_enale_it.mbx

In anyway, Xiong already submitted v4 with all the tags added:
https://lore.kernel.org/linux-gpio/20231219101620.4617-1-xiongxin@kylinos.cn/
which you've already noticed.

-Serge(y)

> Analyzing 4 messages in the thread
> Checking attestation on all messages, may take a moment...
> ---
>   [PATCH v2] gpio: dwapb: mask/unmask IRQ when disable/enale it
>     + Reviewed-by: Andy Shevchenko <andy@kernel.org>
>     + Acked-by: Serge Semin <fancer.lancer@gmail.com> (✓ DKIM/gmail.com)
> 
> The flow you need to follow is that
> 
> 	git checkout $BASE # gpio/for-next in this case
> 	b4 am ... # as above
> 	git am ...
> 	...address comments...
> 	git commit -a -s --amend
> 	git format-patch ... -1 HEAD~0
> 	git send-email ...
> 
> -- 
> With Best Regards,
> Andy Shevchenko
> 
> 
>