drm/amd/powerplay: fix bug that NULL checks are reversed.
authorRex Zhu <Rex.Zhu@amd.com>
Wed, 6 Jan 2016 08:22:07 +0000 (16:22 +0800)
committerAlex Deucher <alexander.deucher@amd.com>
Fri, 8 Jan 2016 20:39:22 +0000 (15:39 -0500)
&& was used instead of ||.

Signed-off-by: Rex Zhu <Rex.Zhu@amd.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Reviewed-by: Ken Wang <Qingqing.Wang@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/powerplay/hwmgr/cz_hwmgr.c

index 5bac36baa13c9f029ba6b7feaed09efe791e417c..c0e6aae8fa1db0454b744d62d5aa3b691ef2df4f 100644 (file)
@@ -579,7 +579,7 @@ static int cz_tf_init_sclk_limit(struct pp_hwmgr *hwmgr, void *input,
                                        hwmgr->dyn_state.vddc_dependency_on_sclk;
        unsigned long clock = 0, level;
 
-       if (NULL == table && table->count <= 0)
+       if (NULL == table || table->count <= 0)
                return -EINVAL;
 
        cz_hwmgr->sclk_dpm.soft_min_clk = table->entries[0].clk;
@@ -606,7 +606,7 @@ static int cz_tf_init_uvd_limit(struct pp_hwmgr *hwmgr, void *input,
                                hwmgr->dyn_state.uvd_clock_voltage_dependency_table;
        unsigned long clock = 0, level;
 
-       if (NULL == table && table->count <= 0)
+       if (NULL == table || table->count <= 0)
                return -EINVAL;
 
        cz_hwmgr->uvd_dpm.soft_min_clk = 0;
@@ -634,7 +634,7 @@ static int cz_tf_init_vce_limit(struct pp_hwmgr *hwmgr, void *input,
                                hwmgr->dyn_state.vce_clock_voltage_dependency_table;
        unsigned long clock = 0, level;
 
-       if (NULL == table && table->count <= 0)
+       if (NULL == table || table->count <= 0)
                return -EINVAL;
 
        cz_hwmgr->vce_dpm.soft_min_clk = 0;
@@ -662,7 +662,7 @@ static int cz_tf_init_acp_limit(struct pp_hwmgr *hwmgr, void *input,
                                hwmgr->dyn_state.acp_clock_voltage_dependency_table;
        unsigned long clock = 0, level;
 
-       if (NULL == table && table->count <= 0)
+       if (NULL == table || table->count <= 0)
                return -EINVAL;
 
        cz_hwmgr->acp_dpm.soft_min_clk = 0;
@@ -1183,7 +1183,7 @@ int cz_phm_unforce_dpm_levels(struct pp_hwmgr *hwmgr)
                                hwmgr->dyn_state.vddc_dependency_on_sclk;
        unsigned long clock = 0, level;
 
-       if (NULL == table && table->count <= 0)
+       if (NULL == table || table->count <= 0)
                return -EINVAL;
 
        cz_hwmgr->sclk_dpm.soft_min_clk = table->entries[0].clk;