[bug report] drm/amdgpu/virt: implement VI virt operation interfaces

Dan Carpenter dan.carpenter at oracle.com
Mon Feb 6 11:18:38 UTC 2017


Hello Xiangliang Yu,

The patch ab71ac56f6d8: "drm/amdgpu/virt: implement VI virt operation
interfaces" from Jan 12, 2017, leads to the following static checker
warning:

	drivers/gpu/drm/amd/amdgpu/mxgpu_vi.c:310 xgpu_vi_init_golden_registers()
	warn: string literals are always true.

drivers/gpu/drm/amd/amdgpu/mxgpu_vi.c
   294                  break;
   295          case CHIP_TONGA:
   296                  amdgpu_program_register_sequence(adev,
   297                                                   xgpu_tonga_mgcg_cgcg_init,
   298                                                   (const u32)ARRAY_SIZE(
   299                                                   xgpu_tonga_mgcg_cgcg_init));
   300                  amdgpu_program_register_sequence(adev,
   301                                                   xgpu_tonga_golden_settings_a11,
   302                                                   (const u32)ARRAY_SIZE(
   303                                                   xgpu_tonga_golden_settings_a11));
   304                  amdgpu_program_register_sequence(adev,
   305                                                   xgpu_tonga_golden_common_all,
   306                                                   (const u32)ARRAY_SIZE(
   307                                                   xgpu_tonga_golden_common_all));
   308                  break;
   309          default:
   310                  BUG_ON("Doesn't support chip type.\n");

BUG_ON() takes a condition argument, not a format string.

   311                  break;
   312          }
   313  }

regards,
dan carpenter


More information about the amd-gfx mailing list