A spelling typo in subject, s/admgpu/amdgpu :)
Also maybe it's necessary to add a body text in commit message.
Regards,
Guchun
> -----Original Message-----
> From: amd-gfx <amd-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx> On Behalf Of
> Stanley.Yang
> Sent: Monday, April 10, 2023 7:48 PM
> To: amd-gfx@xxxxxxxxxxxxxxxxxxxxx; Zhang, Hawking
> <Hawking.Zhang@xxxxxxx>; Zhou1, Tao <Tao.Zhou1@xxxxxxx>
> Cc: Yang, Stanley <Stanley.Yang@xxxxxxx>
> Subject: [PATCH Review 1/2] drm/admgpu: fix unexpected block id
>
> Change-Id: Icceb43556eec802f11c2077c1c58a1e92c9df599
> Signed-off-by: Stanley.Yang <Stanley.Yang@xxxxxxx>
> ---
> drivers/gpu/drm/amd/amdgpu/amdgpu_ras.h | 4 ++++
> drivers/gpu/drm/amd/amdgpu/ta_ras_if.h | 2 ++
> 2 files changed, 6 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.h
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.h
> index ef38f4c93df0..17b3d1992e80 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.h
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.h
> @@ -583,6 +583,10 @@ amdgpu_ras_block_to_ta(enum amdgpu_ras_block
> block) {
> return TA_RAS_BLOCK__FUSE;
> case AMDGPU_RAS_BLOCK__MCA:
> return TA_RAS_BLOCK__MCA;
> + case AMDGPU_RAS_BLOCK__VCN:
> + return TA_RAS_BLOCK__VCN;
> + case AMDGPU_RAS_BLOCK__JPEG:
> + return TA_RAS_BLOCK__JPEG;
> default:
> WARN_ONCE(1, "RAS ERROR: unexpected block id %d\n",
> block);
> return TA_RAS_BLOCK__UMC;
> diff --git a/drivers/gpu/drm/amd/amdgpu/ta_ras_if.h
> b/drivers/gpu/drm/amd/amdgpu/ta_ras_if.h
> index 509d8a1945eb..30d0482ac466 100644
> --- a/drivers/gpu/drm/amd/amdgpu/ta_ras_if.h
> +++ b/drivers/gpu/drm/amd/amdgpu/ta_ras_if.h
> @@ -84,6 +84,8 @@ enum ta_ras_block {
> TA_RAS_BLOCK__MP1,
> TA_RAS_BLOCK__FUSE,
> TA_RAS_BLOCK__MCA,
> + TA_RAS_BLOCK__VCN,
> + TA_RAS_BLOCK__JPEG,
> TA_NUM_BLOCK_MAX
> };
>
> --
> 2.17.1
