]> www.infradead.org Git - users/jedix/linux-maple.git/commitdiff
accel/habanalabs: fix a static warning - 'dubious: x & !y'
authorKoby Elbaz <kelbaz@habana.ai>
Thu, 20 Apr 2023 11:17:31 +0000 (14:17 +0300)
committerOded Gabbay <ogabbay@kernel.org>
Thu, 8 Jun 2023 09:35:54 +0000 (12:35 +0300)
Use a straight forward approach to get a conditional result.

Signed-off-by: Koby Elbaz <kelbaz@habana.ai>
Reviewed-by: Oded Gabbay <ogabbay@kernel.org>
Signed-off-by: Oded Gabbay <ogabbay@kernel.org>
drivers/accel/habanalabs/gaudi2/gaudi2.c

index 058741698d7137fc4eaec1218997b863541adabd..240fecfab6087eb2652d1096388070426d21c4aa 100644 (file)
@@ -4527,7 +4527,7 @@ static int gaudi2_set_tpc_engine_mode(struct hl_device *hdev, u32 engine_id, u32
        reg_base = gaudi2_tpc_cfg_blocks_bases[tpc_id];
        reg_addr = reg_base + TPC_CFG_STALL_OFFSET;
        reg_val = FIELD_PREP(DCORE0_TPC0_CFG_TPC_STALL_V_MASK,
-                       !!(engine_command == HL_ENGINE_STALL));
+                       (engine_command == HL_ENGINE_STALL) ? 1 : 0);
        WREG32(reg_addr, reg_val);
 
        if (engine_command == HL_ENGINE_RESUME) {
@@ -4551,7 +4551,7 @@ static int gaudi2_set_mme_engine_mode(struct hl_device *hdev, u32 engine_id, u32
        reg_base = gaudi2_mme_ctrl_lo_blocks_bases[mme_id];
        reg_addr = reg_base + MME_CTRL_LO_QM_STALL_OFFSET;
        reg_val = FIELD_PREP(DCORE0_MME_CTRL_LO_QM_STALL_V_MASK,
-                       !!(engine_command == HL_ENGINE_STALL));
+                       (engine_command == HL_ENGINE_STALL) ? 1 : 0);
        WREG32(reg_addr, reg_val);
 
        return 0;
@@ -4572,7 +4572,7 @@ static int gaudi2_set_edma_engine_mode(struct hl_device *hdev, u32 engine_id, u3
        reg_base = gaudi2_dma_core_blocks_bases[edma_id];
        reg_addr = reg_base + EDMA_CORE_CFG_STALL_OFFSET;
        reg_val = FIELD_PREP(DCORE0_EDMA0_CORE_CFG_1_HALT_MASK,
-                       !!(engine_command == HL_ENGINE_STALL));
+                       (engine_command == HL_ENGINE_STALL) ? 1 : 0);
        WREG32(reg_addr, reg_val);
 
        if (engine_command == HL_ENGINE_STALL) {