qed: Remove set but not used variable 'p_archipelago' (470b9254) · Commits · 戴 / test

Fixes gcc '-Wunused-but-set-variable' warning: drivers/net/ethernet/qlogic/qed/qed_ooo.c: In function 'qed_ooo_delete_isles': drivers/net/ethernet/qlogic/qed/qed_ooo.c:354:30: warning: variable 'p_archipelago' set but not used [-Wunused-but-set-variable] drivers/net/ethernet/qlogic/qed/qed_ooo.c: In function 'qed_ooo_join_isles': drivers/net/ethernet/qlogic/qed/qed_ooo.c:463:30: warning: variable 'p_archipelago' set but not used [-Wunused-but-set-variable] Since commit ("qed: Make OOO archipelagos into an array"), 'p_archipelago' is no longer in use. Signed-off-by:

YueHaibing <yuehaibing@huawei.com> Signed-off-by:

David S. Miller <davem@davemloft.net>

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

+0 −4

Original line number	Diff line number	Diff line
		@@ -351,11 +351,9 @@ void qed_ooo_delete_isles(struct qed_hwfn *p_hwfn,
		struct qed_ooo_info *p_ooo_info,
		u32 cid, u8 drop_isle, u8 drop_size)
		{
		struct qed_ooo_archipelago *p_archipelago = NULL;
		struct qed_ooo_isle *p_isle = NULL;
		u8 isle_idx;

		p_archipelago = qed_ooo_seek_archipelago(p_hwfn, p_ooo_info, cid);
		for (isle_idx = 0; isle_idx < drop_size; isle_idx++) {
		p_isle = qed_ooo_seek_isle(p_hwfn, p_ooo_info, cid, drop_isle);
		if (!p_isle) {
		@@ -460,7 +458,6 @@ void qed_ooo_add_new_buffer(struct qed_hwfn *p_hwfn,
		void qed_ooo_join_isles(struct qed_hwfn *p_hwfn,
		struct qed_ooo_info *p_ooo_info, u32 cid, u8 left_isle)
		{
		struct qed_ooo_archipelago *p_archipelago = NULL;
		struct qed_ooo_isle *p_right_isle = NULL;
		struct qed_ooo_isle *p_left_isle = NULL;

		@@ -473,7 +470,6 @@ void qed_ooo_join_isles(struct qed_hwfn *p_hwfn,
		return;
		}

		p_archipelago = qed_ooo_seek_archipelago(p_hwfn, p_ooo_info, cid);
		list_del(&p_right_isle->list_entry);
		p_ooo_info->cur_isles_number--;
		if (left_isle) {

Admin message