From: Luo bin <luob...@huawei.com>
Date: Thu, 23 Jul 2020 22:40:37 +0800

> +static int hinic_fw_reporter_dump(struct devlink_health_reporter *reporter,
> +                               struct devlink_fmsg *fmsg, void *priv_ctx,
> +                               struct netlink_ext_ack *extack)
> +{
> +     struct hinic_mgmt_watchdog_info *watchdog_info;
> +     int err;
> +
> +     if (priv_ctx) {
> +             watchdog_info = priv_ctx;
> +             err = mgmt_watchdog_report_show(fmsg, watchdog_info);
> +             if (err)
> +                     return err;
> +     }
> +
> +     return 0;
> +}

This 'watchdog_info' variable is completely unnecessary, just pass
'priv_ctx' as-is into mgmt_watchdog_report_show().

Reply via email to