From: kernel test robot <lkp@intel.com> To: Evan Quan <evan.quan@amd.com> Cc: kbuild-all@lists.01.org, linux-kernel@vger.kernel.org, Alex Deucher <alexander.deucher@amd.com> Subject: drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c:1186:21: warning: Variable 'adev->in_hibernate' is reassigned a value before the old one has been used. Date: Fri, 31 Jul 2020 09:48:23 +0800 [thread overview] Message-ID: <202007310904.RMLh8Rsr%lkp@intel.com> (raw) tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: e2c46b5762c616c249201688d3b9846627f78d2c commit: b2b6290a23986a5c88384887b8a589a3c4ebe292 drm/amdgpu: enable hibernate support on Navi1X date: 3 months ago compiler: arc-elf-gcc (GCC) 9.3.0 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp@intel.com> cppcheck warnings: (new ones prefixed by >>) In file included from drivers/gpu/drm/amd/amdgpu/../powerplay/navi10_ppt.c: >> drivers/gpu/drm/amd/powerplay/navi10_ppt.c:728:9: warning: Identical condition 'ret', second condition is always false [identicalConditionAfterEarlyExit] return ret; ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:719:6: note: first condition if (ret) ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:728:9: note: second condition return ret; ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1006:9: warning: Identical condition 'ret', second condition is always false [identicalConditionAfterEarlyExit] return ret; ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1001:6: note: first condition if (ret) ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1006:9: note: second condition return ret; ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1208:9: warning: Identical condition 'ret', second condition is always false [identicalConditionAfterEarlyExit] return ret; ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1203:6: note: first condition if (ret) ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1208:9: note: second condition return ret; ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1226:9: warning: Identical condition 'ret', second condition is always false [identicalConditionAfterEarlyExit] return ret; ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1220:6: note: first condition if (ret) ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1226:9: note: second condition return ret; ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1695:9: warning: Identical condition 'ret', second condition is always false [identicalConditionAfterEarlyExit] return ret; ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1692:6: note: first condition if (ret) ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1695:9: note: second condition return ret; ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1763:9: warning: Identical condition 'ret', second condition is always false [identicalConditionAfterEarlyExit] return ret; ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1760:6: note: first condition if (ret) ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1763:9: note: second condition return ret; ^ >> drivers/gpu/drm/amd/powerplay/navi10_ppt.c:887:12: warning: %u in format string (no. 3) requires 'unsigned int' but the argument type is 'signed int'. [invalidPrintfArgType_uint] size += sprintf(buf + size, "%d: %uMHz @ %umVn", i, curve_settings[0], curve_settings[1] / NAVI10_VOLTAGE_SCALE); ^ >> drivers/gpu/drm/amd/powerplay/navi10_ppt.c:918:12: warning: %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint] size += sprintf(buf + size, "VDDC_CURVE_VOLT[0]: %7dmV %11dmVn", ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:918:12: warning: %d in format string (no. 2) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint] size += sprintf(buf + size, "VDDC_CURVE_VOLT[0]: %7dmV %11dmVn", ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:926:12: warning: %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint] size += sprintf(buf + size, "VDDC_CURVE_VOLT[1]: %7dmV %11dmVn", ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:926:12: warning: %d in format string (no. 2) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint] size += sprintf(buf + size, "VDDC_CURVE_VOLT[1]: %7dmV %11dmVn", ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:934:12: warning: %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint] size += sprintf(buf + size, "VDDC_CURVE_VOLT[2]: %7dmV %11dmVn", ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:934:12: warning: %d in format string (no. 2) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint] size += sprintf(buf + size, "VDDC_CURVE_VOLT[2]: %7dmV %11dmVn", ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1277:11: warning: %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint] size += sprintf(buf + size, "%2d %14s%s:n", ^ >> drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c:811:6: warning: Variable 'res' is reassigned a value before the old one has been used. 'break;' missing? [redundantAssignInSwitch] res = 0; ^ drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c:768:0: note: Variable 'res' is reassigned a value before the old one has been used. 'break;' missing? int res = -ENODEV; ^ drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c:811:6: note: Variable 'res' is reassigned a value before the old one has been used. 'break;' missing? res = 0; ^ >> drivers/gpu/drm/amd/display/dc/dce110/dce110_resource.c:1247:33: warning: Variable 'ctx->dc->caps.max_slave_planes' is reassigned a value before the old one has been used. [redundantAssignment] ctx->dc->caps.max_slave_planes = 1; ^ drivers/gpu/drm/amd/display/dc/dce110/dce110_resource.c:1246:33: note: Variable 'ctx->dc->caps.max_slave_planes' is reassigned a value before the old one has been used. ctx->dc->caps.max_slave_planes = 1; ^ drivers/gpu/drm/amd/display/dc/dce110/dce110_resource.c:1247:33: note: Variable 'ctx->dc->caps.max_slave_planes' is reassigned a value before the old one has been used. ctx->dc->caps.max_slave_planes = 1; ^ -- In file included from drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_resource.c: >> drivers/gpu/drm/amd/display/dc/core/dc_resource.c:2355:10: warning: Variable 'support' is reassigned a value before the old one has been used. 'break;' missing? [redundantAssignInSwitch] support = stream->content_support; ^ drivers/gpu/drm/amd/display/dc/core/dc_resource.c:2251:0: note: Variable 'support' is reassigned a value before the old one has been used. 'break;' missing? union display_content_support support = {0}; ^ drivers/gpu/drm/amd/display/dc/core/dc_resource.c:2355:10: note: Variable 'support' is reassigned a value before the old one has been used. 'break;' missing? support = stream->content_support; ^ >> drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:1897:15: warning: Variable 'max_link_cap' is reassigned a value before the old one has been used. [redundantAssignment] max_link_cap = get_max_link_cap(link); ^ drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:1877:0: note: Variable 'max_link_cap' is reassigned a value before the old one has been used. struct dc_link_settings max_link_cap = {0}; ^ drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:1897:15: note: Variable 'max_link_cap' is reassigned a value before the old one has been used. max_link_cap = get_max_link_cap(link); ^ >> drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:1919:24: warning: Variable 'initial_link_settings' is reassigned a value before the old one has been used. [redundantAssignment] initial_link_settings = get_common_supported_link_settings( ^ drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:1880:0: note: Variable 'initial_link_settings' is reassigned a value before the old one has been used. struct dc_link_settings initial_link_settings = {0}; ^ drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:1919:24: note: Variable 'initial_link_settings' is reassigned a value before the old one has been used. initial_link_settings = get_common_supported_link_settings( ^ drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:2030:15: warning: Variable 'max_link_cap' is reassigned a value before the old one has been used. [redundantAssignment] max_link_cap = get_max_link_cap(link); ^ drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:2028:0: note: Variable 'max_link_cap' is reassigned a value before the old one has been used. struct dc_link_settings max_link_cap = {0}; ^ drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:2030:15: note: Variable 'max_link_cap' is reassigned a value before the old one has been used. max_link_cap = get_max_link_cap(link); ^ >> drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:3491:10: warning: Variable 'status' is reassigned a value before the old one has been used. [redundantAssignment] status = core_link_read_dpcd( ^ drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:3486:10: note: Variable 'status' is reassigned a value before the old one has been used. status = core_link_read_dpcd( ^ drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:3491:10: note: Variable 'status' is reassigned a value before the old one has been used. status = core_link_read_dpcd( ^ drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:3496:10: warning: Variable 'status' is reassigned a value before the old one has been used. [redundantAssignment] status = core_link_read_dpcd( ^ drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:3491:10: note: Variable 'status' is reassigned a value before the old one has been used. status = core_link_read_dpcd( ^ drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:3496:10: note: Variable 'status' is reassigned a value before the old one has been used. status = core_link_read_dpcd( ^ >> drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c:1186:21: warning: Variable 'adev->in_hibernate' is reassigned a value before the old one has been used. [redundantAssignment] adev->in_hibernate = false; ^ drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c:1184:21: note: Variable 'adev->in_hibernate' is reassigned a value before the old one has been used. adev->in_hibernate = true; ^ drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c:1186:21: note: Variable 'adev->in_hibernate' is reassigned a value before the old one has been used. adev->in_hibernate = false; ^ >> drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c:180:2: warning: Address of local auto-variable assigned to a function parameter. [autoVariables] *f = &fence->base; ^ >> drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c:524:17: warning: Boolean result is used in bitwise operation. Clarify expression with parentheses. [clarifyCondition] if (!(!!enable ^ !!amdgpu_ras_is_feature_enabled(adev, head))) ^ drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c:573:17: warning: Boolean result is used in bitwise operation. Clarify expression with parentheses. [clarifyCondition] if (!(!!enable ^ !!amdgpu_ras_is_feature_enabled(adev, head))) ^ >> drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c:4589:9: warning: Identical condition 'r', second condition is always false [identicalConditionAfterEarlyExit] return r; ^ drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c:4586:6: note: first condition if (r) ^ drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c:4589:9: note: second condition return r; ^ >> drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c:1113:9: warning: Identical condition 'r', second condition is always false [identicalConditionAfterEarlyExit] return r; ^ drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c:1110:6: note: first condition if (r) ^ drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c:1113:9: note: second condition return r; ^ >> drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c:1313:40: warning: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] info->prot_valid = protections & 0x7 ? true : false; ^ drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c:1314:39: warning: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] info->prot_read = protections & 0x8 ? true : false; ^ drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c:1315:41: warning: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] info->prot_write = protections & 0x10 ? true : false; ^ drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c:1316:40: warning: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] info->prot_exec = protections & 0x20 ? true : false; ^ >> drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c:1246:9: warning: Identical condition 'r', second condition is always false [identicalConditionAfterEarlyExit] return r; ^ drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c:1243:6: note: first condition if (r) ^ drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c:1246:9: note: second condition return r; ^ >> drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c:1497:40: warning: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] info->prot_valid = protections & 0x7 ? true : false; ^ drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c:1498:39: warning: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] info->prot_read = protections & 0x8 ? true : false; ^ drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c:1499:41: warning: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] info->prot_write = protections & 0x10 ? true : false; ^ drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c:1500:40: warning: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] info->prot_exec = protections & 0x20 ? true : false; ^ >> drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:137:2: warning: Local variable tmp shadows outer variable [shadowVar] WREG32_SOC15_RLC(GC, 0, mmMC_VM_MX_L1_TLB_CNTL, tmp); ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:121:11: note: Shadowed declaration uint32_t tmp; ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:137:2: note: Shadow variable WREG32_SOC15_RLC(GC, 0, mmMC_VM_MX_L1_TLB_CNTL, tmp); ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:154:2: warning: Local variable tmp shadows outer variable [shadowVar] WREG32_SOC15_RLC(GC, 0, mmVM_L2_CNTL, tmp); ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:142:11: note: Shadowed declaration uint32_t tmp; ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:154:2: note: Shadow variable WREG32_SOC15_RLC(GC, 0, mmVM_L2_CNTL, tmp); ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:159:2: warning: Local variable tmp shadows outer variable [shadowVar] WREG32_SOC15_RLC(GC, 0, mmVM_L2_CNTL2, tmp); ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:142:11: note: Shadowed declaration uint32_t tmp; ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:159:2: note: Shadow variable WREG32_SOC15_RLC(GC, 0, mmVM_L2_CNTL2, tmp); ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:171:2: warning: Local variable tmp shadows outer variable [shadowVar] WREG32_SOC15_RLC(GC, 0, mmVM_L2_CNTL3, tmp); ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:142:11: note: Shadowed declaration uint32_t tmp; ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:171:2: note: Shadow variable WREG32_SOC15_RLC(GC, 0, mmVM_L2_CNTL3, tmp); ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:176:2: warning: Local variable tmp shadows outer variable [shadowVar] WREG32_SOC15_RLC(GC, 0, mmVM_L2_CNTL4, tmp); ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:142:11: note: Shadowed declaration uint32_t tmp; ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:176:2: note: Shadow variable WREG32_SOC15_RLC(GC, 0, mmVM_L2_CNTL4, tmp); ^ >> drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:316:2: warning: Local variable i shadows outer variable [shadowVar] WREG32_SOC15_RLC(GC, 0, mmMC_VM_MX_L1_TLB_CNTL, tmp); ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:303:6: note: Shadowed declaration u32 i; ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:316:2: note: Shadow variable WREG32_SOC15_RLC(GC, 0, mmMC_VM_MX_L1_TLB_CNTL, tmp); ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:316:2: warning: Local variable tmp shadows outer variable [shadowVar] WREG32_SOC15_RLC(GC, 0, mmMC_VM_MX_L1_TLB_CNTL, tmp); ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:302:6: note: Shadowed declaration u32 tmp; ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:316:2: note: Shadow variable WREG32_SOC15_RLC(GC, 0, mmMC_VM_MX_L1_TLB_CNTL, tmp); ^ >> drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c:1180:4: warning: Variable 'r' is reassigned a value before the old one has been used. 'break;' missing? [redundantAssignInSwitch] r = amdgpu_irq_add_id(adev, SOC15_IH_CLIENTID_VMC, VMC_1_0__SRCID__VM_FAULT, ^ drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c:1112:4: note: Variable 'r' is reassigned a value before the old one has been used. 'break;' missing? r = amdgpu_atomfirmware_get_vram_info(adev, ^ drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c:1180:4: note: Variable 'r' is reassigned a value before the old one has been used. 'break;' missing? r = amdgpu_irq_add_id(adev, SOC15_IH_CLIENTID_VMC, VMC_1_0__SRCID__VM_FAULT, ^ >> drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c:1117:7: warning: Variable 'err' is reassigned a value before the old one has been used. [redundantAssignment] err = request_firmware(&adev->gfx.mec_fw, fw_name, adev->dev); ^ drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c:1070:6: note: Variable 'err' is reassigned a value before the old one has been used. err = amdgpu_ucode_validate(adev->gfx.rlc_fw); ^ drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c:1117:7: note: Variable 'err' is reassigned a value before the old one has been used. err = request_firmware(&adev->gfx.mec_fw, fw_name, adev->dev); ^ >> drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c:1896:7: warning: Variable 'ring' is reassigned a value before the old one has been used. [redundantAssignment] ring = &adev->gfx.compute_ring[ring_id]; ^ drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c:1894:0: note: Variable 'ring' is reassigned a value before the old one has been used. struct amdgpu_ring *ring = &adev->gfx.compute_ring[ring_id]; ^ drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c:1896:7: note: Variable 'ring' is reassigned a value before the old one has been used. ring = &adev->gfx.compute_ring[ring_id]; ^ >> drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c:4400:7: warning: Shifting signed 32-bit value by 31 bits is undefined behaviour [shiftTooManyBitsSigned] PACKET3_MAP_QUEUES_ME(ring->me == 1 ? 0 : 1)); /* doorbell */ ^ >> drivers/gpu/drm/amd/amdgpu/uvd_v7_0.c:490:9: warning: Identical condition 'r', second condition is always false [identicalConditionAfterEarlyExit] return r; ^ drivers/gpu/drm/amd/amdgpu/uvd_v7_0.c:487:6: note: first condition if (r) ^ drivers/gpu/drm/amd/amdgpu/uvd_v7_0.c:490:9: note: second condition return r; ^ >> drivers/gpu/drm/amd/amdgpu/vce_v4_0.c:493:9: warning: Identical condition 'r', second condition is always false [identicalConditionAfterEarlyExit] return r; ^ drivers/gpu/drm/amd/amdgpu/vce_v4_0.c:490:6: note: first condition if (r) ^ drivers/gpu/drm/amd/amdgpu/vce_v4_0.c:493:9: note: second condition return r; ^ >> drivers/gpu/drm/amd/amdgpu/vce_v4_0.c:446:12: warning: Local variable size shadows outer variable [shadowVar] unsigned size = amdgpu_bo_size(adev->vce.vcpu_bo); ^ drivers/gpu/drm/amd/amdgpu/vce_v4_0.c:429:11: note: Shadowed declaration unsigned size; ^ drivers/gpu/drm/amd/amdgpu/vce_v4_0.c:446:12: note: Shadow variable unsigned size = amdgpu_bo_size(adev->vce.vcpu_bo); ^ vim +1186 drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c 1177 1178 static int amdgpu_pmops_freeze(struct device *dev) 1179 { 1180 struct drm_device *drm_dev = dev_get_drvdata(dev); 1181 struct amdgpu_device *adev = drm_dev->dev_private; 1182 int r; 1183 1184 adev->in_hibernate = true; 1185 r = amdgpu_device_suspend(drm_dev, true); > 1186 adev->in_hibernate = false; 1187 if (r) 1188 return r; 1189 return amdgpu_asic_reset(adev); 1190 } 1191 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com> To: kbuild-all@lists.01.org Subject: drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c:1186:21: warning: Variable 'adev->in_hibernate' is reassigned a value before the old one has been used. Date: Fri, 31 Jul 2020 09:48:23 +0800 [thread overview] Message-ID: <202007310904.RMLh8Rsr%lkp@intel.com> (raw) [-- Attachment #1: Type: text/plain, Size: 21733 bytes --] tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: e2c46b5762c616c249201688d3b9846627f78d2c commit: b2b6290a23986a5c88384887b8a589a3c4ebe292 drm/amdgpu: enable hibernate support on Navi1X date: 3 months ago compiler: arc-elf-gcc (GCC) 9.3.0 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp@intel.com> cppcheck warnings: (new ones prefixed by >>) In file included from drivers/gpu/drm/amd/amdgpu/../powerplay/navi10_ppt.c: >> drivers/gpu/drm/amd/powerplay/navi10_ppt.c:728:9: warning: Identical condition 'ret', second condition is always false [identicalConditionAfterEarlyExit] return ret; ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:719:6: note: first condition if (ret) ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:728:9: note: second condition return ret; ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1006:9: warning: Identical condition 'ret', second condition is always false [identicalConditionAfterEarlyExit] return ret; ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1001:6: note: first condition if (ret) ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1006:9: note: second condition return ret; ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1208:9: warning: Identical condition 'ret', second condition is always false [identicalConditionAfterEarlyExit] return ret; ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1203:6: note: first condition if (ret) ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1208:9: note: second condition return ret; ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1226:9: warning: Identical condition 'ret', second condition is always false [identicalConditionAfterEarlyExit] return ret; ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1220:6: note: first condition if (ret) ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1226:9: note: second condition return ret; ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1695:9: warning: Identical condition 'ret', second condition is always false [identicalConditionAfterEarlyExit] return ret; ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1692:6: note: first condition if (ret) ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1695:9: note: second condition return ret; ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1763:9: warning: Identical condition 'ret', second condition is always false [identicalConditionAfterEarlyExit] return ret; ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1760:6: note: first condition if (ret) ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1763:9: note: second condition return ret; ^ >> drivers/gpu/drm/amd/powerplay/navi10_ppt.c:887:12: warning: %u in format string (no. 3) requires 'unsigned int' but the argument type is 'signed int'. [invalidPrintfArgType_uint] size += sprintf(buf + size, "%d: %uMHz @ %umVn", i, curve_settings[0], curve_settings[1] / NAVI10_VOLTAGE_SCALE); ^ >> drivers/gpu/drm/amd/powerplay/navi10_ppt.c:918:12: warning: %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint] size += sprintf(buf + size, "VDDC_CURVE_VOLT[0]: %7dmV %11dmVn", ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:918:12: warning: %d in format string (no. 2) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint] size += sprintf(buf + size, "VDDC_CURVE_VOLT[0]: %7dmV %11dmVn", ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:926:12: warning: %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint] size += sprintf(buf + size, "VDDC_CURVE_VOLT[1]: %7dmV %11dmVn", ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:926:12: warning: %d in format string (no. 2) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint] size += sprintf(buf + size, "VDDC_CURVE_VOLT[1]: %7dmV %11dmVn", ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:934:12: warning: %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint] size += sprintf(buf + size, "VDDC_CURVE_VOLT[2]: %7dmV %11dmVn", ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:934:12: warning: %d in format string (no. 2) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint] size += sprintf(buf + size, "VDDC_CURVE_VOLT[2]: %7dmV %11dmVn", ^ drivers/gpu/drm/amd/powerplay/navi10_ppt.c:1277:11: warning: %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint] size += sprintf(buf + size, "%2d %14s%s:n", ^ >> drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c:811:6: warning: Variable 'res' is reassigned a value before the old one has been used. 'break;' missing? [redundantAssignInSwitch] res = 0; ^ drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c:768:0: note: Variable 'res' is reassigned a value before the old one has been used. 'break;' missing? int res = -ENODEV; ^ drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c:811:6: note: Variable 'res' is reassigned a value before the old one has been used. 'break;' missing? res = 0; ^ >> drivers/gpu/drm/amd/display/dc/dce110/dce110_resource.c:1247:33: warning: Variable 'ctx->dc->caps.max_slave_planes' is reassigned a value before the old one has been used. [redundantAssignment] ctx->dc->caps.max_slave_planes = 1; ^ drivers/gpu/drm/amd/display/dc/dce110/dce110_resource.c:1246:33: note: Variable 'ctx->dc->caps.max_slave_planes' is reassigned a value before the old one has been used. ctx->dc->caps.max_slave_planes = 1; ^ drivers/gpu/drm/amd/display/dc/dce110/dce110_resource.c:1247:33: note: Variable 'ctx->dc->caps.max_slave_planes' is reassigned a value before the old one has been used. ctx->dc->caps.max_slave_planes = 1; ^ -- In file included from drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_resource.c: >> drivers/gpu/drm/amd/display/dc/core/dc_resource.c:2355:10: warning: Variable 'support' is reassigned a value before the old one has been used. 'break;' missing? [redundantAssignInSwitch] support = stream->content_support; ^ drivers/gpu/drm/amd/display/dc/core/dc_resource.c:2251:0: note: Variable 'support' is reassigned a value before the old one has been used. 'break;' missing? union display_content_support support = {0}; ^ drivers/gpu/drm/amd/display/dc/core/dc_resource.c:2355:10: note: Variable 'support' is reassigned a value before the old one has been used. 'break;' missing? support = stream->content_support; ^ >> drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:1897:15: warning: Variable 'max_link_cap' is reassigned a value before the old one has been used. [redundantAssignment] max_link_cap = get_max_link_cap(link); ^ drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:1877:0: note: Variable 'max_link_cap' is reassigned a value before the old one has been used. struct dc_link_settings max_link_cap = {0}; ^ drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:1897:15: note: Variable 'max_link_cap' is reassigned a value before the old one has been used. max_link_cap = get_max_link_cap(link); ^ >> drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:1919:24: warning: Variable 'initial_link_settings' is reassigned a value before the old one has been used. [redundantAssignment] initial_link_settings = get_common_supported_link_settings( ^ drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:1880:0: note: Variable 'initial_link_settings' is reassigned a value before the old one has been used. struct dc_link_settings initial_link_settings = {0}; ^ drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:1919:24: note: Variable 'initial_link_settings' is reassigned a value before the old one has been used. initial_link_settings = get_common_supported_link_settings( ^ drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:2030:15: warning: Variable 'max_link_cap' is reassigned a value before the old one has been used. [redundantAssignment] max_link_cap = get_max_link_cap(link); ^ drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:2028:0: note: Variable 'max_link_cap' is reassigned a value before the old one has been used. struct dc_link_settings max_link_cap = {0}; ^ drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:2030:15: note: Variable 'max_link_cap' is reassigned a value before the old one has been used. max_link_cap = get_max_link_cap(link); ^ >> drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:3491:10: warning: Variable 'status' is reassigned a value before the old one has been used. [redundantAssignment] status = core_link_read_dpcd( ^ drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:3486:10: note: Variable 'status' is reassigned a value before the old one has been used. status = core_link_read_dpcd( ^ drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:3491:10: note: Variable 'status' is reassigned a value before the old one has been used. status = core_link_read_dpcd( ^ drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:3496:10: warning: Variable 'status' is reassigned a value before the old one has been used. [redundantAssignment] status = core_link_read_dpcd( ^ drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:3491:10: note: Variable 'status' is reassigned a value before the old one has been used. status = core_link_read_dpcd( ^ drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:3496:10: note: Variable 'status' is reassigned a value before the old one has been used. status = core_link_read_dpcd( ^ >> drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c:1186:21: warning: Variable 'adev->in_hibernate' is reassigned a value before the old one has been used. [redundantAssignment] adev->in_hibernate = false; ^ drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c:1184:21: note: Variable 'adev->in_hibernate' is reassigned a value before the old one has been used. adev->in_hibernate = true; ^ drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c:1186:21: note: Variable 'adev->in_hibernate' is reassigned a value before the old one has been used. adev->in_hibernate = false; ^ >> drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c:180:2: warning: Address of local auto-variable assigned to a function parameter. [autoVariables] *f = &fence->base; ^ >> drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c:524:17: warning: Boolean result is used in bitwise operation. Clarify expression with parentheses. [clarifyCondition] if (!(!!enable ^ !!amdgpu_ras_is_feature_enabled(adev, head))) ^ drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c:573:17: warning: Boolean result is used in bitwise operation. Clarify expression with parentheses. [clarifyCondition] if (!(!!enable ^ !!amdgpu_ras_is_feature_enabled(adev, head))) ^ >> drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c:4589:9: warning: Identical condition 'r', second condition is always false [identicalConditionAfterEarlyExit] return r; ^ drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c:4586:6: note: first condition if (r) ^ drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c:4589:9: note: second condition return r; ^ >> drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c:1113:9: warning: Identical condition 'r', second condition is always false [identicalConditionAfterEarlyExit] return r; ^ drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c:1110:6: note: first condition if (r) ^ drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c:1113:9: note: second condition return r; ^ >> drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c:1313:40: warning: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] info->prot_valid = protections & 0x7 ? true : false; ^ drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c:1314:39: warning: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] info->prot_read = protections & 0x8 ? true : false; ^ drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c:1315:41: warning: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] info->prot_write = protections & 0x10 ? true : false; ^ drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c:1316:40: warning: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] info->prot_exec = protections & 0x20 ? true : false; ^ >> drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c:1246:9: warning: Identical condition 'r', second condition is always false [identicalConditionAfterEarlyExit] return r; ^ drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c:1243:6: note: first condition if (r) ^ drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c:1246:9: note: second condition return r; ^ >> drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c:1497:40: warning: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] info->prot_valid = protections & 0x7 ? true : false; ^ drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c:1498:39: warning: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] info->prot_read = protections & 0x8 ? true : false; ^ drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c:1499:41: warning: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] info->prot_write = protections & 0x10 ? true : false; ^ drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c:1500:40: warning: Clarify calculation precedence for '&' and '?'. [clarifyCalculation] info->prot_exec = protections & 0x20 ? true : false; ^ >> drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:137:2: warning: Local variable tmp shadows outer variable [shadowVar] WREG32_SOC15_RLC(GC, 0, mmMC_VM_MX_L1_TLB_CNTL, tmp); ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:121:11: note: Shadowed declaration uint32_t tmp; ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:137:2: note: Shadow variable WREG32_SOC15_RLC(GC, 0, mmMC_VM_MX_L1_TLB_CNTL, tmp); ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:154:2: warning: Local variable tmp shadows outer variable [shadowVar] WREG32_SOC15_RLC(GC, 0, mmVM_L2_CNTL, tmp); ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:142:11: note: Shadowed declaration uint32_t tmp; ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:154:2: note: Shadow variable WREG32_SOC15_RLC(GC, 0, mmVM_L2_CNTL, tmp); ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:159:2: warning: Local variable tmp shadows outer variable [shadowVar] WREG32_SOC15_RLC(GC, 0, mmVM_L2_CNTL2, tmp); ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:142:11: note: Shadowed declaration uint32_t tmp; ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:159:2: note: Shadow variable WREG32_SOC15_RLC(GC, 0, mmVM_L2_CNTL2, tmp); ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:171:2: warning: Local variable tmp shadows outer variable [shadowVar] WREG32_SOC15_RLC(GC, 0, mmVM_L2_CNTL3, tmp); ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:142:11: note: Shadowed declaration uint32_t tmp; ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:171:2: note: Shadow variable WREG32_SOC15_RLC(GC, 0, mmVM_L2_CNTL3, tmp); ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:176:2: warning: Local variable tmp shadows outer variable [shadowVar] WREG32_SOC15_RLC(GC, 0, mmVM_L2_CNTL4, tmp); ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:142:11: note: Shadowed declaration uint32_t tmp; ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:176:2: note: Shadow variable WREG32_SOC15_RLC(GC, 0, mmVM_L2_CNTL4, tmp); ^ >> drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:316:2: warning: Local variable i shadows outer variable [shadowVar] WREG32_SOC15_RLC(GC, 0, mmMC_VM_MX_L1_TLB_CNTL, tmp); ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:303:6: note: Shadowed declaration u32 i; ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:316:2: note: Shadow variable WREG32_SOC15_RLC(GC, 0, mmMC_VM_MX_L1_TLB_CNTL, tmp); ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:316:2: warning: Local variable tmp shadows outer variable [shadowVar] WREG32_SOC15_RLC(GC, 0, mmMC_VM_MX_L1_TLB_CNTL, tmp); ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:302:6: note: Shadowed declaration u32 tmp; ^ drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c:316:2: note: Shadow variable WREG32_SOC15_RLC(GC, 0, mmMC_VM_MX_L1_TLB_CNTL, tmp); ^ >> drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c:1180:4: warning: Variable 'r' is reassigned a value before the old one has been used. 'break;' missing? [redundantAssignInSwitch] r = amdgpu_irq_add_id(adev, SOC15_IH_CLIENTID_VMC, VMC_1_0__SRCID__VM_FAULT, ^ drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c:1112:4: note: Variable 'r' is reassigned a value before the old one has been used. 'break;' missing? r = amdgpu_atomfirmware_get_vram_info(adev, ^ drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c:1180:4: note: Variable 'r' is reassigned a value before the old one has been used. 'break;' missing? r = amdgpu_irq_add_id(adev, SOC15_IH_CLIENTID_VMC, VMC_1_0__SRCID__VM_FAULT, ^ >> drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c:1117:7: warning: Variable 'err' is reassigned a value before the old one has been used. [redundantAssignment] err = request_firmware(&adev->gfx.mec_fw, fw_name, adev->dev); ^ drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c:1070:6: note: Variable 'err' is reassigned a value before the old one has been used. err = amdgpu_ucode_validate(adev->gfx.rlc_fw); ^ drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c:1117:7: note: Variable 'err' is reassigned a value before the old one has been used. err = request_firmware(&adev->gfx.mec_fw, fw_name, adev->dev); ^ >> drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c:1896:7: warning: Variable 'ring' is reassigned a value before the old one has been used. [redundantAssignment] ring = &adev->gfx.compute_ring[ring_id]; ^ drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c:1894:0: note: Variable 'ring' is reassigned a value before the old one has been used. struct amdgpu_ring *ring = &adev->gfx.compute_ring[ring_id]; ^ drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c:1896:7: note: Variable 'ring' is reassigned a value before the old one has been used. ring = &adev->gfx.compute_ring[ring_id]; ^ >> drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c:4400:7: warning: Shifting signed 32-bit value by 31 bits is undefined behaviour [shiftTooManyBitsSigned] PACKET3_MAP_QUEUES_ME(ring->me == 1 ? 0 : 1)); /* doorbell */ ^ >> drivers/gpu/drm/amd/amdgpu/uvd_v7_0.c:490:9: warning: Identical condition 'r', second condition is always false [identicalConditionAfterEarlyExit] return r; ^ drivers/gpu/drm/amd/amdgpu/uvd_v7_0.c:487:6: note: first condition if (r) ^ drivers/gpu/drm/amd/amdgpu/uvd_v7_0.c:490:9: note: second condition return r; ^ >> drivers/gpu/drm/amd/amdgpu/vce_v4_0.c:493:9: warning: Identical condition 'r', second condition is always false [identicalConditionAfterEarlyExit] return r; ^ drivers/gpu/drm/amd/amdgpu/vce_v4_0.c:490:6: note: first condition if (r) ^ drivers/gpu/drm/amd/amdgpu/vce_v4_0.c:493:9: note: second condition return r; ^ >> drivers/gpu/drm/amd/amdgpu/vce_v4_0.c:446:12: warning: Local variable size shadows outer variable [shadowVar] unsigned size = amdgpu_bo_size(adev->vce.vcpu_bo); ^ drivers/gpu/drm/amd/amdgpu/vce_v4_0.c:429:11: note: Shadowed declaration unsigned size; ^ drivers/gpu/drm/amd/amdgpu/vce_v4_0.c:446:12: note: Shadow variable unsigned size = amdgpu_bo_size(adev->vce.vcpu_bo); ^ vim +1186 drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c 1177 1178 static int amdgpu_pmops_freeze(struct device *dev) 1179 { 1180 struct drm_device *drm_dev = dev_get_drvdata(dev); 1181 struct amdgpu_device *adev = drm_dev->dev_private; 1182 int r; 1183 1184 adev->in_hibernate = true; 1185 r = amdgpu_device_suspend(drm_dev, true); > 1186 adev->in_hibernate = false; 1187 if (r) 1188 return r; 1189 return amdgpu_asic_reset(adev); 1190 } 1191 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
next reply other threads:[~2020-07-31 1:49 UTC|newest] Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-07-31 1:48 kernel test robot [this message] 2020-07-31 1:48 ` drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c:1186:21: warning: Variable 'adev->in_hibernate' is reassigned a value before the old one has been used kernel test robot
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=202007310904.RMLh8Rsr%lkp@intel.com \ --to=lkp@intel.com \ --cc=alexander.deucher@amd.com \ --cc=evan.quan@amd.com \ --cc=kbuild-all@lists.01.org \ --cc=linux-kernel@vger.kernel.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.