drm/amd/powerplay: Remove extra pair of parentheses
authorMatthias Kaehlcke <mka@chromium.org>
Wed, 7 Feb 2018 19:10:17 +0000 (11:10 -0800)
committerAlex Deucher <alexander.deucher@amd.com>
Mon, 19 Feb 2018 19:19:42 +0000 (14:19 -0500)
The double parentheses are not needed. Removing them fixes the following
warning when building with clang:

drivers/gpu/drm/amd/powerplay/smumgr/tonga_smumgr.c:419:29:
  error: equality comparison with extraneous parentheses
    [-Werror,-Wparentheses-equality]
  if ((data->vdd_gfx_control == SMU7_VOLTAGE_CONTROL_BY_SVID2)) {

Reviewed-by: Guenter Roeck <groeck@chromium.org>
Signed-off-by: Matthias Kaehlcke <mka@chromium.org>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/powerplay/smumgr/tonga_smumgr.c

index 748d98528c0707ca0eeaef85ab56a9d487f276a4..97404a578542c4eb64b6a53b553fe6273b98d36a 100644 (file)
@@ -412,7 +412,7 @@ static int tonga_populate_cac_tables(struct pp_hwmgr *hwmgr,
                        convert_to_vid(vddc_lookup_table->entries[index].us_cac_high);
        }
 
-       if ((data->vdd_gfx_control == SMU7_VOLTAGE_CONTROL_BY_SVID2)) {
+       if (data->vdd_gfx_control == SMU7_VOLTAGE_CONTROL_BY_SVID2) {
                /* We are populating vddgfx CAC data to BapmVddgfx table in split mode */
                for (count = 0; count < vddgfx_level_count; count++) {
                        index = phm_get_voltage_index(vddgfx_lookup_table,