diff options
| author | Johannes Berg <johannes.berg@intel.com> | 2013-11-14 20:14:42 +0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2013-11-15 02:10:40 +0400 |
| commit | b61a5eea5904198d45e775c342be010a23decbe3 (patch) | |
| tree | 1b1a6b78601a8d414a46344e1f3ef9ab5fae510e /net/wimax/op-state-get.c | |
| parent | 1c582d915da13ca21ad375ae04ec1bd6193418b2 (diff) | |
| download | linux-b61a5eea5904198d45e775c342be010a23decbe3.tar.xz | |
wimax: use genl_register_family_with_ops()
This simplifies the code since there's no longer a need to
have error handling in the registration.
Unfortunately it means more extern function declarations are
needed, but the overall goal would seem to justify this.
Due to the removal of duplication in the netlink policies,
this reduces the size of wimax by almost 1k.
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/wimax/op-state-get.c')
| -rw-r--r-- | net/wimax/op-state-get.c | 17 |
1 files changed, 0 insertions, 17 deletions
diff --git a/net/wimax/op-state-get.c b/net/wimax/op-state-get.c index aff8776e2d41..995c08c827b5 100644 --- a/net/wimax/op-state-get.c +++ b/net/wimax/op-state-get.c @@ -33,13 +33,6 @@ #include "debug-levels.h" -static const struct nla_policy wimax_gnl_state_get_policy[WIMAX_GNL_ATTR_MAX + 1] = { - [WIMAX_GNL_STGET_IFIDX] = { - .type = NLA_U32, - }, -}; - - /* * Exporting to user space over generic netlink * @@ -48,7 +41,6 @@ static const struct nla_policy wimax_gnl_state_get_policy[WIMAX_GNL_ATTR_MAX + 1 * * No attributes. */ -static int wimax_gnl_doit_state_get(struct sk_buff *skb, struct genl_info *info) { int result, ifindex; @@ -72,12 +64,3 @@ error_no_wimax_dev: d_fnend(3, NULL, "(skb %p info %p) = %d\n", skb, info, result); return result; } - - -struct genl_ops wimax_gnl_state_get = { - .cmd = WIMAX_GNL_OP_STATE_GET, - .flags = GENL_ADMIN_PERM, - .policy = wimax_gnl_state_get_policy, - .doit = wimax_gnl_doit_state_get, - .dumpit = NULL, -}; |