net: sparx5: flower: add extack to sparx5_tc_flower_handler_control_usage()

Define extack locally, to reduce line lengths and aid future users.

Only compile tested.

Signed-off-by: Asbjørn Sloth Tønnesen <ast@fiberby.net>
Reviewed-by: Daniel Machon <daniel.machon@microchip.com>
Tested-by: Daniel Machon <daniel.machon@microchip.com>
Link: https://lore.kernel.org/r/20240424121632.459022-3-ast@fiberby.net
Signed-off-by: Jakub Kicinski <kuba@kernel.org>

diff --git a/drivers/net/ethernet/microchip/sparx5/sparx5_tc_flower.c b/drivers/net/ethernet/microchip/sparx5/sparx5_tc_flower.c
index ca9e69d39439e9030bf718acf59b4015cb0ca474..d0dca65f8e7677d515dce6945e643a4168554afc 100644 (file)
--- a/drivers/net/ethernet/microchip/sparx5/sparx5_tc_flower.c
+++ b/drivers/net/ethernet/microchip/sparx5/sparx5_tc_flower.c
@@ -159,6 +159,7 @@ out:
 static int
 sparx5_tc_flower_handler_control_usage(struct vcap_tc_flower_parse_usage *st)
 {
+       struct netlink_ext_ack *extack = st->fco->common.extack;
        struct flow_match_control mt;
        u32 value, mask;
        int err = 0;
@@ -178,7 +179,7 @@ sparx5_tc_flower_handler_control_usage(struct vcap_tc_flower_parse_usage *st)
                u8 vdt = sparx5_vcap_frag_map[is_frag_idx][first_frag_idx];
 
                if (vdt == FRAG_INVAL) {
-                       NL_SET_ERR_MSG_MOD(st->fco->common.extack,
+                       NL_SET_ERR_MSG_MOD(extack,
                                           "Match on invalid fragment flag combination");
                        return -EINVAL;
                }
@@ -199,7 +200,7 @@ sparx5_tc_flower_handler_control_usage(struct vcap_tc_flower_parse_usage *st)
        return err;
 
 out:
-       NL_SET_ERR_MSG_MOD(st->fco->common.extack, "ip_frag parse error");
+       NL_SET_ERR_MSG_MOD(extack, "ip_frag parse error");
        return err;
 }