diff mbox series

[PATCHv3,net-next,2/4] net: team: rename team to team_core for linking

Message ID 20240329082847.1902685-3-liuhangbin@gmail.com (mailing list archive)
State Superseded
Delegated to: Netdev Maintainers
Headers show
Series doc/netlink: add a YAML spec for team | expand

Checks

Context Check Description
netdev/series_format success Posting correctly formatted
netdev/tree_selection success Clearly marked for net-next
netdev/ynl success Generated files up to date; no warnings/errors; GEN HAS DIFF 2 files changed, 879 insertions(+);
netdev/fixes_present success Fixes tag not required for -next series
netdev/header_inline success No static functions without inline keyword in header files
netdev/build_32bit success Errors and warnings before: 943 this patch: 943
netdev/build_tools success No tools touched, skip
netdev/cc_maintainers success CCed 5 of 5 maintainers
netdev/build_clang success Errors and warnings before: 954 this patch: 954
netdev/verify_signedoff success Signed-off-by tag matches author and committer
netdev/deprecated_api success None detected
netdev/check_selftest success No net selftest shell script
netdev/verify_fixes success No Fixes tag
netdev/build_allmodconfig_warn success Errors and warnings before: 954 this patch: 954
netdev/checkpatch warning WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
netdev/build_clang_rust success No Rust files in patch. Skipping build
netdev/kdoc success Errors and warnings before: 0 this patch: 0
netdev/source_inline success Was 0 now: 0
netdev/contest success net-next-2024-03-29--15-00 (tests: 950)

Commit Message

Hangbin Liu March 29, 2024, 8:28 a.m. UTC
Similar with commit 08d323234d10 ("net: fou: rename the source for linking"),
We'll need to link two objects together to form the team module.
This means the source can't be called team, the build system expects
team.o to be the combined object.

Signed-off-by: Hangbin Liu <liuhangbin@gmail.com>
---
 Documentation/netlink/specs/team.yaml    | 2 --
 drivers/net/team/Makefile                | 1 +
 drivers/net/team/{team.c => team_core.c} | 0
 3 files changed, 1 insertion(+), 2 deletions(-)
 rename drivers/net/team/{team.c => team_core.c} (100%)

Comments

Jakub Kicinski March 29, 2024, 10:08 p.m. UTC | #1
On Fri, 29 Mar 2024 16:28:45 +0800 Hangbin Liu wrote:
> diff --git a/Documentation/netlink/specs/team.yaml b/Documentation/netlink/specs/team.yaml
> index 907f54c1f2e3..c13529e011c9 100644
> --- a/Documentation/netlink/specs/team.yaml
> +++ b/Documentation/netlink/specs/team.yaml
> @@ -202,5 +202,3 @@ operations:
>            attributes:
>              - team-ifindex
>              - list-port
> -            - item-port
> -            - attr-port

I think you squashed this into the wrong patch :S
Hangbin Liu April 1, 2024, 2:41 a.m. UTC | #2
On Fri, Mar 29, 2024 at 03:08:04PM -0700, Jakub Kicinski wrote:
> On Fri, 29 Mar 2024 16:28:45 +0800 Hangbin Liu wrote:
> > diff --git a/Documentation/netlink/specs/team.yaml b/Documentation/netlink/specs/team.yaml
> > index 907f54c1f2e3..c13529e011c9 100644
> > --- a/Documentation/netlink/specs/team.yaml
> > +++ b/Documentation/netlink/specs/team.yaml
> > @@ -202,5 +202,3 @@ operations:
> >            attributes:
> >              - team-ifindex
> >              - list-port
> > -            - item-port
> > -            - attr-port
> 
> I think you squashed this into the wrong patch :S

Ah, my bad. I need more careful and double checks before post them...

Hangbin
diff mbox series

Patch

diff --git a/Documentation/netlink/specs/team.yaml b/Documentation/netlink/specs/team.yaml
index 907f54c1f2e3..c13529e011c9 100644
--- a/Documentation/netlink/specs/team.yaml
+++ b/Documentation/netlink/specs/team.yaml
@@ -202,5 +202,3 @@  operations:
           attributes:
             - team-ifindex
             - list-port
-            - item-port
-            - attr-port
diff --git a/drivers/net/team/Makefile b/drivers/net/team/Makefile
index f582d81a5091..244db32c1060 100644
--- a/drivers/net/team/Makefile
+++ b/drivers/net/team/Makefile
@@ -3,6 +3,7 @@ 
 # Makefile for the network team driver
 #
 
+team-y:= team_core.o
 obj-$(CONFIG_NET_TEAM) += team.o
 obj-$(CONFIG_NET_TEAM_MODE_BROADCAST) += team_mode_broadcast.o
 obj-$(CONFIG_NET_TEAM_MODE_ROUNDROBIN) += team_mode_roundrobin.o
diff --git a/drivers/net/team/team.c b/drivers/net/team/team_core.c
similarity index 100%
rename from drivers/net/team/team.c
rename to drivers/net/team/team_core.c