[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20230613132934.1da72c99@kernel.org>
Date: Tue, 13 Jun 2023 13:29:34 -0700
From: Jakub Kicinski <kuba@...nel.org>
To: Arınç ÜNAL <arinc.unal@...nc9.com>
Cc: Vladimir Oltean <olteanv@...il.com>, Daniel Golle
<daniel@...rotopia.org>, Landen Chao <Landen.Chao@...iatek.com>, DENG
Qingfang <dqfext@...il.com>, Sean Wang <sean.wang@...iatek.com>, Andrew
Lunn <andrew@...n.ch>, Florian Fainelli <f.fainelli@...il.com>, "David S.
Miller" <davem@...emloft.net>, Eric Dumazet <edumazet@...gle.com>, Paolo
Abeni <pabeni@...hat.com>, Matthias Brugger <matthias.bgg@...il.com>,
AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>,
Russell King <linux@...linux.org.uk>, Frank Wunderlich
<frank-w@...lic-files.de>, Bartel Eerdekens
<bartel.eerdekens@...stell8.be>, mithat.guner@...ont.com,
erkin.bozoglu@...ont.com, linux-kernel@...r.kernel.org,
netdev@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
linux-mediatek@...ts.infradead.org
Subject: Re: [PATCH net v2 2/7] net: dsa: mt7530: fix trapping frames with
multiple CPU ports on MT7530
On Tue, 13 Jun 2023 22:09:02 +0300 Arınç ÜNAL wrote:
> >> It's a new capability that nobody has used, yet, so YMMV :)
> >
> > Interesting, I've got some questions regarding this.
> >
> >> For example to mark a series as Changes Requested one needs to send
> >> the following line anywhere in the email thread:
> >>
> >> pw-bot: changes-requested
> >
> > I suppose a reply to the cover letter or under the cover letter thread
> > applies?
Anywhere in the thread, the bot should walk references.
> >> The use of the bot is restricted to authors of the patches (the From:
> >> header on patch submission and command must match!)
> >
> > So, for example, if this patch series was new, Vladimir would reply to
> > the patch they're the author of with 'pw-bot: changes-requested', and
> > the patchworks would mark the whole patch series as changes requested?
Yes.
> Also, replying with 'pw-bot: changes-requested' on an already accepted
> patch series as the author won't change the state from accepted to
> changes requested, correct?
Yes, authors can only changes status from active to inactive. Accepted
is an inactive state.
Powered by blists - more mailing lists