From: Mario Limonciello <mario.limoncie...@amd.com> Messages emitted from amdgpu_ucode are currently with legacy DRM macros. These don't show which device they are using. To make messages clearer in multi-GPU systems adjust to drm_*() macros.
Signed-off-by: Mario Limonciello <mario.limoncie...@amd.com> --- drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c | 456 ++++++++++++---------- 1 file changed, 245 insertions(+), 211 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c index a28cd5e962cce..971f28d2c31f7 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c @@ -33,17 +33,25 @@ static void amdgpu_ucode_print_common_hdr(struct amdgpu_device *adev, const struct common_firmware_header *hdr) { - DRM_DEBUG("size_bytes: %u\n", le32_to_cpu(hdr->size_bytes)); - DRM_DEBUG("header_size_bytes: %u\n", le32_to_cpu(hdr->header_size_bytes)); - DRM_DEBUG("header_version_major: %u\n", le16_to_cpu(hdr->header_version_major)); - DRM_DEBUG("header_version_minor: %u\n", le16_to_cpu(hdr->header_version_minor)); - DRM_DEBUG("ip_version_major: %u\n", le16_to_cpu(hdr->ip_version_major)); - DRM_DEBUG("ip_version_minor: %u\n", le16_to_cpu(hdr->ip_version_minor)); - DRM_DEBUG("ucode_version: 0x%08x\n", le32_to_cpu(hdr->ucode_version)); - DRM_DEBUG("ucode_size_bytes: %u\n", le32_to_cpu(hdr->ucode_size_bytes)); - DRM_DEBUG("ucode_array_offset_bytes: %u\n", - le32_to_cpu(hdr->ucode_array_offset_bytes)); - DRM_DEBUG("crc32: 0x%08x\n", le32_to_cpu(hdr->crc32)); + drm_dbg(adev_to_drm(adev), + "size_bytes: %u\n", le32_to_cpu(hdr->size_bytes)); + drm_dbg(adev_to_drm(adev), + "header_size_bytes: %u\n", le32_to_cpu(hdr->header_size_bytes)); + drm_dbg(adev_to_drm(adev), + "header_version_major: %u\n", le16_to_cpu(hdr->header_version_major)); + drm_dbg(adev_to_drm(adev), + "header_version_minor: %u\n", le16_to_cpu(hdr->header_version_minor)); + drm_dbg(adev_to_drm(adev), + "ip_version_major: %u\n", le16_to_cpu(hdr->ip_version_major)); + drm_dbg(adev_to_drm(adev), + "ip_version_minor: %u\n", le16_to_cpu(hdr->ip_version_minor)); + drm_dbg(adev_to_drm(adev), + "ucode_version: 0x%08x\n", le32_to_cpu(hdr->ucode_version)); + drm_dbg(adev_to_drm(adev), + "ucode_size_bytes: %u\n", le32_to_cpu(hdr->ucode_size_bytes)); + drm_dbg(adev_to_drm(adev), + "ucode_array_offset_bytes: %u\n", le32_to_cpu(hdr->ucode_array_offset_bytes)); + drm_dbg(adev_to_drm(adev), "crc32: 0x%08x\n", le32_to_cpu(hdr->crc32)); } void amdgpu_ucode_print_mc_hdr(struct amdgpu_device *adev, @@ -52,19 +60,20 @@ void amdgpu_ucode_print_mc_hdr(struct amdgpu_device *adev, uint16_t version_major = le16_to_cpu(hdr->header_version_major); uint16_t version_minor = le16_to_cpu(hdr->header_version_minor); - DRM_DEBUG("MC\n"); - amdgpu_ucode_print_common_hdr(hdr); + drm_dbg(adev_to_drm(adev), "MC\n"); + amdgpu_ucode_print_common_hdr(adev, hdr); if (version_major == 1) { const struct mc_firmware_header_v1_0 *mc_hdr = container_of(hdr, struct mc_firmware_header_v1_0, header); - DRM_DEBUG("io_debug_size_bytes: %u\n", + drm_dbg(adev_to_drm(adev), "io_debug_size_bytes: %u\n", le32_to_cpu(mc_hdr->io_debug_size_bytes)); - DRM_DEBUG("io_debug_array_offset_bytes: %u\n", + drm_dbg(adev_to_drm(adev), "io_debug_array_offset_bytes: %u\n", le32_to_cpu(mc_hdr->io_debug_array_offset_bytes)); } else { - DRM_ERROR("Unknown MC ucode version: %u.%u\n", version_major, version_minor); + drm_err(adev_to_drm(adev), + "Unknown MC ucode version: %u.%u\n", version_major, version_minor); } } @@ -77,30 +86,41 @@ void amdgpu_ucode_print_smc_hdr(struct amdgpu_device *adev, const struct smc_firmware_header_v2_0 *v2_0_hdr; const struct smc_firmware_header_v2_1 *v2_1_hdr; - DRM_DEBUG("SMC\n"); - amdgpu_ucode_print_common_hdr(hdr); + drm_dbg(adev_to_drm(adev), "SMC\n"); + amdgpu_ucode_print_common_hdr(adev, hdr); if (version_major == 1) { v1_0_hdr = container_of(hdr, struct smc_firmware_header_v1_0, header); - DRM_DEBUG("ucode_start_addr: %u\n", le32_to_cpu(v1_0_hdr->ucode_start_addr)); + drm_dbg(adev_to_drm(adev), + "ucode_start_addr: %u\n", + le32_to_cpu(v1_0_hdr->ucode_start_addr)); } else if (version_major == 2) { switch (version_minor) { case 0: v2_0_hdr = container_of(hdr, struct smc_firmware_header_v2_0, v1_0.header); - DRM_DEBUG("ppt_offset_bytes: %u\n", le32_to_cpu(v2_0_hdr->ppt_offset_bytes)); - DRM_DEBUG("ppt_size_bytes: %u\n", le32_to_cpu(v2_0_hdr->ppt_size_bytes)); + drm_dbg(adev_to_drm(adev), + "ppt_offset_bytes: %u\n", + le32_to_cpu(v2_0_hdr->ppt_offset_bytes)); + drm_dbg(adev_to_drm(adev), + "ppt_size_bytes: %u\n", + le32_to_cpu(v2_0_hdr->ppt_size_bytes)); break; case 1: v2_1_hdr = container_of(hdr, struct smc_firmware_header_v2_1, v1_0.header); - DRM_DEBUG("pptable_count: %u\n", le32_to_cpu(v2_1_hdr->pptable_count)); - DRM_DEBUG("pptable_entry_offset: %u\n", le32_to_cpu(v2_1_hdr->pptable_entry_offset)); + drm_dbg(adev_to_drm(adev), + "pptable_count: %u\n", + le32_to_cpu(v2_1_hdr->pptable_count)); + drm_dbg(adev_to_drm(adev), + "pptable_entry_offset: %u\n", + le32_to_cpu(v2_1_hdr->pptable_entry_offset)); break; default: break; } } else { - DRM_ERROR("Unknown SMC ucode version: %u.%u\n", version_major, version_minor); + drm_err(adev_to_drm(adev), + "Unknown SMC ucode version: %u.%u\n", version_major, version_minor); } } @@ -110,25 +130,26 @@ void amdgpu_ucode_print_gfx_hdr(struct amdgpu_device *adev, uint16_t version_major = le16_to_cpu(hdr->header_version_major); uint16_t version_minor = le16_to_cpu(hdr->header_version_minor); - DRM_DEBUG("GFX\n"); - amdgpu_ucode_print_common_hdr(hdr); + drm_dbg(adev_to_drm(adev), "GFX\n"); + amdgpu_ucode_print_common_hdr(adev, hdr); if (version_major == 1) { const struct gfx_firmware_header_v1_0 *gfx_hdr = container_of(hdr, struct gfx_firmware_header_v1_0, header); - DRM_DEBUG("ucode_feature_version: %u\n", + drm_dbg(adev_to_drm(adev), "ucode_feature_version: %u\n", le32_to_cpu(gfx_hdr->ucode_feature_version)); - DRM_DEBUG("jt_offset: %u\n", le32_to_cpu(gfx_hdr->jt_offset)); - DRM_DEBUG("jt_size: %u\n", le32_to_cpu(gfx_hdr->jt_size)); + drm_dbg(adev_to_drm(adev), "jt_offset: %u\n", le32_to_cpu(gfx_hdr->jt_offset)); + drm_dbg(adev_to_drm(adev), "jt_size: %u\n", le32_to_cpu(gfx_hdr->jt_size)); } else if (version_major == 2) { const struct gfx_firmware_header_v2_0 *gfx_hdr = container_of(hdr, struct gfx_firmware_header_v2_0, header); - DRM_DEBUG("ucode_feature_version: %u\n", - le32_to_cpu(gfx_hdr->ucode_feature_version)); + drm_dbg(adev_to_drm(adev), + "ucode_feature_version: %u\n", le32_to_cpu(gfx_hdr->ucode_feature_version)); } else { - DRM_ERROR("Unknown GFX ucode version: %u.%u\n", version_major, version_minor); + drm_err(adev_to_drm(adev), + "Unknown GFX ucode version: %u.%u\n", version_major, version_minor); } } @@ -138,22 +159,22 @@ void amdgpu_ucode_print_rlc_hdr(struct amdgpu_device *adev, uint16_t version_major = le16_to_cpu(hdr->header_version_major); uint16_t version_minor = le16_to_cpu(hdr->header_version_minor); - DRM_DEBUG("RLC\n"); - amdgpu_ucode_print_common_hdr(hdr); + drm_dbg(adev_to_drm(adev), "RLC\n"); + amdgpu_ucode_print_common_hdr(adev, hdr); if (version_major == 1) { const struct rlc_firmware_header_v1_0 *rlc_hdr = container_of(hdr, struct rlc_firmware_header_v1_0, header); - DRM_DEBUG("ucode_feature_version: %u\n", + drm_dbg(adev_to_drm(adev), "ucode_feature_version: %u\n", le32_to_cpu(rlc_hdr->ucode_feature_version)); - DRM_DEBUG("save_and_restore_offset: %u\n", + drm_dbg(adev_to_drm(adev), "save_and_restore_offset: %u\n", le32_to_cpu(rlc_hdr->save_and_restore_offset)); - DRM_DEBUG("clear_state_descriptor_offset: %u\n", + drm_dbg(adev_to_drm(adev), "clear_state_descriptor_offset: %u\n", le32_to_cpu(rlc_hdr->clear_state_descriptor_offset)); - DRM_DEBUG("avail_scratch_ram_locations: %u\n", + drm_dbg(adev_to_drm(adev), "avail_scratch_ram_locations: %u\n", le32_to_cpu(rlc_hdr->avail_scratch_ram_locations)); - DRM_DEBUG("master_pkt_description_offset: %u\n", + drm_dbg(adev_to_drm(adev), "master_pkt_description_offset: %u\n", le32_to_cpu(rlc_hdr->master_pkt_description_offset)); } else if (version_major == 2) { const struct rlc_firmware_header_v2_0 *rlc_hdr = @@ -170,129 +191,133 @@ void amdgpu_ucode_print_rlc_hdr(struct amdgpu_device *adev, switch (version_minor) { case 0: /* rlc_hdr v2_0 */ - DRM_DEBUG("ucode_feature_version: %u\n", + drm_dbg(adev_to_drm(adev), "ucode_feature_version: %u\n", le32_to_cpu(rlc_hdr->ucode_feature_version)); - DRM_DEBUG("jt_offset: %u\n", le32_to_cpu(rlc_hdr->jt_offset)); - DRM_DEBUG("jt_size: %u\n", le32_to_cpu(rlc_hdr->jt_size)); - DRM_DEBUG("save_and_restore_offset: %u\n", - le32_to_cpu(rlc_hdr->save_and_restore_offset)); - DRM_DEBUG("clear_state_descriptor_offset: %u\n", - le32_to_cpu(rlc_hdr->clear_state_descriptor_offset)); - DRM_DEBUG("avail_scratch_ram_locations: %u\n", - le32_to_cpu(rlc_hdr->avail_scratch_ram_locations)); - DRM_DEBUG("reg_restore_list_size: %u\n", - le32_to_cpu(rlc_hdr->reg_restore_list_size)); - DRM_DEBUG("reg_list_format_start: %u\n", - le32_to_cpu(rlc_hdr->reg_list_format_start)); - DRM_DEBUG("reg_list_format_separate_start: %u\n", - le32_to_cpu(rlc_hdr->reg_list_format_separate_start)); - DRM_DEBUG("starting_offsets_start: %u\n", - le32_to_cpu(rlc_hdr->starting_offsets_start)); - DRM_DEBUG("reg_list_format_size_bytes: %u\n", - le32_to_cpu(rlc_hdr->reg_list_format_size_bytes)); - DRM_DEBUG("reg_list_format_array_offset_bytes: %u\n", - le32_to_cpu(rlc_hdr->reg_list_format_array_offset_bytes)); - DRM_DEBUG("reg_list_size_bytes: %u\n", - le32_to_cpu(rlc_hdr->reg_list_size_bytes)); - DRM_DEBUG("reg_list_array_offset_bytes: %u\n", - le32_to_cpu(rlc_hdr->reg_list_array_offset_bytes)); - DRM_DEBUG("reg_list_format_separate_size_bytes: %u\n", - le32_to_cpu(rlc_hdr->reg_list_format_separate_size_bytes)); - DRM_DEBUG("reg_list_format_separate_array_offset_bytes: %u\n", - le32_to_cpu(rlc_hdr->reg_list_format_separate_array_offset_bytes)); - DRM_DEBUG("reg_list_separate_size_bytes: %u\n", - le32_to_cpu(rlc_hdr->reg_list_separate_size_bytes)); - DRM_DEBUG("reg_list_separate_array_offset_bytes: %u\n", - le32_to_cpu(rlc_hdr->reg_list_separate_array_offset_bytes)); + drm_dbg(adev_to_drm(adev), + "jt_offset: %u\n", le32_to_cpu(rlc_hdr->jt_offset)); + drm_dbg(adev_to_drm(adev), + "jt_size: %u\n", le32_to_cpu(rlc_hdr->jt_size)); + drm_dbg(adev_to_drm(adev), + "save_and_restore_offset: %u\n", + le32_to_cpu(rlc_hdr->save_and_restore_offset)); + drm_dbg(adev_to_drm(adev), "clear_state_descriptor_offset: %u\n", + le32_to_cpu(rlc_hdr->clear_state_descriptor_offset)); + drm_dbg(adev_to_drm(adev), "avail_scratch_ram_locations: %u\n", + le32_to_cpu(rlc_hdr->avail_scratch_ram_locations)); + drm_dbg(adev_to_drm(adev), "reg_restore_list_size: %u\n", + le32_to_cpu(rlc_hdr->reg_restore_list_size)); + drm_dbg(adev_to_drm(adev), "reg_list_format_start: %u\n", + le32_to_cpu(rlc_hdr->reg_list_format_start)); + drm_dbg(adev_to_drm(adev), "reg_list_format_separate_start: %u\n", + le32_to_cpu(rlc_hdr->reg_list_format_separate_start)); + drm_dbg(adev_to_drm(adev), "starting_offsets_start: %u\n", + le32_to_cpu(rlc_hdr->starting_offsets_start)); + drm_dbg(adev_to_drm(adev), "reg_list_format_size_bytes: %u\n", + le32_to_cpu(rlc_hdr->reg_list_format_size_bytes)); + drm_dbg(adev_to_drm(adev), "reg_list_format_array_offset_bytes: %u\n", + le32_to_cpu(rlc_hdr->reg_list_format_array_offset_bytes)); + drm_dbg(adev_to_drm(adev), "reg_list_size_bytes: %u\n", + le32_to_cpu(rlc_hdr->reg_list_size_bytes)); + drm_dbg(adev_to_drm(adev), "reg_list_array_offset_bytes: %u\n", + le32_to_cpu(rlc_hdr->reg_list_array_offset_bytes)); + drm_dbg(adev_to_drm(adev), "reg_list_format_separate_size_bytes: %u\n", + le32_to_cpu(rlc_hdr->reg_list_format_separate_size_bytes)); + drm_dbg(adev_to_drm(adev), "reg_list_format_separate_array_offset_bytes: %u\n", + le32_to_cpu(rlc_hdr->reg_list_format_separate_array_offset_bytes)); + drm_dbg(adev_to_drm(adev), "reg_list_separate_size_bytes: %u\n", + le32_to_cpu(rlc_hdr->reg_list_separate_size_bytes)); + drm_dbg(adev_to_drm(adev), "reg_list_separate_array_offset_bytes: %u\n", + le32_to_cpu(rlc_hdr->reg_list_separate_array_offset_bytes)); break; case 1: /* rlc_hdr v2_1 */ - DRM_DEBUG("reg_list_format_direct_reg_list_length: %u\n", - le32_to_cpu(rlc_hdr_v2_1->reg_list_format_direct_reg_list_length)); - DRM_DEBUG("save_restore_list_cntl_ucode_ver: %u\n", - le32_to_cpu(rlc_hdr_v2_1->save_restore_list_cntl_ucode_ver)); - DRM_DEBUG("save_restore_list_cntl_feature_ver: %u\n", - le32_to_cpu(rlc_hdr_v2_1->save_restore_list_cntl_feature_ver)); - DRM_DEBUG("save_restore_list_cntl_size_bytes %u\n", - le32_to_cpu(rlc_hdr_v2_1->save_restore_list_cntl_size_bytes)); - DRM_DEBUG("save_restore_list_cntl_offset_bytes: %u\n", - le32_to_cpu(rlc_hdr_v2_1->save_restore_list_cntl_offset_bytes)); - DRM_DEBUG("save_restore_list_gpm_ucode_ver: %u\n", - le32_to_cpu(rlc_hdr_v2_1->save_restore_list_gpm_ucode_ver)); - DRM_DEBUG("save_restore_list_gpm_feature_ver: %u\n", - le32_to_cpu(rlc_hdr_v2_1->save_restore_list_gpm_feature_ver)); - DRM_DEBUG("save_restore_list_gpm_size_bytes %u\n", - le32_to_cpu(rlc_hdr_v2_1->save_restore_list_gpm_size_bytes)); - DRM_DEBUG("save_restore_list_gpm_offset_bytes: %u\n", - le32_to_cpu(rlc_hdr_v2_1->save_restore_list_gpm_offset_bytes)); - DRM_DEBUG("save_restore_list_srm_ucode_ver: %u\n", - le32_to_cpu(rlc_hdr_v2_1->save_restore_list_srm_ucode_ver)); - DRM_DEBUG("save_restore_list_srm_feature_ver: %u\n", - le32_to_cpu(rlc_hdr_v2_1->save_restore_list_srm_feature_ver)); - DRM_DEBUG("save_restore_list_srm_size_bytes %u\n", - le32_to_cpu(rlc_hdr_v2_1->save_restore_list_srm_size_bytes)); - DRM_DEBUG("save_restore_list_srm_offset_bytes: %u\n", - le32_to_cpu(rlc_hdr_v2_1->save_restore_list_srm_offset_bytes)); + drm_dbg(adev_to_drm(adev), "reg_list_format_direct_reg_list_length: %u\n", + le32_to_cpu(rlc_hdr_v2_1->reg_list_format_direct_reg_list_length)); + drm_dbg(adev_to_drm(adev), "save_restore_list_cntl_ucode_ver: %u\n", + le32_to_cpu(rlc_hdr_v2_1->save_restore_list_cntl_ucode_ver)); + drm_dbg(adev_to_drm(adev), "save_restore_list_cntl_feature_ver: %u\n", + le32_to_cpu(rlc_hdr_v2_1->save_restore_list_cntl_feature_ver)); + drm_dbg(adev_to_drm(adev), "save_restore_list_cntl_size_bytes %u\n", + le32_to_cpu(rlc_hdr_v2_1->save_restore_list_cntl_size_bytes)); + drm_dbg(adev_to_drm(adev), "save_restore_list_cntl_offset_bytes: %u\n", + le32_to_cpu(rlc_hdr_v2_1->save_restore_list_cntl_offset_bytes)); + drm_dbg(adev_to_drm(adev), "save_restore_list_gpm_ucode_ver: %u\n", + le32_to_cpu(rlc_hdr_v2_1->save_restore_list_gpm_ucode_ver)); + drm_dbg(adev_to_drm(adev), "save_restore_list_gpm_feature_ver: %u\n", + le32_to_cpu(rlc_hdr_v2_1->save_restore_list_gpm_feature_ver)); + drm_dbg(adev_to_drm(adev), "save_restore_list_gpm_size_bytes %u\n", + le32_to_cpu(rlc_hdr_v2_1->save_restore_list_gpm_size_bytes)); + drm_dbg(adev_to_drm(adev), "save_restore_list_gpm_offset_bytes: %u\n", + le32_to_cpu(rlc_hdr_v2_1->save_restore_list_gpm_offset_bytes)); + drm_dbg(adev_to_drm(adev), "save_restore_list_srm_ucode_ver: %u\n", + le32_to_cpu(rlc_hdr_v2_1->save_restore_list_srm_ucode_ver)); + drm_dbg(adev_to_drm(adev), "save_restore_list_srm_feature_ver: %u\n", + le32_to_cpu(rlc_hdr_v2_1->save_restore_list_srm_feature_ver)); + drm_dbg(adev_to_drm(adev), "save_restore_list_srm_size_bytes %u\n", + le32_to_cpu(rlc_hdr_v2_1->save_restore_list_srm_size_bytes)); + drm_dbg(adev_to_drm(adev), "save_restore_list_srm_offset_bytes: %u\n", + le32_to_cpu(rlc_hdr_v2_1->save_restore_list_srm_offset_bytes)); break; case 2: /* rlc_hdr v2_2 */ - DRM_DEBUG("rlc_iram_ucode_size_bytes: %u\n", - le32_to_cpu(rlc_hdr_v2_2->rlc_iram_ucode_size_bytes)); - DRM_DEBUG("rlc_iram_ucode_offset_bytes: %u\n", - le32_to_cpu(rlc_hdr_v2_2->rlc_iram_ucode_offset_bytes)); - DRM_DEBUG("rlc_dram_ucode_size_bytes: %u\n", - le32_to_cpu(rlc_hdr_v2_2->rlc_dram_ucode_size_bytes)); - DRM_DEBUG("rlc_dram_ucode_offset_bytes: %u\n", - le32_to_cpu(rlc_hdr_v2_2->rlc_dram_ucode_offset_bytes)); + drm_dbg(adev_to_drm(adev), "rlc_iram_ucode_size_bytes: %u\n", + le32_to_cpu(rlc_hdr_v2_2->rlc_iram_ucode_size_bytes)); + drm_dbg(adev_to_drm(adev), "rlc_iram_ucode_offset_bytes: %u\n", + le32_to_cpu(rlc_hdr_v2_2->rlc_iram_ucode_offset_bytes)); + drm_dbg(adev_to_drm(adev), "rlc_dram_ucode_size_bytes: %u\n", + le32_to_cpu(rlc_hdr_v2_2->rlc_dram_ucode_size_bytes)); + drm_dbg(adev_to_drm(adev), "rlc_dram_ucode_offset_bytes: %u\n", + le32_to_cpu(rlc_hdr_v2_2->rlc_dram_ucode_offset_bytes)); break; case 3: /* rlc_hdr v2_3 */ - DRM_DEBUG("rlcp_ucode_version: %u\n", - le32_to_cpu(rlc_hdr_v2_3->rlcp_ucode_version)); - DRM_DEBUG("rlcp_ucode_feature_version: %u\n", - le32_to_cpu(rlc_hdr_v2_3->rlcp_ucode_feature_version)); - DRM_DEBUG("rlcp_ucode_size_bytes: %u\n", - le32_to_cpu(rlc_hdr_v2_3->rlcp_ucode_size_bytes)); - DRM_DEBUG("rlcp_ucode_offset_bytes: %u\n", - le32_to_cpu(rlc_hdr_v2_3->rlcp_ucode_offset_bytes)); - DRM_DEBUG("rlcv_ucode_version: %u\n", - le32_to_cpu(rlc_hdr_v2_3->rlcv_ucode_version)); - DRM_DEBUG("rlcv_ucode_feature_version: %u\n", - le32_to_cpu(rlc_hdr_v2_3->rlcv_ucode_feature_version)); - DRM_DEBUG("rlcv_ucode_size_bytes: %u\n", - le32_to_cpu(rlc_hdr_v2_3->rlcv_ucode_size_bytes)); - DRM_DEBUG("rlcv_ucode_offset_bytes: %u\n", - le32_to_cpu(rlc_hdr_v2_3->rlcv_ucode_offset_bytes)); + drm_dbg(adev_to_drm(adev), "rlcp_ucode_version: %u\n", + le32_to_cpu(rlc_hdr_v2_3->rlcp_ucode_version)); + drm_dbg(adev_to_drm(adev), "rlcp_ucode_feature_version: %u\n", + le32_to_cpu(rlc_hdr_v2_3->rlcp_ucode_feature_version)); + drm_dbg(adev_to_drm(adev), "rlcp_ucode_size_bytes: %u\n", + le32_to_cpu(rlc_hdr_v2_3->rlcp_ucode_size_bytes)); + drm_dbg(adev_to_drm(adev), "rlcp_ucode_offset_bytes: %u\n", + le32_to_cpu(rlc_hdr_v2_3->rlcp_ucode_offset_bytes)); + drm_dbg(adev_to_drm(adev), "rlcv_ucode_version: %u\n", + le32_to_cpu(rlc_hdr_v2_3->rlcv_ucode_version)); + drm_dbg(adev_to_drm(adev), "rlcv_ucode_feature_version: %u\n", + le32_to_cpu(rlc_hdr_v2_3->rlcv_ucode_feature_version)); + drm_dbg(adev_to_drm(adev), "rlcv_ucode_size_bytes: %u\n", + le32_to_cpu(rlc_hdr_v2_3->rlcv_ucode_size_bytes)); + drm_dbg(adev_to_drm(adev), "rlcv_ucode_offset_bytes: %u\n", + le32_to_cpu(rlc_hdr_v2_3->rlcv_ucode_offset_bytes)); break; case 4: /* rlc_hdr v2_4 */ - DRM_DEBUG("global_tap_delays_ucode_size_bytes :%u\n", - le32_to_cpu(rlc_hdr_v2_4->global_tap_delays_ucode_size_bytes)); - DRM_DEBUG("global_tap_delays_ucode_offset_bytes: %u\n", - le32_to_cpu(rlc_hdr_v2_4->global_tap_delays_ucode_offset_bytes)); - DRM_DEBUG("se0_tap_delays_ucode_size_bytes :%u\n", - le32_to_cpu(rlc_hdr_v2_4->se0_tap_delays_ucode_size_bytes)); - DRM_DEBUG("se0_tap_delays_ucode_offset_bytes: %u\n", - le32_to_cpu(rlc_hdr_v2_4->se0_tap_delays_ucode_offset_bytes)); - DRM_DEBUG("se1_tap_delays_ucode_size_bytes :%u\n", - le32_to_cpu(rlc_hdr_v2_4->se1_tap_delays_ucode_size_bytes)); - DRM_DEBUG("se1_tap_delays_ucode_offset_bytes: %u\n", - le32_to_cpu(rlc_hdr_v2_4->se1_tap_delays_ucode_offset_bytes)); - DRM_DEBUG("se2_tap_delays_ucode_size_bytes :%u\n", - le32_to_cpu(rlc_hdr_v2_4->se2_tap_delays_ucode_size_bytes)); - DRM_DEBUG("se2_tap_delays_ucode_offset_bytes: %u\n", - le32_to_cpu(rlc_hdr_v2_4->se2_tap_delays_ucode_offset_bytes)); - DRM_DEBUG("se3_tap_delays_ucode_size_bytes :%u\n", - le32_to_cpu(rlc_hdr_v2_4->se3_tap_delays_ucode_size_bytes)); - DRM_DEBUG("se3_tap_delays_ucode_offset_bytes: %u\n", - le32_to_cpu(rlc_hdr_v2_4->se3_tap_delays_ucode_offset_bytes)); + drm_dbg(adev_to_drm(adev), "global_tap_delays_ucode_size_bytes :%u\n", + le32_to_cpu(rlc_hdr_v2_4->global_tap_delays_ucode_size_bytes)); + drm_dbg(adev_to_drm(adev), "global_tap_delays_ucode_offset_bytes: %u\n", + le32_to_cpu(rlc_hdr_v2_4->global_tap_delays_ucode_offset_bytes)); + drm_dbg(adev_to_drm(adev), "se0_tap_delays_ucode_size_bytes :%u\n", + le32_to_cpu(rlc_hdr_v2_4->se0_tap_delays_ucode_size_bytes)); + drm_dbg(adev_to_drm(adev), "se0_tap_delays_ucode_offset_bytes: %u\n", + le32_to_cpu(rlc_hdr_v2_4->se0_tap_delays_ucode_offset_bytes)); + drm_dbg(adev_to_drm(adev), "se1_tap_delays_ucode_size_bytes :%u\n", + le32_to_cpu(rlc_hdr_v2_4->se1_tap_delays_ucode_size_bytes)); + drm_dbg(adev_to_drm(adev), "se1_tap_delays_ucode_offset_bytes: %u\n", + le32_to_cpu(rlc_hdr_v2_4->se1_tap_delays_ucode_offset_bytes)); + drm_dbg(adev_to_drm(adev), "se2_tap_delays_ucode_size_bytes :%u\n", + le32_to_cpu(rlc_hdr_v2_4->se2_tap_delays_ucode_size_bytes)); + drm_dbg(adev_to_drm(adev), "se2_tap_delays_ucode_offset_bytes: %u\n", + le32_to_cpu(rlc_hdr_v2_4->se2_tap_delays_ucode_offset_bytes)); + drm_dbg(adev_to_drm(adev), "se3_tap_delays_ucode_size_bytes :%u\n", + le32_to_cpu(rlc_hdr_v2_4->se3_tap_delays_ucode_size_bytes)); + drm_dbg(adev_to_drm(adev), "se3_tap_delays_ucode_offset_bytes: %u\n", + le32_to_cpu(rlc_hdr_v2_4->se3_tap_delays_ucode_offset_bytes)); break; default: - DRM_ERROR("Unknown RLC v2 ucode: v2.%u\n", version_minor); + drm_err(adev_to_drm(adev), "Unknown RLC v2 ucode: v2.%u\n", version_minor); break; } } else { - DRM_ERROR("Unknown RLC ucode version: %u.%u\n", version_major, version_minor); + drm_err(adev_to_drm(adev), + "Unknown RLC ucode version: %u.%u\n", version_major, version_minor); } } @@ -302,43 +327,49 @@ void amdgpu_ucode_print_sdma_hdr(struct amdgpu_device *adev, uint16_t version_major = le16_to_cpu(hdr->header_version_major); uint16_t version_minor = le16_to_cpu(hdr->header_version_minor); - DRM_DEBUG("SDMA\n"); - amdgpu_ucode_print_common_hdr(hdr); + drm_dbg(adev_to_drm(adev), "SDMA\n"); + amdgpu_ucode_print_common_hdr(adev, hdr); if (version_major == 1) { const struct sdma_firmware_header_v1_0 *sdma_hdr = container_of(hdr, struct sdma_firmware_header_v1_0, header); - DRM_DEBUG("ucode_feature_version: %u\n", + drm_dbg(adev_to_drm(adev), "ucode_feature_version: %u\n", le32_to_cpu(sdma_hdr->ucode_feature_version)); - DRM_DEBUG("ucode_change_version: %u\n", + drm_dbg(adev_to_drm(adev), "ucode_change_version: %u\n", le32_to_cpu(sdma_hdr->ucode_change_version)); - DRM_DEBUG("jt_offset: %u\n", le32_to_cpu(sdma_hdr->jt_offset)); - DRM_DEBUG("jt_size: %u\n", le32_to_cpu(sdma_hdr->jt_size)); + drm_dbg(adev_to_drm(adev), "jt_offset: %u\n", le32_to_cpu(sdma_hdr->jt_offset)); + drm_dbg(adev_to_drm(adev), "jt_size: %u\n", le32_to_cpu(sdma_hdr->jt_size)); if (version_minor >= 1) { const struct sdma_firmware_header_v1_1 *sdma_v1_1_hdr = container_of(sdma_hdr, struct sdma_firmware_header_v1_1, v1_0); - DRM_DEBUG("digest_size: %u\n", le32_to_cpu(sdma_v1_1_hdr->digest_size)); + drm_dbg(adev_to_drm(adev), + "digest_size: %u\n", le32_to_cpu(sdma_v1_1_hdr->digest_size)); } } else if (version_major == 2) { const struct sdma_firmware_header_v2_0 *sdma_hdr = container_of(hdr, struct sdma_firmware_header_v2_0, header); - DRM_DEBUG("ucode_feature_version: %u\n", + drm_dbg(adev_to_drm(adev), "ucode_feature_version: %u\n", le32_to_cpu(sdma_hdr->ucode_feature_version)); - DRM_DEBUG("ctx_jt_offset: %u\n", le32_to_cpu(sdma_hdr->ctx_jt_offset)); - DRM_DEBUG("ctx_jt_size: %u\n", le32_to_cpu(sdma_hdr->ctx_jt_size)); - DRM_DEBUG("ctl_ucode_offset: %u\n", le32_to_cpu(sdma_hdr->ctl_ucode_offset)); - DRM_DEBUG("ctl_jt_offset: %u\n", le32_to_cpu(sdma_hdr->ctl_jt_offset)); - DRM_DEBUG("ctl_jt_size: %u\n", le32_to_cpu(sdma_hdr->ctl_jt_size)); + drm_dbg(adev_to_drm(adev), + "ctx_jt_offset: %u\n", le32_to_cpu(sdma_hdr->ctx_jt_offset)); + drm_dbg(adev_to_drm(adev), + "ctx_jt_size: %u\n", le32_to_cpu(sdma_hdr->ctx_jt_size)); + drm_dbg(adev_to_drm(adev), + "ctl_ucode_offset: %u\n", le32_to_cpu(sdma_hdr->ctl_ucode_offset)); + drm_dbg(adev_to_drm(adev), + "ctl_jt_offset: %u\n", le32_to_cpu(sdma_hdr->ctl_jt_offset)); + drm_dbg(adev_to_drm(adev), + "ctl_jt_size: %u\n", le32_to_cpu(sdma_hdr->ctl_jt_size)); } else if (version_major == 3) { const struct sdma_firmware_header_v3_0 *sdma_hdr = container_of(hdr, struct sdma_firmware_header_v3_0, header); - DRM_DEBUG("ucode_reversion: %u\n", + drm_dbg(adev_to_drm(adev), "ucode_reversion: %u\n", le32_to_cpu(sdma_hdr->ucode_feature_version)); } else { - DRM_ERROR("Unknown SDMA ucode version: %u.%u\n", + drm_err(adev_to_drm(adev), "Unknown SDMA ucode version: %u.%u\n", version_major, version_minor); } } @@ -351,43 +382,43 @@ void amdgpu_ucode_print_psp_hdr(struct amdgpu_device *adev, uint32_t fw_index; const struct psp_fw_bin_desc *desc; - DRM_DEBUG("PSP\n"); - amdgpu_ucode_print_common_hdr(hdr); + drm_dbg(adev_to_drm(adev), "PSP\n"); + amdgpu_ucode_print_common_hdr(adev, hdr); if (version_major == 1) { const struct psp_firmware_header_v1_0 *psp_hdr = container_of(hdr, struct psp_firmware_header_v1_0, header); - DRM_DEBUG("ucode_feature_version: %u\n", + drm_dbg(adev_to_drm(adev), "ucode_feature_version: %u\n", le32_to_cpu(psp_hdr->sos.fw_version)); - DRM_DEBUG("sos_offset_bytes: %u\n", + drm_dbg(adev_to_drm(adev), "sos_offset_bytes: %u\n", le32_to_cpu(psp_hdr->sos.offset_bytes)); - DRM_DEBUG("sos_size_bytes: %u\n", + drm_dbg(adev_to_drm(adev), "sos_size_bytes: %u\n", le32_to_cpu(psp_hdr->sos.size_bytes)); if (version_minor == 1) { const struct psp_firmware_header_v1_1 *psp_hdr_v1_1 = container_of(psp_hdr, struct psp_firmware_header_v1_1, v1_0); - DRM_DEBUG("toc_header_version: %u\n", + drm_dbg(adev_to_drm(adev), "toc_header_version: %u\n", le32_to_cpu(psp_hdr_v1_1->toc.fw_version)); - DRM_DEBUG("toc_offset_bytes: %u\n", + drm_dbg(adev_to_drm(adev), "toc_offset_bytes: %u\n", le32_to_cpu(psp_hdr_v1_1->toc.offset_bytes)); - DRM_DEBUG("toc_size_bytes: %u\n", + drm_dbg(adev_to_drm(adev), "toc_size_bytes: %u\n", le32_to_cpu(psp_hdr_v1_1->toc.size_bytes)); - DRM_DEBUG("kdb_header_version: %u\n", + drm_dbg(adev_to_drm(adev), "kdb_header_version: %u\n", le32_to_cpu(psp_hdr_v1_1->kdb.fw_version)); - DRM_DEBUG("kdb_offset_bytes: %u\n", + drm_dbg(adev_to_drm(adev), "kdb_offset_bytes: %u\n", le32_to_cpu(psp_hdr_v1_1->kdb.offset_bytes)); - DRM_DEBUG("kdb_size_bytes: %u\n", + drm_dbg(adev_to_drm(adev), "kdb_size_bytes: %u\n", le32_to_cpu(psp_hdr_v1_1->kdb.size_bytes)); } if (version_minor == 2) { const struct psp_firmware_header_v1_2 *psp_hdr_v1_2 = container_of(psp_hdr, struct psp_firmware_header_v1_2, v1_0); - DRM_DEBUG("kdb_header_version: %u\n", + drm_dbg(adev_to_drm(adev), "kdb_header_version: %u\n", le32_to_cpu(psp_hdr_v1_2->kdb.fw_version)); - DRM_DEBUG("kdb_offset_bytes: %u\n", + drm_dbg(adev_to_drm(adev), "kdb_offset_bytes: %u\n", le32_to_cpu(psp_hdr_v1_2->kdb.offset_bytes)); - DRM_DEBUG("kdb_size_bytes: %u\n", + drm_dbg(adev_to_drm(adev), "kdb_size_bytes: %u\n", le32_to_cpu(psp_hdr_v1_2->kdb.size_bytes)); } if (version_minor == 3) { @@ -395,23 +426,23 @@ void amdgpu_ucode_print_psp_hdr(struct amdgpu_device *adev, container_of(psp_hdr, struct psp_firmware_header_v1_1, v1_0); const struct psp_firmware_header_v1_3 *psp_hdr_v1_3 = container_of(psp_hdr_v1_1, struct psp_firmware_header_v1_3, v1_1); - DRM_DEBUG("toc_header_version: %u\n", + drm_dbg(adev_to_drm(adev), "toc_header_version: %u\n", le32_to_cpu(psp_hdr_v1_3->v1_1.toc.fw_version)); - DRM_DEBUG("toc_offset_bytes: %u\n", + drm_dbg(adev_to_drm(adev), "toc_offset_bytes: %u\n", le32_to_cpu(psp_hdr_v1_3->v1_1.toc.offset_bytes)); - DRM_DEBUG("toc_size_bytes: %u\n", + drm_dbg(adev_to_drm(adev), "toc_size_bytes: %u\n", le32_to_cpu(psp_hdr_v1_3->v1_1.toc.size_bytes)); - DRM_DEBUG("kdb_header_version: %u\n", + drm_dbg(adev_to_drm(adev), "kdb_header_version: %u\n", le32_to_cpu(psp_hdr_v1_3->v1_1.kdb.fw_version)); - DRM_DEBUG("kdb_offset_bytes: %u\n", + drm_dbg(adev_to_drm(adev), "kdb_offset_bytes: %u\n", le32_to_cpu(psp_hdr_v1_3->v1_1.kdb.offset_bytes)); - DRM_DEBUG("kdb_size_bytes: %u\n", + drm_dbg(adev_to_drm(adev), "kdb_size_bytes: %u\n", le32_to_cpu(psp_hdr_v1_3->v1_1.kdb.size_bytes)); - DRM_DEBUG("spl_header_version: %u\n", + drm_dbg(adev_to_drm(adev), "spl_header_version: %u\n", le32_to_cpu(psp_hdr_v1_3->spl.fw_version)); - DRM_DEBUG("spl_offset_bytes: %u\n", + drm_dbg(adev_to_drm(adev), "spl_offset_bytes: %u\n", le32_to_cpu(psp_hdr_v1_3->spl.offset_bytes)); - DRM_DEBUG("spl_size_bytes: %u\n", + drm_dbg(adev_to_drm(adev), "spl_size_bytes: %u\n", le32_to_cpu(psp_hdr_v1_3->spl.size_bytes)); } } else if (version_major == 2) { @@ -421,72 +452,73 @@ void amdgpu_ucode_print_psp_hdr(struct amdgpu_device *adev, desc = &(psp_hdr_v2_0->psp_fw_bin[fw_index]); switch (desc->fw_type) { case PSP_FW_TYPE_PSP_SOS: - DRM_DEBUG("psp_sos_version: %u\n", + drm_dbg(adev_to_drm(adev), "psp_sos_version: %u\n", le32_to_cpu(desc->fw_version)); - DRM_DEBUG("psp_sos_size_bytes: %u\n", + drm_dbg(adev_to_drm(adev), "psp_sos_size_bytes: %u\n", le32_to_cpu(desc->size_bytes)); break; case PSP_FW_TYPE_PSP_SYS_DRV: - DRM_DEBUG("psp_sys_drv_version: %u\n", + drm_dbg(adev_to_drm(adev), "psp_sys_drv_version: %u\n", le32_to_cpu(desc->fw_version)); - DRM_DEBUG("psp_sys_drv_size_bytes: %u\n", + drm_dbg(adev_to_drm(adev), "psp_sys_drv_size_bytes: %u\n", le32_to_cpu(desc->size_bytes)); break; case PSP_FW_TYPE_PSP_KDB: - DRM_DEBUG("psp_kdb_version: %u\n", + drm_dbg(adev_to_drm(adev), "psp_kdb_version: %u\n", le32_to_cpu(desc->fw_version)); - DRM_DEBUG("psp_kdb_size_bytes: %u\n", + drm_dbg(adev_to_drm(adev), "psp_kdb_size_bytes: %u\n", le32_to_cpu(desc->size_bytes)); break; case PSP_FW_TYPE_PSP_TOC: - DRM_DEBUG("psp_toc_version: %u\n", + drm_dbg(adev_to_drm(adev), "psp_toc_version: %u\n", le32_to_cpu(desc->fw_version)); - DRM_DEBUG("psp_toc_size_bytes: %u\n", + drm_dbg(adev_to_drm(adev), "psp_toc_size_bytes: %u\n", le32_to_cpu(desc->size_bytes)); break; case PSP_FW_TYPE_PSP_SPL: - DRM_DEBUG("psp_spl_version: %u\n", + drm_dbg(adev_to_drm(adev), "psp_spl_version: %u\n", le32_to_cpu(desc->fw_version)); - DRM_DEBUG("psp_spl_size_bytes: %u\n", + drm_dbg(adev_to_drm(adev), "psp_spl_size_bytes: %u\n", le32_to_cpu(desc->size_bytes)); break; case PSP_FW_TYPE_PSP_RL: - DRM_DEBUG("psp_rl_version: %u\n", + drm_dbg(adev_to_drm(adev), "psp_rl_version: %u\n", le32_to_cpu(desc->fw_version)); - DRM_DEBUG("psp_rl_size_bytes: %u\n", + drm_dbg(adev_to_drm(adev), "psp_rl_size_bytes: %u\n", le32_to_cpu(desc->size_bytes)); break; case PSP_FW_TYPE_PSP_SOC_DRV: - DRM_DEBUG("psp_soc_drv_version: %u\n", + drm_dbg(adev_to_drm(adev), "psp_soc_drv_version: %u\n", le32_to_cpu(desc->fw_version)); - DRM_DEBUG("psp_soc_drv_size_bytes: %u\n", + drm_dbg(adev_to_drm(adev), "psp_soc_drv_size_bytes: %u\n", le32_to_cpu(desc->size_bytes)); break; case PSP_FW_TYPE_PSP_INTF_DRV: - DRM_DEBUG("psp_intf_drv_version: %u\n", + drm_dbg(adev_to_drm(adev), "psp_intf_drv_version: %u\n", le32_to_cpu(desc->fw_version)); - DRM_DEBUG("psp_intf_drv_size_bytes: %u\n", + drm_dbg(adev_to_drm(adev), "psp_intf_drv_size_bytes: %u\n", le32_to_cpu(desc->size_bytes)); break; case PSP_FW_TYPE_PSP_DBG_DRV: - DRM_DEBUG("psp_dbg_drv_version: %u\n", + drm_dbg(adev_to_drm(adev), "psp_dbg_drv_version: %u\n", le32_to_cpu(desc->fw_version)); - DRM_DEBUG("psp_dbg_drv_size_bytes: %u\n", + drm_dbg(adev_to_drm(adev), "psp_dbg_drv_size_bytes: %u\n", le32_to_cpu(desc->size_bytes)); break; case PSP_FW_TYPE_PSP_RAS_DRV: - DRM_DEBUG("psp_ras_drv_version: %u\n", + drm_dbg(adev_to_drm(adev), "psp_ras_drv_version: %u\n", le32_to_cpu(desc->fw_version)); - DRM_DEBUG("psp_ras_drv_size_bytes: %u\n", + drm_dbg(adev_to_drm(adev), "psp_ras_drv_size_bytes: %u\n", le32_to_cpu(desc->size_bytes)); break; default: - DRM_DEBUG("Unsupported PSP fw type: %d\n", desc->fw_type); + drm_dbg(adev_to_drm(adev), + "Unsupported PSP fw type: %d\n", desc->fw_type); break; } } } else { - DRM_ERROR("Unknown PSP ucode version: %u.%u\n", + drm_err(adev_to_drm(adev), "Unknown PSP ucode version: %u.%u\n", version_major, version_minor); } } @@ -497,19 +529,20 @@ void amdgpu_ucode_print_gpu_info_hdr(struct amdgpu_device *adev, uint16_t version_major = le16_to_cpu(hdr->header_version_major); uint16_t version_minor = le16_to_cpu(hdr->header_version_minor); - DRM_DEBUG("GPU_INFO\n"); - amdgpu_ucode_print_common_hdr(hdr); + drm_dbg(adev_to_drm(adev), "GPU_INFO\n"); + amdgpu_ucode_print_common_hdr(adev, hdr); if (version_major == 1) { const struct gpu_info_firmware_header_v1_0 *gpu_info_hdr = container_of(hdr, struct gpu_info_firmware_header_v1_0, header); - DRM_DEBUG("version_major: %u\n", + drm_dbg(adev_to_drm(adev), "version_major: %u\n", le16_to_cpu(gpu_info_hdr->version_major)); - DRM_DEBUG("version_minor: %u\n", + drm_dbg(adev_to_drm(adev), "version_minor: %u\n", le16_to_cpu(gpu_info_hdr->version_minor)); } else { - DRM_ERROR("Unknown gpu_info ucode version: %u.%u\n", version_major, version_minor); + drm_err(adev_to_drm(adev), + "Unknown gpu_info ucode version: %u.%u\n", version_major, version_minor); } } @@ -1122,7 +1155,8 @@ int amdgpu_ucode_create_bo(struct amdgpu_device *adev) &adev->firmware.fw_buf_mc, &adev->firmware.fw_buf_ptr); if (!adev->firmware.fw_buf) { - dev_err(adev->dev, "failed to create kernel buffer for firmware.fw_buf\n"); + drm_err(adev_to_drm(adev), + "failed to create kernel buffer for firmware.fw_buf\n"); return -ENOMEM; } else if (amdgpu_sriov_vf(adev)) { memset(adev->firmware.fw_buf_ptr, 0, adev->firmware.fw_size); @@ -1462,7 +1496,7 @@ int amdgpu_ucode_request(struct amdgpu_device *adev, const struct firmware **fw, r = vsnprintf(fname, sizeof(fname), fmt, ap); va_end(ap); if (r == sizeof(fname)) { - dev_warn(adev->dev, "amdgpu firmware name buffer overflow\n"); + drm_warn(adev_to_drm(adev), "amdgpu firmware name buffer overflow\n"); return -EOVERFLOW; } -- 2.43.0