RE: [PATCH 1/4 V2] drm/amdgpu: fix invadate operation for umsch

2024-05-22 Thread Yu, Lang
:20 PM >> To: Zhang, Jesse(Jie) ; >> amd-gfx@lists.freedesktop.org >> Cc: Deucher, Alexander ; Koenig, Christian >> ; Huang, Tim ; Yu, >Lang >> >> Subject: Re: [PATCH 1/4 V2] drm/amdgpu: fix invadate operation for >> umsch >> >> >> >&

Re: [PATCH 1/4 V2] drm/amdgpu: fix invadate operation for umsch

2024-05-21 Thread Lazar, Lijo
> Cc: Deucher, Alexander ; Koenig, Christian > ; Huang, Tim ; Yu, Lang > > Subject: Re: [PATCH 1/4 V2] drm/amdgpu: fix invadate operation for umsch > > > > On 5/21/2024 12:46 PM, Jesse Zhang wrote: >> Since the type of data_size is uint32_t, adev->umsch_mm.data_si

RE: [PATCH 1/4 V2] drm/amdgpu: fix invadate operation for umsch

2024-05-21 Thread Zhang, Jesse(Jie)
V2] drm/amdgpu: fix invadate operation for umsch On 5/21/2024 12:46 PM, Jesse Zhang wrote: > Since the type of data_size is uint32_t, adev->umsch_mm.data_size - 1 > >> 16 >> 16 is 0 regardless of the values of its operands > > So removing the operations upper_32_bits an

Re: [PATCH 1/4 V2] drm/amdgpu: fix invadate operation for umsch

2024-05-21 Thread Lazar, Lijo
On 5/21/2024 12:46 PM, Jesse Zhang wrote: > Since the type of data_size is uint32_t, adev->umsch_mm.data_size - 1 >> 16 > >> 16 is 0 > regardless of the values of its operands > > So removing the operations upper_32_bits and lower_32_bits. > > Signed-off-by: Jesse Zhang > Suggested-by: Tim