Commit ceb20ec6 authored Apr 21, 2025 by Junxian Huang Committed by Greg Kroah-Hartman Jun 27, 2025

RDMA/hns: Include hnae3.h in hns_roce_hw_v2.h



[ Upstream commit 2b11d33de23262cb20d1dcb24b586dbb8f54d463 ]

hns_roce_hw_v2.h has a direct dependency on hnae3.h due to the
inline function hns_roce_write64(), but it doesn't include this
header currently. This leads to that files including
hns_roce_hw_v2.h must also include hnae3.h to avoid compilation
errors, even if they themselves don't really rely on hnae3.h.
This doesn't make sense, hns_roce_hw_v2.h should include hnae3.h
directly.

Fixes: d3743fa9 ("RDMA/hns: Fix the chip hanging caused by sending doorbell during reset")
Signed-off-by: Junxian Huang <huangjunxian6@hisilicon.com>
Link: https://patch.msgid.link/20250421132750.1363348-6-huangjunxian6@hisilicon.com


Signed-off-by: Leon Romanovsky <leon@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>

parent a052c91a

drivers/infiniband/hw/hns/hns_roce_hw_v2.c

+0 −1

Original line number	Diff line number	Diff line
		@@ -41,7 +41,6 @@
		#include <rdma/ib_umem.h>
		#include <rdma/uverbs_ioctl.h>

		#include "hnae3.h"
		#include "hns_roce_common.h"
		#include "hns_roce_device.h"
		#include "hns_roce_cmd.h"

drivers/infiniband/hw/hns/hns_roce_hw_v2.h

+1 −0

Original line number	Diff line number	Diff line
		@@ -34,6 +34,7 @@
		#define _HNS_ROCE_HW_V2_H

		#include <linux/bitops.h>
		#include "hnae3.h"

		#define HNS_ROCE_VF_QPC_BT_NUM 256
		#define HNS_ROCE_VF_SCCC_BT_NUM 64

drivers/infiniband/hw/hns/hns_roce_restrack.c

+0 −1

Original line number	Diff line number	Diff line
		@@ -4,7 +4,6 @@
		#include <rdma/rdma_cm.h>
		#include <rdma/restrack.h>
		#include <uapi/rdma/rdma_netlink.h>
		#include "hnae3.h"
		#include "hns_roce_common.h"
		#include "hns_roce_device.h"
		#include "hns_roce_hw_v2.h"