On Tue, Oct 17, 2017 at 4:22 PM, Georgiana Chelu <georgiana.chelu93@xxxxxxxxx> wrote: > Move the constant on the right side of the comparison in order to > make the code easier to read. > > Issue found by checkpatch script: > * WARNING: Comparisons should place the constant on the right side of > the test > > Signed-off-by: Georgiana Chelu <georgiana.chelu93@xxxxxxxxx> Applied the series. Thanks you! Alex > --- > > Changes in v2: > * adjusted the subject prefix of the patches > > drivers/gpu/drm/amd/powerplay/hwmgr/vega10_processpptables.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_processpptables.c b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_processpptables.c > index 769ac11a9215..f14c7611fad3 100644 > --- a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_processpptables.c > +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_processpptables.c > @@ -1021,7 +1021,7 @@ static int get_vddc_lookup_table( > > table = kzalloc(table_size, GFP_KERNEL); > > - if (NULL == table) > + if (table == NULL) > return -ENOMEM; > > table->count = vddc_lookup_pp_tables->ucNumEntries; > @@ -1130,12 +1130,12 @@ int vega10_pp_tables_initialize(struct pp_hwmgr *hwmgr) > > hwmgr->pptable = kzalloc(sizeof(struct phm_ppt_v2_information), GFP_KERNEL); > > - PP_ASSERT_WITH_CODE((NULL != hwmgr->pptable), > + PP_ASSERT_WITH_CODE((hwmgr->pptable != NULL), > "Failed to allocate hwmgr->pptable!", return -ENOMEM); > > powerplay_table = get_powerplay_table(hwmgr); > > - PP_ASSERT_WITH_CODE((NULL != powerplay_table), > + PP_ASSERT_WITH_CODE((powerplay_table != NULL), > "Missing PowerPlay Table!", return -1); > > result = check_powerplay_tables(hwmgr, powerplay_table); > @@ -1229,7 +1229,7 @@ int vega10_get_number_of_powerplay_table_entries(struct pp_hwmgr *hwmgr) > const ATOM_Vega10_State_Array *state_arrays; > const ATOM_Vega10_POWERPLAYTABLE *pp_table = get_powerplay_table(hwmgr); > > - PP_ASSERT_WITH_CODE((NULL != pp_table), > + PP_ASSERT_WITH_CODE((pp_table != NULL), > "Missing PowerPlay Table!", return -1); > PP_ASSERT_WITH_CODE((pp_table->sHeader.format_revision >= > ATOM_Vega10_TABLE_REVISION_VEGA10), > -- > 2.11.0 > > _______________________________________________ > dri-devel mailing list > dri-devel@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/dri-devel _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
