]> git.dujemihanovic.xyz Git - linux.git/commit
net: qede: use return from qede_parse_flow_attr() for flower
authorAsbjørn Sloth Tønnesen <ast@fiberby.net>
Fri, 26 Apr 2024 09:12:24 +0000 (09:12 +0000)
committerDavid S. Miller <davem@davemloft.net>
Mon, 29 Apr 2024 09:02:43 +0000 (10:02 +0100)
commitfcee2065a178f78be6fd516302830378b17dba3d
treeee2ed634f5a8b7d8256e92c9a9338f4341353fd8
parente25714466abd9d96901b15efddf82c60a38abd86
net: qede: use return from qede_parse_flow_attr() for flower

In qede_add_tc_flower_fltr(), when calling
qede_parse_flow_attr() then the return code
was only used for a non-zero check, and then
-EINVAL was returned.

qede_parse_flow_attr() can currently fail with:
* -EINVAL
* -EOPNOTSUPP
* -EPROTONOSUPPORT

This patch changes the code to use the actual
return code, not just return -EINVAL.

The blaimed commit introduced these functions.

Only compile tested.

Fixes: 2ce9c93eaca6 ("qede: Ingress tc flower offload (drop action) support.")
Signed-off-by: Asbjørn Sloth Tønnesen <ast@fiberby.net>
Reviewed-by: Simon Horman <horms@kernel.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/qlogic/qede/qede_filter.c