On Wed, 2018-01-31 at 10:28 -0500, Devesh Sharma wrote: > + if (cq->deferred_arm) { > + bnxt_re_ring_cq_arm_db(cq, cq->deferred_arm_flags); > + cq->deferred_arm = false; > + cq->deferred_arm_flags = 0; Minor nit: setting the flags to 0 each time serves no purpose as the flags are always universally set whenever deferred_arm is set to true. -- Doug Ledford <dledford@xxxxxxxxxx> GPG KeyID: B826A3330E572FDD Key fingerprint = AE6B 1BDA 122B 23B4 265B 1274 B826 A333 0E57 2FDD
Attachment:
signature.asc
Description: This is a digitally signed message part