All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH iproute2-next v2 0/2] bridge: vlan: add compressvlans manpage
@ 2024-03-22 12:39 Date Huang
  2024-03-22 12:58 ` Nikolay Aleksandrov
  2024-03-26 17:20 ` patchwork-bot+netdevbpf
  0 siblings, 2 replies; 3+ messages in thread
From: Date Huang @ 2024-03-22 12:39 UTC (permalink / raw)
  To: roopa, razor, jiri; +Cc: netdev, bridge, tjjh89017

Hi maintainers

I followed Nikolay and Jiri's comment and updated the patch to v2.
Please check it.

Date Huang (2):
  bridge: vlan: fix compressvlans usage
  bridge: vlan: add compressvlans manpage

 bridge/bridge.c   | 2 +-
 man/man8/bridge.8 | 6 ++++++
 2 files changed, 7 insertions(+), 1 deletion(-)

-- 
2.34.1


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH iproute2-next v2 0/2] bridge: vlan: add compressvlans manpage
  2024-03-22 12:39 [PATCH iproute2-next v2 0/2] bridge: vlan: add compressvlans manpage Date Huang
@ 2024-03-22 12:58 ` Nikolay Aleksandrov
  2024-03-26 17:20 ` patchwork-bot+netdevbpf
  1 sibling, 0 replies; 3+ messages in thread
From: Nikolay Aleksandrov @ 2024-03-22 12:58 UTC (permalink / raw)
  To: Date Huang, roopa, jiri; +Cc: netdev, bridge

On 3/22/24 14:39, Date Huang wrote:
> Hi maintainers
> 
> I followed Nikolay and Jiri's comment and updated the patch to v2.
> Please check it.
> 

Your cover letter should contain an overview of what the set is doing.

> Date Huang (2):
>    bridge: vlan: fix compressvlans usage
>    bridge: vlan: add compressvlans manpage
> 
>   bridge/bridge.c   | 2 +-
>   man/man8/bridge.8 | 6 ++++++
>   2 files changed, 7 insertions(+), 1 deletion(-)
> 

Generally it is good to give people time to review and wait at least
24 hours before reposting another version. Also please write the
changes between versions, something like:

v3: ...
v2: split the patch into two separate patches
     changed the option in patch 01
     changed the man page description in patch 02

Keep the history from previous postings. Also you can note what changed
in each individual patch for the version under the first ---, for 
example for patch 02:
---
v2: changed the man page description

Cheers,
  Nik


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH iproute2-next v2 0/2] bridge: vlan: add compressvlans manpage
  2024-03-22 12:39 [PATCH iproute2-next v2 0/2] bridge: vlan: add compressvlans manpage Date Huang
  2024-03-22 12:58 ` Nikolay Aleksandrov
@ 2024-03-26 17:20 ` patchwork-bot+netdevbpf
  1 sibling, 0 replies; 3+ messages in thread
From: patchwork-bot+netdevbpf @ 2024-03-26 17:20 UTC (permalink / raw)
  To: Date Huang; +Cc: roopa, razor, jiri, netdev, bridge

Hello:

This series was applied to iproute2/iproute2.git (main)
by Stephen Hemminger <stephen@networkplumber.org>:

On Fri, 22 Mar 2024 20:39:21 +0800 you wrote:
> Hi maintainers
> 
> I followed Nikolay and Jiri's comment and updated the patch to v2.
> Please check it.
> 
> Date Huang (2):
>   bridge: vlan: fix compressvlans usage
>   bridge: vlan: add compressvlans manpage
> 
> [...]

Here is the summary with links:
  - [iproute2-next,v2,1/2] bridge: vlan: fix compressvlans usage
    https://git.kernel.org/pub/scm/network/iproute2/iproute2.git/commit/?id=43b53968634e
  - [iproute2-next,v2,2/2] bridge: vlan: fix compressvlans usage
    https://git.kernel.org/pub/scm/network/iproute2/iproute2.git/commit/?id=9ccf8fa8d4d7

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html



^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2024-03-26 17:20 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-03-22 12:39 [PATCH iproute2-next v2 0/2] bridge: vlan: add compressvlans manpage Date Huang
2024-03-22 12:58 ` Nikolay Aleksandrov
2024-03-26 17:20 ` patchwork-bot+netdevbpf

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.