Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

drm/radeon: remove boolean checks in if statements.

Remove unnecessary variable comparisions to true/false in if statements
and check the value of the variable directly.

Signed-off-by: Wambui Karuga <wambui.karugax@gmail.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

authored by

Wambui Karuga and committed by
Alex Deucher
fbd62354 2cacd20e

+16 -16
+1 -1
drivers/gpu/drm/radeon/cik_sdma.c
··· 333 333 u32 me_cntl, reg_offset; 334 334 int i; 335 335 336 - if (enable == false) { 336 + if (!enable) { 337 337 cik_sdma_gfx_stop(rdev); 338 338 cik_sdma_rlc_stop(rdev); 339 339 }
+1 -1
drivers/gpu/drm/radeon/r100.c
··· 2815 2815 uint32_t temp; 2816 2816 2817 2817 temp = RREG32(RADEON_CONFIG_CNTL); 2818 - if (state == false) { 2818 + if (!state) { 2819 2819 temp &= ~RADEON_CFG_VGA_RAM_EN; 2820 2820 temp |= RADEON_CFG_VGA_IO_DIS; 2821 2821 } else {
+1 -1
drivers/gpu/drm/radeon/r600.c
··· 3191 3191 uint32_t temp; 3192 3192 3193 3193 temp = RREG32(CONFIG_CNTL); 3194 - if (state == false) { 3194 + if (!state) { 3195 3195 temp &= ~(1<<0); 3196 3196 temp |= (1<<1); 3197 3197 } else {
+6 -6
drivers/gpu/drm/radeon/radeon_bios.c
··· 664 664 uint16_t tmp; 665 665 666 666 r = radeon_atrm_get_bios(rdev); 667 - if (r == false) 667 + if (!r) 668 668 r = radeon_acpi_vfct_bios(rdev); 669 - if (r == false) 669 + if (!r) 670 670 r = igp_read_bios_from_vram(rdev); 671 - if (r == false) 671 + if (!r) 672 672 r = radeon_read_bios(rdev); 673 - if (r == false) 673 + if (!r) 674 674 r = radeon_read_disabled_bios(rdev); 675 - if (r == false) 675 + if (!r) 676 676 r = radeon_read_platform_bios(rdev); 677 - if (r == false || rdev->bios == NULL) { 677 + if (!r || rdev->bios == NULL) { 678 678 DRM_ERROR("Unable to locate a BIOS ROM\n"); 679 679 rdev->bios = NULL; 680 680 return false;
+2 -2
drivers/gpu/drm/radeon/radeon_connectors.c
··· 440 440 if (radeon_conflict->use_digital) 441 441 continue; 442 442 443 - if (priority == true) { 443 + if (priority) { 444 444 DRM_DEBUG_KMS("1: conflicting encoders switching off %s\n", 445 445 conflict->name); 446 446 DRM_DEBUG_KMS("in favor of %s\n", ··· 700 700 else 701 701 ret = radeon_legacy_get_tmds_info_from_combios(radeon_encoder, tmds); 702 702 } 703 - if (val == 1 || ret == false) { 703 + if (val == 1 || !ret) { 704 704 radeon_legacy_get_tmds_info_from_table(radeon_encoder, tmds); 705 705 } 706 706 radeon_property_change_mode(&radeon_encoder->base);
+2 -2
drivers/gpu/drm/radeon/radeon_display.c
··· 847 847 if (rdev->bios) { 848 848 if (rdev->is_atom_bios) { 849 849 ret = radeon_get_atom_connector_info_from_supported_devices_table(dev); 850 - if (ret == false) 850 + if (!ret) 851 851 ret = radeon_get_atom_connector_info_from_object_table(dev); 852 852 } else { 853 853 ret = radeon_get_legacy_connector_info_from_bios(dev); 854 - if (ret == false) 854 + if (!ret) 855 855 ret = radeon_get_legacy_connector_info_from_table(dev); 856 856 } 857 857 } else {
+2 -2
drivers/gpu/drm/radeon/radeon_legacy_encoders.c
··· 1712 1712 else 1713 1713 ret = radeon_legacy_get_tmds_info_from_combios(encoder, tmds); 1714 1714 1715 - if (ret == false) 1715 + if (!ret) 1716 1716 radeon_legacy_get_tmds_info_from_table(encoder, tmds); 1717 1717 1718 1718 return tmds; ··· 1735 1735 1736 1736 ret = radeon_legacy_get_ext_tmds_info_from_combios(encoder, tmds); 1737 1737 1738 - if (ret == false) 1738 + if (!ret) 1739 1739 radeon_legacy_get_ext_tmds_info_from_table(encoder, tmds); 1740 1740 1741 1741 return tmds;
+1 -1
drivers/gpu/drm/radeon/radeon_pm.c
··· 1789 1789 u32 stat_crtc = 0; 1790 1790 bool in_vbl = radeon_pm_in_vbl(rdev); 1791 1791 1792 - if (in_vbl == false) 1792 + if (!in_vbl) 1793 1793 DRM_DEBUG_DRIVER("not in vbl for pm change %08x at %s\n", stat_crtc, 1794 1794 finish ? "exit" : "entry"); 1795 1795 return in_vbl;