Commit 18256cb2 authored by Przemek Kitszel's avatar Przemek Kitszel Committed by David S. Miller
Browse files

qed: devlink health: use retained error fmsg API 



Drop unneeded error checking.

devlink_fmsg_*() family of functions is now retaining errors,
so there is no need to check for them after each call.

Reviewed-by: default avatarJiri Pirko <jiri@nvidia.com>
Signed-off-by: default avatarPrzemek Kitszel <przemyslaw.kitszel@intel.com>
Reviewed-by: default avatarSimon Horman <horms@kernel.org>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent d17f98bf

drivers/net/ethernet/qlogic/qed/qed_devlink.c

+3 −3
Original line number Diff line number Diff line
@@ -66,12 +66,12 @@ qed_fw_fatal_reporter_dump(struct devlink_health_reporter *reporter, 
		return err;

	}



	err = devlink_fmsg_binary_pair_put(fmsg, "dump_data",

					   p_dbg_data_buf, dbg_data_buf_size);

	devlink_fmsg_binary_pair_put(fmsg, "dump_data", p_dbg_data_buf,

				     dbg_data_buf_size);



	vfree(p_dbg_data_buf);



	return err;

	return 0;

}



static int