Re: [PATCH v2 3/3] drm/amd/powerplay: Place the constant on the right side of the test

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux