nfp: handle cls_flower command default case

jpirko · davem330 · commit eba7927b5593 · 2018-06-25T16:14:03.000+09:00
Currently the default case is not handled, which with future command
introductions would introduce a warning. So handle it.

Signed-off-by: Jiri Pirko &lt;jiri@mellanox.com&gt;
Acked-by: Jakub Kicinski &lt;jakub.kicinski@netronome.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
diff --git a/drivers/net/ethernet/netronome/nfp/flower/offload.c b/drivers/net/ethernet/netronome/nfp/flower/offload.c
@@ -576,9 +576,9 @@ nfp_flower_repr_offload(struct nfp_app *app, struct net_device *netdev,
 		return nfp_flower_del_offload(app, netdev, flower, egress);
 	case TC_CLSFLOWER_STATS:
 		return nfp_flower_get_stats(app, netdev, flower, egress);
+	default:
+		return -EOPNOTSUPP;
 	}
-
-	return -EOPNOTSUPP;
 }
 
 int nfp_flower_setup_tc_egress_cb(enum tc_setup_type type, void *type_data,

Original file line number	Diff line number	Diff line change
`@@ -576,9 +576,9 @@ nfp_flower_repr_offload(struct nfp_app app, struct net_device netdev,`
`576`	`576`	`return nfp_flower_del_offload(app, netdev, flower, egress);`
`577`	`577`	`case TC_CLSFLOWER_STATS:`
`578`	`578`	`return nfp_flower_get_stats(app, netdev, flower, egress);`
	`579`	`+ default:`
	`580`	`+ return -EOPNOTSUPP;`
`579`	`581`	`}`
`580`		`-`
`581`		`- return -EOPNOTSUPP;`
`582`	`582`	`}`
`583`	`583`
`584`	`584`	`int nfp_flower_setup_tc_egress_cb(enum tc_setup_type type, void *type_data,`