Commit c50f8564f283 ("IB: Add CNP opcode enumeration.") added an opcode from the IBTA 3.1 specification. Add a comment in the source referencing the relevant specification sections. Reviewed-by: Dennis Dalessandro <dennis.dalessandro@xxxxxxxxx> Signed-off-by: Mike Marciniszyn <mike.marciniszyn@xxxxxxxxx> --- drivers/staging/hfi1/TODO | 2 -- include/rdma/ib_pack.h | 1 + 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/hfi1/TODO b/drivers/staging/hfi1/TODO index 2529149..cc2e51d 100644 --- a/drivers/staging/hfi1/TODO +++ b/drivers/staging/hfi1/TODO @@ -3,5 +3,3 @@ July, 2015 - Remove unneeded file entries in sysfs - Remove software processing of IB protocol and place in library for use by qib, ipath (if still present), hfi1, and eventually soft-roce -- Clean up comments in code around CNP opcode - diff --git a/include/rdma/ib_pack.h b/include/rdma/ib_pack.h index 03f5217..709a533 100644 --- a/include/rdma/ib_pack.h +++ b/include/rdma/ib_pack.h @@ -76,6 +76,7 @@ enum { IB_OPCODE_UC = 0x20, IB_OPCODE_RD = 0x40, IB_OPCODE_UD = 0x60, + /* per IBTA 3.1 Table 38, A10.3.2 */ IB_OPCODE_CNP = 0x80, /* operations -- just used to define real constants */ -- 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