IB/isert: Simplify signature cap check
authorMax Gurtovoy <mgurtovoy@nvidia.com>
Sun, 10 Jan 2021 11:19:03 +0000 (11:19 +0000)
committerJason Gunthorpe <jgg@nvidia.com>
Mon, 18 Jan 2021 20:59:04 +0000 (16:59 -0400)
Use if/else clause instead of "condition ? val1 : val2" to make the code
cleaner and simpler.

Link: https://lore.kernel.org/r/20210110111903.486681-3-mgurtovoy@nvidia.com
Reviewed-by: Israel Rukshin <israelr@nvidia.com>
Signed-off-by: Max Gurtovoy <mgurtovoy@nvidia.com>
Acked-by: Sagi Grimberg <sagi@grimberg.me>
Signed-off-by: Jason Gunthorpe <jgg@nvidia.com>
drivers/infiniband/ulp/isert/ib_isert.c

index 96514f6..7305ed8 100644 (file)
@@ -230,8 +230,10 @@ isert_create_device_ib_res(struct isert_device *device)
        }
 
        /* Check signature cap */
-       device->pi_capable = ib_dev->attrs.device_cap_flags &
-                            IB_DEVICE_INTEGRITY_HANDOVER ? true : false;
+       if (ib_dev->attrs.device_cap_flags & IB_DEVICE_INTEGRITY_HANDOVER)
+               device->pi_capable = true;
+       else
+               device->pi_capable = false;
 
        return 0;
 }