On Tue, Sep 15, 2020 at 03:16:32PM -0400, Alex Deucher wrote:
I question the value of these warnings. Why even have a boolean typeif (this_is_sparta)
if you are going to get warnings when you use them...
That said, applied to avoid getting these patches again and again
every time someone sees this.
if (this_is_sparta == true)
if (this_is_sparta != false)
I think the first one reads the best, and avoids having to
decide between truth and falsehood :)
Alex
On Wed, Sep 9, 2020 at 9:21 AM Christian König <christian.koenig@xxxxxxx> wrote:
Acked-by: Christian König <christian.koenig@xxxxxxx> for the series._______________________________________________
Am 09.09.20 um 15:07 schrieb Zheng Bin:
Zheng Bin (8):_______________________________________________
drm/amd/amdgpu: fix comparison pointer to bool warning in gfx_v9_0.c
drm/amd/amdgpu: fix comparison pointer to bool warning in gfx_v10_0.c
drm/amd/amdgpu: fix comparison pointer to bool warning in sdma_v5_0.c
drm/amd/amdgpu: fix comparison pointer to bool warning in sdma_v5_2.c
drm/amd/amdgpu: fix comparison pointer to bool warning in si.c
drm/amd/amdgpu: fix comparison pointer to bool warning in uvd_v6_0.c
drm/amd/amdgpu: fix comparison pointer to bool warning in
amdgpu_atpx_handler.c
drm/amd/amdgpu: fix comparison pointer to bool warning in sdma_v4_0.c
drivers/gpu/drm/amd/amdgpu/amdgpu_atpx_handler.c | 4 ++--
drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c | 2 +-
drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 2 +-
drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c | 4 ++--
drivers/gpu/drm/amd/amdgpu/sdma_v5_0.c | 2 +-
drivers/gpu/drm/amd/amdgpu/sdma_v5_2.c | 2 +-
drivers/gpu/drm/amd/amdgpu/si.c | 2 +-
drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c | 4 ++--
8 files changed, 11 insertions(+), 11 deletions(-)
--
2.26.0.106.g9fadedd
amd-gfx mailing list
amd-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel