On 10/23/2017 2:38 PM, Leon Romanovsky wrote:
> diff --git a/drivers/infiniband/hw/hfi1/mad.c
b/drivers/infiniband/hw/hfi1/mad.c
index 07b80fa..dfe6224 100644
--- a/drivers/infiniband/hw/hfi1/mad.c
+++ b/drivers/infiniband/hw/hfi1/mad.c
@@ -98,6 +98,16 @@ static inline void clear_opa_smp_data(struct opa_smp *smp)
memset(data, 0, size);
}
+static inline u16 hfi1_lookup_pkey_value(struct hfi1_ibport *ibp, int pkey_idx)
Please, no "inline-function" in *.c files as it is written in CodingStyle.
Sure about that?
As a rule of thumb I agree we should not put inlines in C files though.
However as you can see from the context, this file already has static
inlines so I wasn't too bothered by it.
-Denny
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
