On Thu, Apr 28, 2016 at 10:58:20AM +0300, omer.dagan@xxxxxxxxxxx wrote:
From: Omer Dagan <omer.dagan@xxxxxxxxxxx>
diff --git a/drivers/staging/rdma/hfi1/aspm.h b/drivers/staging/rdma/hfi1/aspm.h
index 0d58fe3..384df0a 100644
--- a/drivers/staging/rdma/hfi1/aspm.h
+++ b/drivers/staging/rdma/hfi1/aspm.h
@@ -234,7 +234,7 @@ static inline void aspm_disable_all(struct hfi1_devdata *dd)
{
struct hfi1_ctxtdata *rcd;
unsigned long flags;
- unsigned i;
+ unsigned int i;
Can you please explain the value in doing "unsigned int" vs "unsigned"?
None of the static checkers like checkpatch, sparse, or coccinelle seem to
mind.
u64 read_csr(const struct hfi1_devdata *dd, u32 offset)
{
- if (dd->flags & HFI1_PRESENT) {
+ if (dd->flags & HFI1_PRESENT)
return readq((void __iomem *)dd->kregbase + offset);
- }
+
return -1;
}
Also please don't fix multiple types of issues in one patch. This is very
easy to lose sight of in the review and stands on its own.
This is a checkpatch issue. Likely the result of a previous patch removing a
line of code, something that is only seen when you run checkpatch on the
entire file not just the patch so this is a good find.
-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