public inbox for netdev@vger.kernel.org 
 help / color / mirror / Atom feed
From: Jiri Pirko <jiri@resnulli•us>
To: sfeldma@gmail•com
Cc: netdev@vger•kernel.org, roopa@cumulusnetworks•com,
	linux@roeck-us•net, f.fainelli@gmail•com,
	sridhar.samudrala@intel•com, ronen.arad@intel•com,
	andrew@lunn•ch
Subject: Re: [PATCH net-next v4 07/24] switchdev: add port vlan obj
Date: Mon, 13 Apr 2015 15:27:03 +0200	[thread overview]
Message-ID: <20150413132703.GD2090@nanopsycho.lan> (raw)
In-Reply-To: <20150413131625.GC2090@nanopsycho.lan>

Mon, Apr 13, 2015 at 03:16:25PM CEST, jiri@resnulli•us wrote:
>Mon, Apr 13, 2015 at 08:17:01AM CEST, sfeldma@gmail•com wrote:
>>From: Scott Feldman <sfeldma@gmail•com>
>>
>>VLAN obj has flags (PVID and untagged) as well as start and end vid ranges.
>>The swdev driver can optimize programing the device using the ranges.
>>
>>Signed-off-by: Scott Feldman <sfeldma@gmail•com>
>>---
>> include/net/switchdev.h |    8 ++++++++
>> 1 file changed, 8 insertions(+)
>>
>>diff --git a/include/net/switchdev.h b/include/net/switchdev.h
>>index f5f7733..d3cc8eb 100644
>>--- a/include/net/switchdev.h
>>+++ b/include/net/switchdev.h
>>@@ -45,12 +45,20 @@ struct fib_info;
>> 
>> enum swdev_obj_id {
>> 	SWDEV_OBJ_UNDEFINED,
>>+	SWDEV_OBJ_PORT_VLAN,
>> };
>> 
>> struct swdev_obj {
>> 	enum swdev_obj_id id;
>> 	enum swdev_trans trans;
>> 	u32 flags;
>>+	union {
>>+		struct swdev_obj_vlan {			/* PORT_VLAN */
>>+			u16 flags;
>>+			u16 vid_start;
>>+			u16 vid_end;
>>+		} vlan;
>>+	};
>
>Same here. Please squash this with the next patch.

I mean 09. The infrastructure should be first, usage (by driver) after
that.

>
>
>> };
>> 
>> /**
>>-- 
>>1.7.10.4
>>

  reply	other threads:[~2015-04-13 13:27 UTC|newest]

Thread overview: 46+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-04-13  6:16 [PATCH net-next v4 00/24] switchdev: spring cleanup sfeldma
2015-04-13  6:16 ` [PATCH net-next v4 01/24] switchdev: introduce get/set attrs ops sfeldma
2015-04-13 10:43   ` Jiri Pirko
2015-04-14  7:02     ` Scott Feldman
2015-04-13  6:16 ` [PATCH net-next v4 02/24] switchdev: convert parent_id_get to swdev attr get sfeldma
2015-04-13  6:16 ` [PATCH net-next v4 03/24] switchdev: convert STP update to swdev attr set sfeldma
2015-04-13 19:22   ` Florian Fainelli
2015-04-14  7:51     ` Scott Feldman
2015-04-13  6:16 ` [PATCH net-next v4 04/24] switchdev: add bridge port flags attr sfeldma
2015-04-13  6:16 ` [PATCH net-next v4 05/24] rocker: use swdev get/set attr for bridge port flags sfeldma
2015-04-13 10:48   ` Jiri Pirko
2015-04-15  5:25   ` Simon Horman
2015-04-15  6:03     ` Scott Feldman
2015-04-15  8:03       ` Simon Horman
2015-04-13  6:17 ` [PATCH net-next v4 06/24] switchdev: introduce swdev add/del obj ops sfeldma
2015-04-13  6:17 ` [PATCH net-next v4 07/24] switchdev: add port vlan obj sfeldma
2015-04-13 13:16   ` Jiri Pirko
2015-04-13 13:27     ` Jiri Pirko [this message]
2015-04-13 17:49   ` Florian Fainelli
2015-04-14  7:28     ` Scott Feldman
2015-04-13  6:17 ` [PATCH net-next v4 08/24] rocker: use swdev add/del obj for bridge port vlans sfeldma
2015-04-13  6:17 ` [PATCH net-next v4 09/24] switchdev: add new swdev bridge setlink sfeldma
2015-04-14  4:09   ` roopa
2015-04-14  5:30     ` Jiri Pirko
2015-04-14  5:46   ` roopa
2015-04-13  6:17 ` [PATCH net-next v4 10/24] rocker: cut over to new swdev_port_bridge_setlink sfeldma
2015-04-13  6:17 ` [PATCH net-next v4 11/24] bonding: " sfeldma
2015-04-13  6:17 ` [PATCH net-next v4 12/24] team: " sfeldma
2015-04-13  6:17 ` [PATCH net-next v4 13/24] switchdev: remove old netdev_switch_port_bridge_setlink sfeldma
2015-04-14  4:07   ` roopa
2015-04-13  6:17 ` [PATCH net-next v4 14/24] switchdev: add new swdev_port_bridge_dellink sfeldma
2015-04-13  6:17 ` [PATCH net-next v4 15/24] rocker: cut over to " sfeldma
2015-04-13  6:17 ` [PATCH net-next v4 16/24] bonding: " sfeldma
2015-04-13  6:17 ` [PATCH net-next v4 17/24] team: " sfeldma
2015-04-13  6:17 ` [PATCH net-next v4 18/24] switchdev: remove unused netdev_switch_port_bridge_dellink sfeldma
2015-04-13  6:17 ` [PATCH net-next v4 19/24] switchdev: add new swdev_port_bridge_getlink sfeldma
2015-04-14  5:47   ` roopa
2015-04-14  7:59     ` Scott Feldman
2015-04-13  6:17 ` [PATCH net-next v4 20/24] rocker: cut over to " sfeldma
2015-04-13  6:17 ` [PATCH net-next v4 21/24] bonding: " sfeldma
2015-04-13  6:17 ` [PATCH net-next v4 22/24] team: " sfeldma
2015-04-13  6:17 ` [PATCH net-next v4 23/24] switchdev: convert fib_ipv4_add/del over to swdev_port_obj_add/del sfeldma
2015-04-13  6:17 ` [PATCH net-next v4 24/24] switchdev: bring documentation up-to-date sfeldma
2015-04-13  8:24 ` [PATCH net-next v4 00/24] switchdev: spring cleanup Jiri Pirko
2015-04-14  5:47 ` roopa
2015-04-17  4:48   ` roopa

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150413132703.GD2090@nanopsycho.lan \
    --to=jiri@resnulli$(echo .)us \
    --cc=andrew@lunn$(echo .)ch \
    --cc=f.fainelli@gmail$(echo .)com \
    --cc=linux@roeck-us$(echo .)net \
    --cc=netdev@vger$(echo .)kernel.org \
    --cc=ronen.arad@intel$(echo .)com \
    --cc=roopa@cumulusnetworks$(echo .)com \
    --cc=sfeldma@gmail$(echo .)com \
    --cc=sridhar.samudrala@intel$(echo .)com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox